# Code Review: RailsConf6

**URL:** <https://www.ruby-forum.com/t/code-review-railsconf6/139310>\
**Category:** IronRuby\
**Created:** [June 2, 2008, 8:41pm UTC](https://www.ruby-forum.com/t/code-review-railsconf6/139310 "2008-06-02T20:41:06Z")\
**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:** [June 2, 2008, 8:41pm UTC](https://www.ruby-forum.com/t/code-review-railsconf6/139310/1 "2008-06-02T20:41:06Z")

</div>

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

Refactoring:

- 

```
  Use CreateTemporaryVariable/CreateLocalVariable that takes a 

```

string instead of a symbol.

- 

```
  Don't use local variables for temporary variables.

```

- 

```
  Move scope related functionality to RubyScope from RubyOps and 

```

RubyExecutionContext.

- 

```
  Removes SuperCallTargetInfo struct - replaced by out parameters.

```

Add CLR type creation and sting based evals logging.  
Adds SaveToDisk (-save) LoadFromDisk (-load) options and an experimental  
implementation.  
Adds SetName, TryLookupName to RubyContext to enable local variable  
lookup in DLR interpreter.

Fixes bug [#20390] Regexp::MULTILINE doesn’t make the regex multiline.

Tomas