# Code Review: interpret1

**URL:** <https://www.ruby-forum.com/t/code-review-interpret1/145090>\
**Category:** IronRuby\
**Created:** [August 5, 2008, 10:39pm UTC](https://www.ruby-forum.com/t/code-review-interpret1/145090 "2008-08-05T22:39:05Z")\
**Posts on this page:** 3\
**Page:** 1

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [August 5, 2008, 10:39pm UTC](https://www.ruby-forum.com/t/code-review-interpret1/145090/1 "2008-08-05T22:39:05Z")

</div>

tfpt review “/shelveset:interpret1;REDMOND\jdeville”  
Comment :  
Modifies run.rb and common.rb to clean up command line flags and run  
tests under -D -X:Interpret, -D and -D -X:SaveAssemblies. All three are  
ran in snap with this change. This also modifies the rake.cmd file in  
$MERLIN\_EXTERNAL to use %dp0~ instead of a fixed path.

---

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [August 5, 2008, 11:44pm UTC](https://www.ruby-forum.com/t/code-review-interpret1/145090/2 "2008-08-05T23:44:09Z")

</div>

Yep, check it in!

Tomas

---

<div class="post-metadata">

**Author:** ![raker\_t](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/r/ecae2f/32.png) [@raker\_t](https://www.ruby-forum.com/u/raker_t)\
**Post date:** [August 5, 2008, 11:44pm UTC](https://www.ruby-forum.com/t/code-review-interpret1/145090/3 "2008-08-05T23:44:09Z")

</div>

Looks good to me.