[Ironruby-core] Code Review: BF.3

Tomas Matousek Tomas.Matousek at microsoft.com
Fri Oct 16 19:26:16 EDT 2009


  tfpt review "/shelveset:BF.3;REDMOND\tomat"
  Comment  : 
  DLR:
  Fixes ScriptScope's BindSetMember - it needs to convert the result expression to Object.
  
  Fixes http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=2421 - if a CLR array is passed to params-array parameter it is not treated as a single parameter. Instead the overload resolver "splats" it by default (implements C# semantics). This is not the right behavior for Ruby. I've added ExpandParams virtual method that allows the language to specify for a given MethodCandidate whether its params array/dictionary should always be expanded and thus never use the incoming array as the params array.
  
  Interpreter - implements TypeIs instruction (fixes http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=2462).
  
  Ruby:
  Moves methods related to libraries from RubyModule to the library initializer.
  
  Fixes 
  http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1849
  http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=2243 
  http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=2171
  http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=1989
  http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=2146

Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: BF.3.diff
Type: application/octet-stream
Size: 803149 bytes
Desc: BF.3.diff
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20091016/ec052282/attachment-0001.obj>


More information about the Ironruby-core mailing list