Fixed bug #11: Inability to set field to empty string

Mike Farah · Mike Farah · commit cdd78af4bc37 · 2017-06-06T10:19:32.000+10:00
diff --git a/yaml.go b/yaml.go
@@ -195,7 +195,7 @@ func updateYaml(args []string) interface{} {
 
 func parseValue(argument string) interface{} {
 	var value, err interface{}
-	var inQuotes = argument[0] == '"'
+	var inQuotes = len(argument) > 0 && argument[0] == '"'
 	if !inQuotes {
 		value, err = strconv.ParseFloat(argument, 64)
 		if err == nil {
diff --git a/yaml_test.go b/yaml_test.go
@@ -14,6 +14,7 @@ var parseValueTests = []struct {
 	{"\"true\"", "true", "boolean as string"},
 	{"3.4", 3.4, "number"},
 	{"\"3.4\"", "3.4", "number as string"},
+	{"", "", "empty string"},
 }
 
 func TestParseValue(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@ var parseValueTests = []struct {`
`14`	`14`	`{"\"true\"", "true", "boolean as string"},`
`15`	`15`	`{"3.4", 3.4, "number"},`
`16`	`16`	`{"\"3.4\"", "3.4", "number as string"},`
	`17`	`+ {"", "", "empty string"},`
`17`	`18`	`}`
`18`	`19`
`19`	`20`	`func TestParseValue(t *testing.T) {`