wit
/
arg
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

simplify processing of positionals a little

This commit is contained in:
Alex Flint 2019-04-14 17:30:53 -07:00
parent 6266d3e5b7
commit 7b1d9ef23f
1 changed files with 17 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
parse.go
View File

@ -383,25 +383,24 @@ func process(specs []*spec, args []string) error {
// process positionals // process positionals
for _, spec := range specs { for _, spec := range specs {
if spec.positional { if !spec.positional {
if spec.multiple { continue
if spec.required && len(positionals) == 0 { }
return fmt.Errorf("%s is required", spec.long) if spec.required && len(positionals) == 0 {
} return fmt.Errorf("%s is required", spec.long)
err := setSlice(spec.dest, positionals, true) }
if err != nil { if spec.multiple {
return fmt.Errorf("error processing %s: %v", spec.long, err) err := setSlice(spec.dest, positionals, true)
} if err != nil {
positionals = nil return fmt.Errorf("error processing %s: %v", spec.long, err)
} else if len(positionals) > 0 {
err := scalar.ParseValue(spec.dest, positionals[0])
if err != nil {
return fmt.Errorf("error processing %s: %v", spec.long, err)
}
positionals = positionals[1:]
} else if spec.required {
return fmt.Errorf("%s is required", spec.long)
} }
positionals = nil
} else if len(positionals) > 0 {
err := scalar.ParseValue(spec.dest, positionals[0])
if err != nil {
return fmt.Errorf("error processing %s: %v", spec.long, err)
}
positionals = positionals[1:]
} }
} }
if len(positionals) > 0 { if len(positionals) > 0 {