diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-05-05 10:52:44 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-05-05 10:52:44 +0000 |
commit | 6a4bec09af7a79ffe1799015e6faae2984269ef8 (patch) | |
tree | d28d2c2af3482d03b1be2ca5809d82738203e458 | |
parent | f9279caef2696406439a7f069d1771f08ca06410 (diff) | |
download | ruby-6a4bec09af7a79ffe1799015e6faae2984269ef8.tar.gz ruby-6a4bec09af7a79ffe1799015e6faae2984269ef8.tar.xz ruby-6a4bec09af7a79ffe1799015e6faae2984269ef8.zip |
* struct.c (rb_struct_s_def): Struct.new(0) should not SEGV.
based on the patch from wanabe <s.wanabe at gmail.com> in
[ruby-dev:34594].
* struct.c (make_struct): call to_str on name object.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@16295 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 8 | ||||
-rw-r--r-- | struct.c | 12 |
2 files changed, 15 insertions, 5 deletions
@@ -1,3 +1,11 @@ +Mon May 5 19:49:59 2008 Yukihiro Matsumoto <matz@ruby-lang.org> + + * struct.c (rb_struct_s_def): Struct.new(0) should not SEGV. + based on the patch from wanabe <s.wanabe at gmail.com> in + [ruby-dev:34594]. + + * struct.c (make_struct): call to_str on name object. + Mon May 5 17:17:40 2008 Tanaka Akira <akr@fsij.org> * eval.c (ruby_cleanup): wrap ruby_finalize_0 by SAVE_ROOT_JMPBUF to @@ -181,7 +181,9 @@ make_struct(VALUE name, VALUE members, VALUE klass) rb_class_inherited(klass, nstr); } else { - id = SYM2ID(rb_str_intern(name)); + /* old style: should we warn? */ + name = rb_str_to_str(name); + id = rb_to_id(name); if (!rb_is_const_id(id)) { rb_name_error(id, "identifier %s needs to be constant", StringValuePtr(name)); } @@ -322,14 +324,14 @@ rb_struct_s_def(int argc, VALUE *argv, VALUE klass) ID id; rb_scan_args(argc, argv, "1*", &name, &rest); - for (i=0; i<RARRAY_LEN(rest); i++) { - id = rb_to_id(RARRAY_PTR(rest)[i]); - RARRAY_PTR(rest)[i] = ID2SYM(id); - } if (!NIL_P(name) && SYMBOL_P(name)) { rb_ary_unshift(rest, name); name = Qnil; } + for (i=0; i<RARRAY_LEN(rest); i++) { + id = rb_to_id(RARRAY_PTR(rest)[i]); + RARRAY_PTR(rest)[i] = ID2SYM(id); + } st = make_struct(name, rest, klass); if (rb_block_given_p()) { rb_mod_module_eval(0, 0, st); |