compiler: support nested string flags definitions

This commit adds support for flags definitions such as:

    flags1 = "string1", "string2"
    flags2 = flags1, "string3"

Signed-off-by: Paul Chaignon <paul.chaignon@gmail.com>

1 files changed, 15 insertions, 0 deletions

diff --git a/pkg/compiler/compiler.go b/pkg/compiler/compiler.go
index 66da5e2b4..34b8b1fd6 100644
--- a/pkg/compiler/compiler.go
+++ b/pkg/compiler/compiler.go
@@ -361,6 +361,7 @@ func arrayContains(a []string, v string) bool {
 
 func (comp *compiler) flattenFlags() {
 	comp.flattenIntFlags()
+	comp.flattenStrFlags()
 
 	for _, n := range comp.desc.Nodes {
 		switch flags := n.(type) {
@@ -371,6 +372,11 @@ func (comp *compiler) flattenFlags() {
 			if f, ok := comp.intFlags[flags.Name.Name]; ok {
 				flags.Values = f.Values
 			}
+		case *ast.StrFlags:
+			// Same as for intFlags above.
+			if f, ok := comp.strFlags[flags.Name.Name]; ok {
+				flags.Values = f.Values
+			}
 		}
 	}
 }
@@ -384,6 +390,15 @@ func (comp *compiler) flattenIntFlags() {
 	}
 }
 
+func (comp *compiler) flattenStrFlags() {
+	for name, flags := range comp.strFlags {
+		if err := recurFlattenFlags[*ast.StrFlags, *ast.String](comp, name, flags, comp.strFlags,
+			map[string]bool{}); err != nil {
+			comp.error(flags.Pos, "%v", err)
+		}
+	}
+}
+
 func recurFlattenFlags[F ast.Flags[V], V ast.FlagValue](comp *compiler, name string, flags F,
 	allFlags map[string]F, visitedFlags map[string]bool) error {
 	if _, visited := visitedFlags[name]; visited {