Fixes bug with handling of empty identifier

Author: tomnomnom

.travis.yml

@@ -2,4 +2,6 @@ language: go
 
 go:
   - 1.7
+  - 1.8
+  - 1.9
   - tip

identifier.go

@@ -49,7 +49,7 @@ var reservedWords = map[string]bool{
 //     a key with spaces      -> false
 //     1startsWithANumber	  -> false
 func validIdentifier(s string) bool {
-	if reservedWords[s] {
+	if reservedWords[s] || s == "" {
 		return false
 	}
 

identifier_test.go

@@ -21,6 +21,9 @@ func TestValidIdentifier(t *testing.T) {
 		{"true", false},
 		{"else", false},
 		{"null", false},
+
+		// Empty string
+		{"", false},
 	}
 
 	for _, test := range tests {

script/precommit

@@ -4,4 +4,3 @@ PROJDIR=$(cd `dirname $0`/.. && pwd)
 cd ${PROJDIR}
 
 ./script/test
-./script/lint

statements_test.go

@@ -29,7 +29,8 @@ func TestStatementsSimple(t *testing.T) {
 			"foo": "bar"
 		},
 		"else": 1,
-		"id": 66912849
+		"id": 66912849,
+		"": 2
 	}`)
 
 	ss, err := statementsFromJSON(bytes.NewReader(j), statement{{"json", typBare}})
@@ -52,6 +53,7 @@ func TestStatementsSimple(t *testing.T) {
 		`json.anob.foo = "bar";`,
 		`json["else"] = 1;`,
 		`json.id = 66912849;`,
+		`json[""] = 2;`,
 	})
 
 	t.Logf("Have: %#v", ss)