1 files changed, 15 insertions, 13 deletions

diff --git a/lib/ostruct.rb b/lib/ostruct.rb
index f51eb7b5db..9e2761dad9 100644
--- a/lib/ostruct.rb
+++ b/lib/ostruct.rb
@@ -90,7 +90,7 @@ class OpenStruct
       hash.each_pair do |k, v|
         k = k.to_sym
         @table[k] = v
-        new_ostruct_member(k)
+        new_ostruct_member!(k)
       end
     end
   end
@@ -99,7 +99,7 @@ class OpenStruct
   def initialize_copy(orig)
     super
     @table = @table.dup
-    @table.each_key{|key| new_ostruct_member(key)}
+    @table.each_key{|key| new_ostruct_member!(key)}
   end
 
   #
@@ -141,14 +141,14 @@ class OpenStruct
   #
   def marshal_load(x)
     @table = x
-    @table.each_key{|key| new_ostruct_member(key)}
+    @table.each_key{|key| new_ostruct_member!(key)}
   end
 
   #
   # Used internally to check if the OpenStruct is able to be
   # modified before granting access to the internal Hash table to be modified.
   #
-  def modifiable
+  def modifiable?
     begin
       @modifiable = true
     rescue
@@ -156,22 +156,22 @@ class OpenStruct
     end
     @table
   end
-  protected :modifiable
+  protected :modifiable?
 
   #
   # Used internally to defined properties on the
   # OpenStruct. It does this by using the metaprogramming function
   # define_singleton_method for both the getter method and the setter method.
   #
-  def new_ostruct_member(name)
+  def new_ostruct_member!(name)
     name = name.to_sym
     unless respond_to?(name)
       define_singleton_method(name) { @table[name] }
-      define_singleton_method("#{name}=") { |x| modifiable[name] = x }
+      define_singleton_method("#{name}=") { |x| modifiable?[name] = x }
     end
     name
   end
-  protected :new_ostruct_member
+  protected :new_ostruct_member!
 
   def method_missing(mid, *args) # :nodoc:
     mname = mid.id2name
@@ -180,7 +180,7 @@ class OpenStruct
       if len != 1
         raise ArgumentError, "wrong number of arguments (#{len} for 1)", caller(1)
       end
-      modifiable[new_ostruct_member(mname)] = args[0]
+      modifiable?[new_ostruct_member!(mname)] = args[0]
     elsif len == 0
       @table[mid]
     else
@@ -207,7 +207,7 @@ class OpenStruct
   #   person.age # => 42
   #
   def []=(name, value)
-    modifiable[new_ostruct_member(name)] = value
+    modifiable?[new_ostruct_member!(name)] = value
   end
 
   #
@@ -255,7 +255,9 @@ class OpenStruct
   alias :to_s :inspect
 
   attr_reader :table # :nodoc:
-  protected :table
+  alias table! table
+  undef table
+  protected :table!
 
   #
   # Compares this object and +other+ for equality.  An OpenStruct is equal to
@@ -264,7 +266,7 @@ class OpenStruct
   #
   def ==(other)
     return false unless other.kind_of?(OpenStruct)
-    @table == other.table
+    @table == other.table!
   end
 
   #
@@ -274,7 +276,7 @@ class OpenStruct
   #
   def eql?(other)
     return false unless other.kind_of?(OpenStruct)
-    @table.eql?(other.table)
+    @table.eql?(other.table!)
   end
 
   # Compute a hash-code for this OpenStruct.