From 1ecb29fbf81f9fee8e599c51b071d4e4ae558fc1 Mon Sep 17 00:00:00 2001 From: shyouhei Date: Wed, 4 Jun 2008 10:17:30 +0000 Subject: merge revision(s) 13586: * eval.c (remove_method): should not remove undef place holder. [ruby-dev:31817] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_8_6@16816 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ eval.c | 9 +++++++-- version.h | 2 +- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 8455acaadf..364e3613d8 100644 --- a/ChangeLog +++ b/ChangeLog @@ -14,6 +14,11 @@ Tue Jun 3 15:45:00 2008 Nobuyoshi Nakada * gc.c (os_obj_of, os_each_obj): hide objects to be finalized. [ruby-dev:31810] +Wed Jun 4 19:16:40 2008 Yukihiro Matsumoto + + * eval.c (remove_method): should not remove undef place holder. + [ruby-dev:31817] + Tue Jun 3 15:22:47 2008 Nobuyoshi Nakada * process.c (struct rb_exec_arg): proc should be a VALUE. diff --git a/eval.c b/eval.c index c12c08b5d9..8bb4bb1f15 100644 --- a/eval.c +++ b/eval.c @@ -569,8 +569,13 @@ remove_method(klass, mid) if (mid == __id__ || mid == __send__ || mid == init) { rb_warn("removing `%s' may cause serious problem", rb_id2name(mid)); } - if (!st_delete(RCLASS(klass)->m_tbl, &mid, (st_data_t *)&body) || - !body->nd_body) { + if (st_lookup(RCLASS(klass)->m_tbl, mid, (st_data_t *)&body)) { + if (!body || !body->nd_body) body = 0; + else { + st_delete(RCLASS(klass)->m_tbl, &mid, (st_data_t *)&body); + } + } + if (!body) { rb_name_error(mid, "method `%s' not defined in %s", rb_id2name(mid), rb_class2name(klass)); } diff --git a/version.h b/version.h index 513e84085c..de04faea90 100644 --- a/version.h +++ b/version.h @@ -2,7 +2,7 @@ #define RUBY_RELEASE_DATE "2008-06-04" #define RUBY_VERSION_CODE 186 #define RUBY_RELEASE_CODE 20080604 -#define RUBY_PATCHLEVEL 124 +#define RUBY_PATCHLEVEL 125 #define RUBY_VERSION_MAJOR 1 #define RUBY_VERSION_MINOR 8 -- cgit v1.2.3