Merge pull request rubinius#1331 from krekoten/core_module_class_vari…

…able_set_19 Module#class_variable_set raises a RuntimeError when self is frozen
terceiro · Oct 22, 2011 · 33ad0df · 33ad0df
2 parents ae8f034 + fbe4193
commit 33ad0df
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/kernel/common/module.rb b/kernel/common/module.rb
@@ -47,7 +47,7 @@ def verify_class_variable_name(name)
   end
   private :verify_class_variable_name
 
-  def class_variable_set(name, val)
+  def __class_variable_set__(name, val)
     Rubinius.primitive :module_cvar_set
 
     class_variable_set verify_class_variable_name(name), val

diff --git a/kernel/common/module18.rb b/kernel/common/module18.rb
@@ -25,4 +25,6 @@ def attr(name, writeable = false)
   end
 
   private :attr
+
+  alias_method :class_variable_set, :__class_variable_set__
 end
diff --git a/kernel/common/module19.rb b/kernel/common/module19.rb
@@ -40,4 +40,9 @@ def attr(*attributes)
   end
 
   private :attr
+
+  def class_variable_set(key, value)
+    raise RuntimeError, "can't modify frozen #{self.class}" if frozen?
+    __class_variable_set__(key, value)
+  end
 end
diff --git a/spec/tags/19/ruby/core/module/class_variable_set_tags.txt b/spec/tags/19/ruby/core/module/class_variable_set_tags.txt