Added comment to generated ...String function. Removed usage of _ in identifiers.

Author: wlbr

enumer.go

@@ -4,8 +4,10 @@ import "fmt"
 
 // Arguments to format are:
 //	[1]: type name
-const stringValueToNameMap = `func %[1]sString(s string) (%[1]s, error) {
-	if val, ok := _%[1]sNameToValue_map[s]; ok {
+const stringValueToNameMap = `// %[1]sString retrieves an enum value from the enum constants string name.
+// Throws an error if the param is not part of the enum. 
+func %[1]sString(s string) (%[1]s, error) {
+	if val, ok := _%[1]sNameToValueMap[s]; ok {
 		return val, nil
 	}
 	return 0, fmt.Errorf("%%s does not belong to %[1]s values", s)
@@ -14,7 +16,7 @@ const stringValueToNameMap = `func %[1]sString(s string) (%[1]s, error) {
 
 func (g *Generator) buildValueToNameMap(runs [][]Value, typeName string, runsThreshold int) {
 	// At this moment, either "g.declareIndexAndNameVars()" or "g.declareNameVars()" has been called
-	g.Printf("\nvar _%sNameToValue_map = map[string]%s{\n", typeName, typeName)
+	g.Printf("\nvar _%sNameToValueMap = map[string]%s{\n", typeName, typeName)
 	thereAreRuns := len(runs) > 1 && len(runs) <= runsThreshold
 	var n int
 	var runID string
@@ -27,7 +29,7 @@ func (g *Generator) buildValueToNameMap(runs [][]Value, typeName string, runsThr
 		}
 
 		for _, value := range values {
-			g.Printf("\t_%s_name%s[%d:%d]: %s,\n", typeName, runID, n, n+len(value.name), &value)
+			g.Printf("\t_%sName%s[%d:%d]: %s,\n", typeName, runID, n, n+len(value.name), &value)
 			n += len(value.name)
 		}
 	}