# Code Review: shrigitpull

**URL:** <https://www.ruby-forum.com/t/code-review-shrigitpull/162957>\
**Category:** IronRuby\
**Created:** [March 16, 2009, 7:26am UTC](https://www.ruby-forum.com/t/code-review-shrigitpull/162957 "2009-03-16T07:26:48Z")\
**Posts on this page:** 2\
**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:** [March 16, 2009, 7:26am UTC](https://www.ruby-forum.com/t/code-review-shrigitpull/162957/1 "2009-03-16T07:26:48Z")

</div>

tfpt review “/shelveset:shrigitpull;REDMOND\jdeville”  
Comment :  
This shelveset is an external contribution from Shri. Tests have been  
modified by me.

Added support to YAML.quick\_emit to be called on nested objects where  
the  
same emitter needs to be flowed through  
Array.pack(“a”) was not dealing with nil input  
yaml.rb should to “require stringio” like MRI does  
Deleted a second copy of yaml.rb from the IronRuby.Libraries.Yaml  
folder  
Added MutableString.Dump property to be able to inspect large strings  
in  
VS.  
ir.exe -X:ExceptionDetail was not working in interactive mode because  
ir.exe catches the exception and does not call  
RubyExceptionData.SetCompiledTrace like a rescue block in Ruby code  
would  
do. Fixed this

---

<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:** [March 16, 2009, 8:10am UTC](https://www.ruby-forum.com/t/code-review-shrigitpull/162957/2 "2009-03-16T08:10:05Z")

</div>

Looks good ofcourse 🙂