diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-02-05 02:29:18 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-02-05 02:29:18 +0000 |
commit | 1069e5d665599ce60086f284b575d3a850ed2b01 (patch) | |
tree | 0ad0acf49f2147bd8405dbc75d592429efadee87 | |
parent | 9cc62abc6b0245e5f0e4a8f0cd9b6f53a4f9bf77 (diff) | |
download | ruby-1069e5d665599ce60086f284b575d3a850ed2b01.tar.gz |
* ext/json/parser/parser.h (GET_PARSER): raise TypeError.
* ext/json/parser/parser.rl (cParser_initialize): ditto.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@30792 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | ext/json/parser/parser.c | 2 | ||||
-rw-r--r-- | ext/json/parser/parser.h | 2 | ||||
-rw-r--r-- | ext/json/parser/parser.rl | 2 | ||||
-rwxr-xr-x | test/json/test_json.rb | 4 |
5 files changed, 10 insertions, 6 deletions
@@ -1,4 +1,8 @@ -Sat Feb 5 10:29:52 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> +Sat Feb 5 11:29:10 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * ext/json/parser/parser.h (GET_PARSER): raise TypeError. + + * ext/json/parser/parser.rl (cParser_initialize): ditto. * ext/json/parser/parser.h (GET_PARSER): check if initialized. [ruby-core:35079] diff --git a/ext/json/parser/parser.c b/ext/json/parser/parser.c index 22c8167fc7..418c1c32f0 100644 --- a/ext/json/parser/parser.c +++ b/ext/json/parser/parser.c @@ -1613,7 +1613,7 @@ static VALUE cParser_initialize(int argc, VALUE *argv, VALUE self) GET_PARSER_INIT; if (json->Vsource) { - rb_raise(rb_eArgError, "already initialized instance"); + rb_raise(rb_eTypeError, "already initialized instance"); } rb_scan_args(argc, argv, "11", &source, &opts); source = convert_encoding(StringValue(source)); diff --git a/ext/json/parser/parser.h b/ext/json/parser/parser.h index 5d4532b948..a344da058c 100644 --- a/ext/json/parser/parser.h +++ b/ext/json/parser/parser.h @@ -45,7 +45,7 @@ typedef struct JSON_ParserStruct { #define GET_PARSER \ GET_PARSER_INIT; \ - if (!json->Vsource) rb_raise(rb_eArgError, "uninitialized instance") + if (!json->Vsource) rb_raise(rb_eTypeError, "uninitialized instance") #define GET_PARSER_INIT \ JSON_Parser *json; \ Data_Get_Struct(self, JSON_Parser, json) diff --git a/ext/json/parser/parser.rl b/ext/json/parser/parser.rl index 680242732e..278386432b 100644 --- a/ext/json/parser/parser.rl +++ b/ext/json/parser/parser.rl @@ -611,7 +611,7 @@ static VALUE cParser_initialize(int argc, VALUE *argv, VALUE self) GET_PARSER_INIT; if (json->Vsource) { - rb_raise(rb_eArgError, "already initialized instance"); + rb_raise(rb_eTypeError, "already initialized instance"); } rb_scan_args(argc, argv, "11", &source, &opts); source = convert_encoding(StringValue(source)); diff --git a/test/json/test_json.rb b/test/json/test_json.rb index cb70085242..aea50485fc 100755 --- a/test/json/test_json.rb +++ b/test/json/test_json.rb @@ -394,8 +394,8 @@ EOT def test_allocate json = JSON::Parser.new("{}") - assert_raises(ArgumentError, '[ruby-core:35079]') {json.__send__(:initialize, "{}")} + assert_raises(TypeError, '[ruby-core:35079]') {json.__send__(:initialize, "{}")} json = JSON::Parser.allocate - assert_raises(ArgumentError, '[ruby-core:35079]') {json.source} + assert_raises(TypeError, '[ruby-core:35079]') {json.source} end end |