diff options
author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-09-10 06:25:40 +0000 |
---|---|---|
committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-09-10 06:25:40 +0000 |
commit | 4261abb2305320ac2a335fed5460adafc265f0bd (patch) | |
tree | 694631f7fd4fcb019e94339af134b584e6a2c552 | |
parent | b3fc59837aa53c67e8167700b27ad79feaa30e33 (diff) |
* range.c: represent initialized state using EXCL instead of FL_USER3.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@13425 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | range.c | 7 | ||||
-rw-r--r-- | test/ruby/test_range.rb | 5 |
3 files changed, 10 insertions, 6 deletions
@@ -1,3 +1,7 @@ +Mon Sep 10 15:22:24 2007 Tanaka Akira <akr@fsij.org> + + * range.c: represent initialized state using EXCL instead of FL_USER3. + Mon Sep 10 13:44:37 2007 Yukihiro Matsumoto <matz@ruby-lang.org> * array.c (rb_ary_cycle): avoid infinite loop for empty array. @@ -22,8 +22,6 @@ static ID id_cmp, id_succ, id_beg, id_end, id_excl; #define EXCL(r) RTEST(RANGE_EXCL(r)) #define SET_EXCL(r,v) (RSTRUCT(r)->as.ary[2] = (v) ? Qtrue : Qfalse) -#define FL_INITIALIZED FL_USER3 - static VALUE range_alloc(VALUE klass) { @@ -37,8 +35,6 @@ range_alloc(VALUE klass) RBASIC(r)->flags |= n << RSTRUCT_EMBED_LEN_SHIFT; rb_mem_clear(r->as.ary, n); - RBASIC(r)->flags &= ~FL_INITIALIZED; - return (VALUE)r; } @@ -101,10 +97,9 @@ range_initialize(int argc, VALUE *argv, VALUE range) rb_scan_args(argc, argv, "21", &beg, &end, &flags); /* Ranges are immutable, so that they should be initialized only once. */ - if (RBASIC(range)->flags & FL_INITIALIZED) { + if (RANGE_EXCL(range) != Qnil) { rb_name_error(rb_intern("initialize"), "`initialize' called twice"); } - RBASIC(range)->flags |= FL_INITIALIZED; range_init(range, beg, end, RTEST(flags)); return Qnil; } diff --git a/test/ruby/test_range.rb b/test/ruby/test_range.rb index 030aeb977b..847e261f97 100644 --- a/test/ruby/test_range.rb +++ b/test/ruby/test_range.rb @@ -59,4 +59,9 @@ class TestRange < Test::Unit::TestCase assert_equal(0, (0..0).max) assert_equal(nil, (0...0).max) end + + def test_initialize_twice + r = eval("1..2") + assert_raise(NameError) { r.instance_eval { initialize 3, 4 } } + end end |