# Code Review: MO-RubyCall2

**URL:** <https://www.ruby-forum.com/t/code-review-mo-rubycall2/145009>\
**Category:** IronRuby\
**Created:** [August 5, 2008, 5:07am UTC](https://www.ruby-forum.com/t/code-review-mo-rubycall2/145009 "2008-08-05T05:07:58Z")\
**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:** [August 5, 2008, 5:07am UTC](https://www.ruby-forum.com/t/code-review-mo-rubycall2/145009/1 "2008-08-05T05:07:58Z")

</div>

tfpt review “/shelveset:MO-RubyCall2;REDMOND\tomat”

Switches all Ruby call actions to use meta-objects.  
Also refactors some methods generating rules for procs and adds  
meta-objects for more Ruby classes (not used so far).

DLR: Makes ComMetaObject.IsComObject work in partial trust.

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:** [August 5, 2008, 5:28pm UTC](https://www.ruby-forum.com/t/code-review-mo-rubycall2/145009/2 "2008-08-05T17:28:12Z")

</div>

There are two typos in IoOps.cs where \_CreateIOSharedSite1 is being  
passed to the Target method of \_CreateIOSharedSite11. It’s a very  
hard-to-spot problem. Maybe one of the names should be changed. 🙂

There’s also a typo in a comment in RubyModule – should be ctor instead  
of ctro.

Otherwise, looks great.