lang: funcs: funcgen: Don't nest unnecessarily
Clean up the code.
This commit is contained in:
@@ -101,7 +101,11 @@ func (obj *golangPackage) parsefuncs() (functions, error) {
|
|||||||
func (obj *golangPackage) extractFuncs(doc string, getHelp bool) (functions, error) {
|
func (obj *golangPackage) extractFuncs(doc string, getHelp bool) (functions, error) {
|
||||||
var funcs []function
|
var funcs []function
|
||||||
for _, line := range strings.Split(doc, "\n") {
|
for _, line := range strings.Split(doc, "\n") {
|
||||||
if validSignature.MatchString(line) {
|
if !validSignature.MatchString(line) {
|
||||||
|
// TODO: improve validSignature regexp for this?
|
||||||
|
fmt.Printf("invalid: %s\n", line)
|
||||||
|
continue
|
||||||
|
}
|
||||||
f, err := obj.parseFunctionLine(line, getHelp)
|
f, err := obj.parseFunctionLine(line, getHelp)
|
||||||
if err != nil && err != errExcluded {
|
if err != nil && err != errExcluded {
|
||||||
return funcs, err
|
return funcs, err
|
||||||
@@ -110,7 +114,6 @@ func (obj *golangPackage) extractFuncs(doc string, getHelp bool) (functions, err
|
|||||||
funcs = append(funcs, *f)
|
funcs = append(funcs, *f)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
|
|
||||||
return funcs, nil
|
return funcs, nil
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user