# Code Review: bugfixes9

**URL:** <https://www.ruby-forum.com/t/code-review-bugfixes9/136506>\
**Category:** IronRuby\
**Created:** [May 2, 2008, 8:40pm UTC](https://www.ruby-forum.com/t/code-review-bugfixes9/136506 "2008-05-02T20:40:49Z")\
**Posts on this page:** 1\
**Page:** 1

<div class="post-metadata">

**Author:** ![John\_LSSIRONRUBY](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/4491bb/32.png) [@John\_LSSIRONRUBY](https://www.ruby-forum.com/u/John_LSSIRONRUBY)\
**Post date:** [May 2, 2008, 8:40pm UTC](https://www.ruby-forum.com/t/code-review-bugfixes9/136506/1 "2008-05-02T20:40:49Z")

</div>

tfpt review /shelveset:bugfixes9;REDMOND\jflam

Ruby only bug fixes:

Passes all local tests.

#19804: Inheriting from Numeric

- added a default ctor

#19806: Wrong number or type of arguments for Regexp

- added overloads and trivially implemented Regexp.compile as well
- removed ctor for Regexp that takes a string - not necessary because we  
have implicit conversion operator for MutableString to String

#19887: Array#reverse\_each not implemented

- implemented and verified that all specs pass

#19893: String#ljust not implemented

- implemented ljust - all specs pass
- also trivially implemented rjust - all specs pass

#19842: undef\_method overload

- added overload that accepts a string

#19891: Module comparison methods unimplemented

- implemented \<, \<=, \>, \>=, \<=\> - all specs pass

#19845: syntax error - unexpected BLOCK\_DO

- fixed by patch #19880

Patches applied:

#19879: Implements the RUBY\_ENGINE constant (Michael L.)

#19880: Fix for Parser CmdArg bug #19845

Thanks,  
-John