[Ironruby-core] Code Review: Copy4
curth at microsoft.com
Sun Oct 12 23:17:50 EDT 2008
The KernelOps._sites dictionary never
There's at least one "#region IRubyCloneableMembers" in RubyObject.cs which should be renamed. (There may be others.)
Some of the comments in RubyModule.Subclass.cs weren't edited after they were copied from MutableString.Subclass.c
From: Tomas Matousek
Sent: Friday, October 10, 2008 6:43 PM
To: IronRuby External Code Reviewers
Cc: ironruby-core at rubyforge.org
Subject: Code Review: Copy4
tfpt review "/shelveset:Copy4;REDMOND\tomat"
Implements initialize_copy for built-in classes.
Removes assertion in Proc, it was wrong.
Improves and fixes Kernel#send.
Implements default protocol for to_ary conversion.
Fixes super call with a block but no other arguments.
[#20849] instance_eval within class_eval causes incorrect method lookup.
[ ironruby-Bugs-20317 ] Kernel#instance_eval.
[ ironruby-Bugs-20705 ] instance_eval allows defining methods on all objects
[ ironruby-Bugs-20408 ] TRUE, FALSE & NIL not defined.
[ ironruby-Bugs-21942 ] a const defined on a metaclass is not preserved when the object is cloned
More information about the Ironruby-core