# Code Review: BF.3

**URL:** <https://www.ruby-forum.com/t/code-review-bf-3/177433>\
**Category:** IronRuby\
**Created:** [October 17, 2009, 1:28am UTC](https://www.ruby-forum.com/t/code-review-bf-3/177433 "2009-10-17T01:28:04Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![grujildo](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/g/94ad74/32.png) [@grujildo](https://www.ruby-forum.com/u/grujildo)\
**Post date:** [October 17, 2009, 1:28am UTC](https://www.ruby-forum.com/t/code-review-bf-3/177433/1 "2009-10-17T01:28:04Z")

</div>

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](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.

Tomas

---

<div class="post-metadata">

**Author:** ![grujildo](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/g/94ad74/32.png) [@grujildo](https://www.ruby-forum.com/u/grujildo)\
**Post date:** [October 17, 2009, 1:29am UTC](https://www.ruby-forum.com/t/code-review-bf-3/177433/2 "2009-10-17T01:29:15Z")

</div>

This should actually be:

tfpt review “/shelveset:DF.4;REDMOND\tomat”

Tomas