# Code Review: rubyclrinterop01

**URL:** <https://www.ruby-forum.com/t/code-review-rubyclrinterop01/145760>\
**Category:** IronRuby\
**Created:** [August 13, 2008, 7:31pm UTC](https://www.ruby-forum.com/t/code-review-rubyclrinterop01/145760 "2008-08-13T19:31:46Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![curthibbs](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/c/ac91a4/32.png) [@curthibbs](https://www.ruby-forum.com/u/curthibbs)\
**Post date:** [August 13, 2008, 7:31pm UTC](https://www.ruby-forum.com/t/code-review-rubyclrinterop01/145760/1 "2008-08-13T19:31:46Z")

</div>

tfpt review “/shelveset:rubyclrinterop01;REDMOND\curth”

Ruby changes:  
Initial changes for improved Ruby CLR interop:  
Defer creation of underlying CLR type until later to allow  
extending with CLR interfaces  
Prevent generic CLR interfaces from being included into modules or  
classes  
Prevent CLR interfaces from being included into classes after the  
underlying type has been created  
Give more reasonable error message when attempting to derive from  
generic CLR type  
Fixed bug 21470 in Marshal.cs  
Fixed misspelled member name in Microsoft.Scripting.SymbolId

Python test change:  
Fixed test\_peverify to work with the VS2008SP1 version of peverify.exe

DLR change:  
Fixed misspelled member name in Microsoft.Scripting.SymbolId

Sorry about combining these, but the Python test and DLR change don’t  
merit separate SNAP jobs

-Curt

---

<div class="post-metadata">

**Author:** ![curthibbs](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/c/ac91a4/32.png) [@curthibbs](https://www.ruby-forum.com/u/curthibbs)\
**Post date:** [August 13, 2008, 8:11pm UTC](https://www.ruby-forum.com/t/code-review-rubyclrinterop01/145760/2 "2008-08-13T20:11:38Z")

</div>

The Python change looks good to me.