diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | lib/ostruct.rb | 2 | ||||
-rw-r--r-- | test/ostruct/test_ostruct.rb | 3 | ||||
-rw-r--r-- | version.h | 2 |
4 files changed, 9 insertions, 3 deletions
@@ -1,3 +1,8 @@ +Fri Feb 24 23:49:05 2012 Marc-Andre Lafortune <ruby-core@marc-andre.ca> + + * lib/ostruct.rb (delete_field): Bug fix so previous value is + returned. Patch by Nick Recobra [Bug #6063] + Fri Feb 24 08:53:28 2012 Nobuyoshi Nakada <nobu@ruby-lang.org> * parse.y (parser_tokadd_string, parser_yylex): insert a backslash diff --git a/lib/ostruct.rb b/lib/ostruct.rb index a57eb08e7c..a4c0d55d09 100644 --- a/lib/ostruct.rb +++ b/lib/ostruct.rb @@ -189,8 +189,8 @@ class OpenStruct # def delete_field(name) sym = name.to_sym - @table.delete sym singleton_class.__send__(:remove_method, sym, "#{name}=") + @table.delete sym end InspectKey = :__inspect_key__ # :nodoc: diff --git a/test/ostruct/test_ostruct.rb b/test/ostruct/test_ostruct.rb index db8e7415eb..dca2382b2b 100644 --- a/test/ostruct/test_ostruct.rb +++ b/test/ostruct/test_ostruct.rb @@ -57,9 +57,10 @@ class TC_OpenStruct < Test::Unit::TestCase o.a = 'a' assert_respond_to(o, :a) assert_respond_to(o, :a=) - o.delete_field :a + a = o.delete_field :a assert_not_respond_to(o, :a, bug) assert_not_respond_to(o, :a=, bug) + assert_equal(a, 'a') end def test_method_missing_handles_square_bracket_equals @@ -1,5 +1,5 @@ #define RUBY_VERSION "1.9.3" -#define RUBY_PATCHLEVEL 142 +#define RUBY_PATCHLEVEL 143 #define RUBY_RELEASE_DATE "2012-02-24" #define RUBY_RELEASE_YEAR 2012 |