github
/
ruby
зеркало из https://github.com/github/ruby.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

[flori/json] Stop including the parser source __LINE__ in exceptions 

It makes testing for JSON errors very tedious. You either have
to use a Regexp or to regularly update all your assertions
when JSON is upgraded.

https://github.com/flori/json/commit/de9eb1d28e
This commit is contained in:
Jean Boussier 2021-05-20 12:40:04 +02:00 коммит произвёл Hiroshi SHIBATA
Родитель 4bf97a8ec4
Коммит 66b52f046f
3 изменённых файлов: 16 добавлений и 16 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

4
ext/json/generator/generator.c
Просмотреть файл

@ -997,10 +997,10 @@ static void generate_json_float(FBuffer *buffer, VALUE Vstate, JSON_Generator_St
if (!allow_nan) { if (!allow_nan) {
if (isinf(value)) { if (isinf(value)) {
fbuffer_free(buffer); fbuffer_free(buffer);
rb_raise(eGeneratorError, "%u: %"PRIsVALUE" not allowed in JSON", __LINE__, RB_OBJ_STRING(tmp)); rb_raise(eGeneratorError, "%"PRIsVALUE" not allowed in JSON", RB_OBJ_STRING(tmp));
} else if (isnan(value)) { } else if (isnan(value)) {
fbuffer_free(buffer); fbuffer_free(buffer);
rb_raise(eGeneratorError, "%u: %"PRIsVALUE" not allowed in JSON", __LINE__, RB_OBJ_STRING(tmp)); rb_raise(eGeneratorError, "%"PRIsVALUE" not allowed in JSON", RB_OBJ_STRING(tmp));
} }
} }
fbuffer_append_str(buffer, tmp); fbuffer_append_str(buffer, tmp);

14
ext/json/parser/parser.c
Просмотреть файл

@ -948,7 +948,7 @@ static char *JSON_parse_value(JSON_Parser *json, char *p, char *pe, VALUE *resul
{p = p - 1; } {p+= 1; cs = 29; goto _out;} {p = p - 1; } {p+= 1; cs = 29; goto _out;}
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
} }
} }
np = JSON_parse_float(json, p, pe, result); np = JSON_parse_float(json, p, pe, result);
@ -990,7 +990,7 @@ static char *JSON_parse_value(JSON_Parser *json, char *p, char *pe, VALUE *resul
if (json->allow_nan) { if (json->allow_nan) {
*result = CInfinity; *result = CInfinity;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p - 8); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p - 8);
} }
} }
@ -1002,7 +1002,7 @@ static char *JSON_parse_value(JSON_Parser *json, char *p, char *pe, VALUE *resul
if (json->allow_nan) { if (json->allow_nan) {
*result = CNaN; *result = CNaN;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p - 2); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p - 2);
} }
} }
@ -2348,7 +2348,7 @@ static char *JSON_parse_array(JSON_Parser *json, char *p, char *pe, VALUE *resul
if(cs >= JSON_array_first_final) { if(cs >= JSON_array_first_final) {
return p + 1; return p + 1;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
return NULL; return NULL;
} }
} }
@ -2413,7 +2413,7 @@ static VALUE json_string_unescape(char *string, char *stringEnd, int intern, int
} }
rb_enc_raise( rb_enc_raise(
EXC_ENCODING eParserError, EXC_ENCODING eParserError,
"%u: incomplete unicode character escape sequence at '%s'", __LINE__, p "incomplete unicode character escape sequence at '%s'", p
); );
} else { } else {
UTF32 ch = unescape_unicode((unsigned char *) ++pe); UTF32 ch = unescape_unicode((unsigned char *) ++pe);
@ -2426,7 +2426,7 @@ static VALUE json_string_unescape(char *string, char *stringEnd, int intern, int
} }
rb_enc_raise( rb_enc_raise(
EXC_ENCODING eParserError, EXC_ENCODING eParserError,
"%u: incomplete surrogate pair at '%s'", __LINE__, p "incomplete surrogate pair at '%s'", p
); );
} }
if (pe[0] == '\\' && pe[1] == 'u') { if (pe[0] == '\\' && pe[1] == 'u') {
@ -3225,7 +3225,7 @@ static VALUE cParser_parse(VALUE self)
if (cs >= JSON_first_final && p == pe) { if (cs >= JSON_first_final && p == pe) {
return result; return result;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
return Qnil; return Qnil;
} }
} }

14
ext/json/parser/parser.rl
Просмотреть файл

@ -222,14 +222,14 @@ static char *JSON_parse_object(JSON_Parser *json, char *p, char *pe, VALUE *resu
if (json->allow_nan) { if (json->allow_nan) {
*result = CNaN; *result = CNaN;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p - 2); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p - 2);
} }
} }
action parse_infinity { action parse_infinity {
if (json->allow_nan) { if (json->allow_nan) {
*result = CInfinity; *result = CInfinity;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p - 8); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p - 8);
} }
} }
action parse_string { action parse_string {
@ -245,7 +245,7 @@ static char *JSON_parse_object(JSON_Parser *json, char *p, char *pe, VALUE *resu
fexec p + 10; fexec p + 10;
fhold; fbreak; fhold; fbreak;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
} }
} }
np = JSON_parse_float(json, fpc, pe, result); np = JSON_parse_float(json, fpc, pe, result);
@ -447,7 +447,7 @@ static char *JSON_parse_array(JSON_Parser *json, char *p, char *pe, VALUE *resul
if(cs >= JSON_array_first_final) { if(cs >= JSON_array_first_final) {
return p + 1; return p + 1;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
return NULL; return NULL;
} }
} }
@ -512,7 +512,7 @@ static VALUE json_string_unescape(char *string, char *stringEnd, int intern, int
} }
rb_enc_raise( rb_enc_raise(
EXC_ENCODING eParserError, EXC_ENCODING eParserError,
"%u: incomplete unicode character escape sequence at '%s'", __LINE__, p "incomplete unicode character escape sequence at '%s'", p
); );
} else { } else {
UTF32 ch = unescape_unicode((unsigned char *) ++pe); UTF32 ch = unescape_unicode((unsigned char *) ++pe);
@ -525,7 +525,7 @@ static VALUE json_string_unescape(char *string, char *stringEnd, int intern, int
} }
rb_enc_raise( rb_enc_raise(
EXC_ENCODING eParserError, EXC_ENCODING eParserError,
"%u: incomplete surrogate pair at '%s'", __LINE__, p "incomplete surrogate pair at '%s'", p
); );
} }
if (pe[0] == '\\' && pe[1] == 'u') { if (pe[0] == '\\' && pe[1] == 'u') {
@ -864,7 +864,7 @@ static VALUE cParser_parse(VALUE self)
if (cs >= JSON_first_final && p == pe) { if (cs >= JSON_first_final && p == pe) {
return result; return result;
} else { } else {
rb_enc_raise(EXC_ENCODING eParserError, "%u: unexpected token at '%s'", __LINE__, p); rb_enc_raise(EXC_ENCODING eParserError, "unexpected token at '%s'", p);
return Qnil; return Qnil;
} }
} }