Improve error when failing to parse float fractional or exponent

byroot · byroot · commit a1cdac180f1a · 2025-10-27T11:03:15.000+01:00
diff --git a/ext/json/ext/parser/parser.c b/ext/json/ext/parser/parser.c
@@ -1135,7 +1135,7 @@ static VALUE json_parse_any(JSON_ParserState *state, JSON_ParserConfig *config)
                 state->cursor++;
 
                 if (state->cursor == state->end || !rb_isdigit(*state->cursor)) {
-                    raise_parse_error("invalid number: %s", state);
+                    raise_parse_error_at("invalid number: %s", state, start);
                 }
 
                 while ((state->cursor < state->end) && rb_isdigit(*state->cursor)) {
@@ -1157,7 +1157,7 @@ static VALUE json_parse_any(JSON_ParserState *state, JSON_ParserConfig *config)
                 }
 
                 if (state->cursor == state->end || !rb_isdigit(*state->cursor)) {
-                    raise_parse_error("invalid number: %s", state);
+                    raise_parse_error_at("invalid number: %s", state, start);
                 }
 
                 while ((state->cursor < state->end) && rb_isdigit(*state->cursor)) {

Original file line number	Diff line number	Diff line change
`@@ -1135,7 +1135,7 @@ static VALUE json_parse_any(JSON_ParserState state, JSON_ParserConfig config)`
`1135`	`1135`	`state->cursor++;`
`1136`	`1136`
`1137`	`1137`	`if (state->cursor == state->end \|\| !rb_isdigit(*state->cursor)) {`
`1138`		`- raise_parse_error("invalid number: %s", state);`
	`1138`	`+ raise_parse_error_at("invalid number: %s", state, start);`
`1139`	`1139`	`}`
`1140`	`1140`
`1141`	`1141`	`while ((state->cursor < state->end) && rb_isdigit(*state->cursor)) {`
`@@ -1157,7 +1157,7 @@ static VALUE json_parse_any(JSON_ParserState state, JSON_ParserConfig config)`
`1157`	`1157`	`}`
`1158`	`1158`
`1159`	`1159`	`if (state->cursor == state->end \|\| !rb_isdigit(*state->cursor)) {`
`1160`		`- raise_parse_error("invalid number: %s", state);`
	`1160`	`+ raise_parse_error_at("invalid number: %s", state, start);`
`1161`	`1161`	`}`
`1162`	`1162`
`1163`	`1163`	`while ((state->cursor < state->end) && rb_isdigit(*state->cursor)) {`