# Code Review: rubyopt

**URL:** <https://www.ruby-forum.com/t/code-review-rubyopt/143091>\
**Category:** IronRuby\
**Created:** [July 15, 2008, 3:58am UTC](https://www.ruby-forum.com/t/code-review-rubyopt/143091 "2008-07-15T03:58:55Z")\
**Posts on this page:** 1\
**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:** [July 15, 2008, 3:58am UTC](https://www.ruby-forum.com/t/code-review-rubyopt/143091/1 "2008-07-15T03:58:55Z")

</div>

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

Misc Ruby optimizations.

- 

```
Optimizes method lookup:

```

1. Adds cache for unsuccessful CLR member lookup. CLR types are  
read-only and so the knowledge of a member lookup failure could be  
stored for the next lookup. This saves about 10% of allocated memory  
since unsuccessful lookups in reflection allocate some MethodInfo  
arrays.
2. Stores successfully resolved CLR members in the module method  
table. We don’t need to create a RubyMethodGroupInfo each time we find  
CLR member and also if we find it once it is available the next time as  
well (unless it is removed/modified).
3. When a new method is added into a module/class and it doesn’t  
hide a method of the same name that was already used in dynamic sites,  
the sites don’t need to be invalidated.

- 

```
Adds some perf counters.

```

- 

```
Fixes message in method_missing.

```

- 

```
Refactors RubyBinder a bit, especially super-call action.

```

Tomas