# Code Review: MoreProtocols8

**URL:** <https://www.ruby-forum.com/t/code-review-moreprotocols8/160577>\
**Category:** IronRuby\
**Created:** [February 17, 2009, 11:03pm UTC](https://www.ruby-forum.com/t/code-review-moreprotocols8/160577 "2009-02-17T23:03:10Z")\
**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:** [February 17, 2009, 11:03pm UTC](https://www.ruby-forum.com/t/code-review-moreprotocols8/160577/1 "2009-02-17T23:03:10Z")

</div>

tfpt review “/shelveset:MoreProtocols8;REDMOND\tomat”

Affects DLR outer ring, IronPython and IronRuby.

DLR:

- Changes names of arguments/parameters in default binder to be  
typed to strings instead of SymbolIds.

- Adds NonNullItems attribute. If applied on a params-array or  
dict-array parameter the binder only binds non-null arguments to the  
items of the array/dict.

IronPython:

- Reflects changes in DLR.

IronRuby:

- Implements missing protocols and removes all static sites used in  
protocol conversions.

- Fixes caching issues in protocol conversions.

- Misc fixes in libraries.

Passed “run 0”.

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:** [February 18, 2009, 5:39pm UTC](https://www.ruby-forum.com/t/code-review-moreprotocols8/160577/2 "2009-02-18T17:39:59Z")

</div>

Ruby changes look good.

From: Dino Viehland  
Sent: Tuesday, February 17, 2009 2:27 PM  
To: Tomas M.; IronRuby External Code R.; Rowan Code  
Reviewers; DLR Code Reviews  
Cc: removed\_email\_address@domain.invalid  
Subject: RE: Code Review: MoreProtocols8

Python changes look good.

From: Tomas M.  
Sent: Tuesday, February 17, 2009 2:02 PM  
To: IronRuby External Code R.; Rowan Code R.; DLR Code  
Reviews  
Cc: removed\_email\_address@domain.invalid  
Subject: Code Review: MoreProtocols8

tfpt review “/shelveset:MoreProtocols8;REDMOND\tomat”

Affects DLR outer ring, IronPython and IronRuby.

DLR:

- Changes names of arguments/parameters in default binder to be  
typed to strings instead of SymbolIds.

- Adds NonNullItems attribute. If applied on a params-array or  
dict-array parameter the binder only binds non-null arguments to the  
items of the array/dict.

IronPython:

- Reflects changes in DLR.

IronRuby:

- Implements missing protocols and removes all static sites used in  
protocol conversions.

- Fixes caching issues in protocol conversions.

- Misc fixes in libraries.

Passed “run 0”.

Tomas