diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2003-05-22 08:30:58 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2003-05-22 08:30:58 +0000 |
commit | 717302e2c4de9b86bf53fe65173500a33586001c (patch) | |
tree | 058e1b7acff0187ab6c193278acfc86c2675c89c | |
parent | df62bb4fc987317e209679ef60ea98b31ebae431 (diff) |
* gc.c (run_final): use rb_thread_critical instead of DEFER_INTS.
[ruby-dev:20272]
* marshal.c: try to make ArgumentError and TypeError consistent.
[ruby-core:01068]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@3853 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 8 | ||||
-rw-r--r-- | gc.c | 7 | ||||
-rw-r--r-- | marshal.c | 14 |
3 files changed, 19 insertions, 10 deletions
@@ -1,3 +1,11 @@ +Thu May 22 17:12:10 2003 Yukihiro Matsumoto <matz@ruby-lang.org> + + * gc.c (run_final): use rb_thread_critical instead of DEFER_INTS. + [ruby-dev:20272] + + * marshal.c: try to make ArgumentError and TypeError consistent. + [ruby-core:01068] + Thu May 22 15:46:37 2003 Yukihiro Matsumoto <matz@ruby-lang.org> * eval.c (rb_define_alloc_func): need not to disable @@ -1519,10 +1519,11 @@ run_final(obj) VALUE obj; { long i; - int status; + int status, critical_save; VALUE args[2], table; - DEFER_INTS; + critical_save = rb_thread_critical; + rb_thread_critical = Qtrue; args[1] = rb_ary_new3(1, rb_obj_id(obj)); /* make obj into id */ for (i=0; i<RARRAY(finalizers)->len; i++) { args[0] = RARRAY(finalizers)->ptr[i]; @@ -1534,7 +1535,7 @@ run_final(obj) rb_protect((VALUE(*)_((VALUE)))run_single_final, (VALUE)args, &status); } } - ENABLE_INTS; + rb_thread_critical = critical_save; } void @@ -327,7 +327,7 @@ w_unique(s, arg) struct dump_arg *arg; { if (s[0] == '#') { - rb_raise(rb_eArgError, "can't dump anonymous class %s", s); + rb_raise(rb_eTypeError, "can't dump anonymous class %s", s); } w_symbol(rb_intern(s), arg); } @@ -502,7 +502,7 @@ w_object(obj, arg, limit) { VALUE path = rb_class_path(obj); if (RSTRING(path)->ptr[0] == '#') { - rb_raise(rb_eArgError, "can't dump anonymous class %s", + rb_raise(rb_eTypeError, "can't dump anonymous class %s", RSTRING(path)->ptr); } w_bytes(RSTRING(path)->ptr, RSTRING(path)->len, arg); @@ -514,7 +514,7 @@ w_object(obj, arg, limit) { VALUE path = rb_class_path(obj); if (RSTRING(path)->ptr[0] == '#') { - rb_raise(rb_eArgError, "can't dump anonymous module %s", + rb_raise(rb_eTypeError, "can't dump anonymous module %s", RSTRING(path)->ptr); } w_bytes(RSTRING(path)->ptr, RSTRING(path)->len, arg); @@ -588,7 +588,7 @@ w_object(obj, arg, limit) } else if (FL_TEST(obj, FL_USER2)) { /* FL_USER2 means HASH_PROC_DEFAULT (see hash.c) */ - rb_raise(rb_eArgError, "cannot dump hash with default proc"); + rb_raise(rb_eTypeError, "cannot dump hash with default proc"); } else { w_byte(TYPE_HASH_DEF, arg); @@ -848,7 +848,7 @@ r_symlink(arg) if (st_lookup(arg->symbol, num, &id)) { return id; } - rb_raise(rb_eTypeError, "bad symbol"); + rb_raise(rb_eArgError, "bad symbol"); } static ID @@ -921,7 +921,7 @@ path2class(path) VALUE v = rb_path2class(path); if (TYPE(v) != T_CLASS) { - rb_raise(rb_eTypeError, "%s does not refer class", path); + rb_raise(rb_eArgError, "%s does not refer class", path); } return v; } @@ -933,7 +933,7 @@ path2module(path) VALUE v = rb_path2class(path); if (TYPE(v) != T_MODULE) { - rb_raise(rb_eTypeError, "%s does not refer module", path); + rb_raise(rb_eArgError, "%s does not refer module", path); } return v; } |