# Code Review: RubyPrecompilation

**URL:** <https://www.ruby-forum.com/t/code-review-rubyprecompilation/139756>\
**Category:** IronRuby\
**Created:** [June 6, 2008, 7:11pm UTC](https://www.ruby-forum.com/t/code-review-rubyprecompilation/139756 "2008-06-06T19:11:23Z")\
**Posts on this page:** 3\
**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:** [June 6, 2008, 7:11pm UTC](https://www.ruby-forum.com/t/code-review-rubyprecompilation/139756/1 "2008-06-06T19:11:23Z")

</div>

Ruby and DLR.

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

Removes dependency of precompiled code on optimized scopes.  
Enables loading/saving precompiled assemblies in Ruby loader.  
Implements IExpressionSerializable on ProtocolConversionAction to enable  
its serialization to IL.

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:** [June 6, 2008, 7:37pm UTC](https://www.ruby-forum.com/t/code-review-rubyprecompilation/139756/2 "2008-06-06T19:37:54Z")

</div>

The implementation is still experimental, so it expects  
IronRubyApplication.dll file located in the current directory.  
We need to design how pre-compilation is exposed to the users.

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:** [June 6, 2008, 7:32pm UTC](https://www.ruby-forum.com/t/code-review-rubyprecompilation/139756/3 "2008-06-06T19:32:01Z")

</div>

The only concerning thing I see in the Ruby changes are the  
Assembly.LoadFrom - what’s up with that? It’s probably file for this  
simple case (because all of the dependencies will already be loaded) but  
it’s a little scary.

Adding DLR Code Reviews just to make sure the DLR devs don’t miss this.
