[Ironruby-core] Code Review: RubyClrInterop06

Tomas Matousek Tomas.Matousek at microsoft.com
Mon Nov 24 19:03:09 EST 2008

In DefineConstructors, newParams seems to be unnecessarily copied.
Wouldn't it be better to do:

Type[] newParams;
Type[] baseParams = baseCtor.GetParameters();

if (has ruby class) {
  newParams = baseParams;
} if (is serializer) {
  newParams = baseParams;
} else {
  newPrams = ArrayUtils.Insert(typeof(RubyClass), baseParams);


I would also split this method to multiple pieces, seems too big already.

Other than that, looks good!


-----Original Message-----
From: Curt Hagenlocher
Sent: Monday, November 24, 2008 3:23 PM
To: IronRuby External Code Reviewers
Cc: ironruby-core at rubyforge.org
Subject: Code Review: RubyClrInterop06

tfpt review "/shelveset:RubyClrInterop06;REDMOND\curth"
Comment  :
Create constructors on generated types that match each base class constructor Allocator logic not yet updated to use new constructors

Curt Hagenlocher
curth at microsoft.com

More information about the Ironruby-core mailing list