[Ironruby-core] Code Review (External): NumericalFixes

John Lam (IRONRUBY) jflam at microsoft.com
Thu Oct 9 12:17:07 EDT 2008


Looks good. Please check into SVN and I'll integrate. Note that I'm pushing out r159 in a few minutes, so you'll need to update and regenerate the initializers before you check in.

Thanks,
-John

From: ironruby-core-bounces at rubyforge.org [mailto:ironruby-core-bounces at rubyforge.org] On Behalf Of Peter Bacon Darwin
Sent: Thursday, October 09, 2008 7:21 AM
To: ironruby-core at rubyforge.org
Subject: Re: [Ironruby-core] Code Review (External): NumericalFixes

Thanks Jim.  I pulled the latest rubyspecs and there are no unexpected failures.

Still waiting for someone to review my code...

Pete

From: ironruby-core-bounces at rubyforge.org [mailto:ironruby-core-bounces at rubyforge.org] On Behalf Of Jim Deville
Sent: Wednesday,08 October 08, 2008 21:44
To: ironruby-core at rubyforge.org
Subject: Re: [Ironruby-core] Code Review (External): NumericalFixes

If you are running against the RubySpec's version of RubySpecs (as opposed to the IronRuby repo) you may want to sync and test again. I pushed some changes last night that make some of the float comparison do numerical comparisons instead of string comparisons.


JD

From: ironruby-core-bounces at rubyforge.org [mailto:ironruby-core-bounces at rubyforge.org] On Behalf Of Peter Bacon Darwin
Sent: Wednesday, October 08, 2008 10:09 AM
To: ironruby-core at rubyforge.org
Subject: Re: [Ironruby-core] Code Review (External): NumericalFixes

Ping!

From: Peter Bacon Darwin [mailto:bacondarwin at googlemail.com]
Sent: Tuesday,07 October 07, 2008 14:38
To: 'ironruby-core at rubyforge.org'
Subject: Code Review (External): NumericalFixes

A number of fixes to pass most of the relevant rubyspecs.
                BignumOps.cs: Leftshift and Rightshift fixes
                FixnumOps.cs: Leftshift, Rightshift, Bitwise_Xor, Bitwise_And and Bitwise_Or fixes
                FloatOps.cs: Constant and IsNegativeZero modification.  Also partial fix to ToS
                Integer.cs: DownTo and UpTo fixes
                Numeric.cs: Remainder and Step fixes.

(There are now only 2 failures in Numeric, 1 in Float and a number in Bignum.)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20081009/0cbf454e/attachment.html>


More information about the Ironruby-core mailing list