fix: Fix the issue where the isNumber method incorrectly identifies Inf as a number, causing the parser to misinterpret the Inf identifier. (#146)

Author: wertycn

parser_test.go

@@ -229,3 +229,29 @@ enum Value {
 		t.Fatal(err) // <input>:7:16: found "-" but expected [range integer]
 	}
 }
+
+func TestParseInfMessage(t *testing.T) {
+	const def = `
+message Inf {
+	string field = 1; 
+}
+message NaN {
+	string field = 1; 
+}
+
+message Infinity {
+	string field = 1; 
+}
+message ExampelMessage {
+	Inf inf_field = 1;
+	NaN nan_field = 2; 
+	Infinity infinity_field = 3; 
+}
+`
+
+	p := NewParser(strings.NewReader(def))
+	_, err := p.Parse()
+	if err != nil {
+		t.Fatal(err) // <input>:7:16: found "-" but expected [range integer]
+	}
+}

token.go

@@ -124,6 +124,9 @@ func isComment(lit string) bool {
 }
 
 func isNumber(lit string) bool {
+	if lit == "NaN" || lit == "nan" || lit == "Inf" || lit == "Infinity" || lit == "inf" || lit == "infinity" {
+		return false
+	}
 	if strings.HasPrefix(lit, "0x") || strings.HasPrefix(lit, "0X") {
 		_, err := strconv.ParseInt(lit, 0, 64)
 		return err == nil

token_test.go

@@ -63,6 +63,12 @@ func TestIsNumber(t *testing.T) {
 		{`a1`, false},
 		{`0x12`, true},
 		{`0X77777`, true},
+		{`NaN`, false},
+		{`nan`, false},
+		{`Inf`, false},
+		{`Infinity`, false},
+		{`inf`, false},
+		{`infinity`, false},
 	} {
 		got := isNumber(each.input)
 		if got != each.isNumber {