From 794a36280887640cc246dfcd279bd85dfd91f433 Mon Sep 17 00:00:00 2001 From: Eric Paris Date: Mon, 6 Apr 2015 15:24:36 -0400 Subject: [PATCH] Remove specail casing in Execute() We had some stuff that created a new empty []string if args was already and empty string (why?) We had some stuff that called Help() if it wasn't runnable (but .execute() already does that) Just remove the special case stuff. --- command.go | 17 +++-------------- 1 file changed, 3 insertions(+), 14 deletions(-) diff --git a/command.go b/command.go index 9c3a7013..d05e05ea 100644 --- a/command.go +++ b/command.go @@ -476,20 +476,9 @@ func (c *Command) Execute() (err error) { args = c.args } - if len(args) == 0 { - // Only the executable is called and the root is runnable, run it - if c.Runnable() { - err = c.execute([]string(nil)) - } else { - c.Help() - } - } else { - cmd, flags, e := c.Find(args) - if e != nil { - err = e - } else { - err = cmd.execute(flags) - } + cmd, flags, err := c.Find(args) + if err == nil { + err = cmd.execute(flags) } if err != nil {