Merge pull request #409 from ShawnJeffersonWang/master

fix: correct argument length check in FlagSet.Parse

2 files changed, 7 insertions, 1 deletions

diff --git a/flag.go b/flag.go
index 64d268a..0b5be7a 100644
--- a/flag.go
+++ b/flag.go
@@ -1156,7 +1156,7 @@ func (f *FlagSet) Parse(arguments []string) error {
 	}
 	f.parsed = true
 
-	if len(arguments) < 0 {
+	if len(arguments) == 0 {
 		return nil
 	}
 
diff --git a/flag_test.go b/flag_test.go
index aa2f434..ead0518 100644
--- a/flag_test.go
+++ b/flag_test.go
@@ -100,6 +100,12 @@ func TestEverything(t *testing.T) {
 	}
 }
 
+func TestNoArgument(t *testing.T) {
+	if GetCommandLine().Parse([]string{}) != nil {
+		t.Error("parse failed for empty argument list")
+	}
+}
+
 func TestUsage(t *testing.T) {
 	called := false
 	ResetForTesting(func() { called = true })