optimize Traverse to avoid fixing the option name twice

xgfone · Aug 24, 2019 · d0e6b91 · d0e6b91
1 parent 3e5c7ca
commit d0e6b91
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/config.go b/config.go
@@ -337,7 +337,8 @@ func (c *Config) PrintGroup(w io.Writer) error {
 func (c *Config) Traverse(f func(group string, opt string, value interface{})) {
 	for _, group := range c.AllGroups() {
 		for _, opt := range group.AllOpts() {
-			f(group.Name(), group.fixOptName(opt.Name), group.Get(opt.Name))
+			name := group.fixOptName(opt.Name)
+			f(group.Name(), name, group.Get(name))
 		}
 	}
 }