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: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Peter Bacon Darwin Sent: Thursday, October 09, 2008 7:21 AM To: [email protected] 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: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Jim Deville Sent: Wednesday,08 October 08, 2008 21:44 To: [email protected] 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: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Peter Bacon Darwin Sent: Wednesday, October 08, 2008 10:09 AM To: [email protected] Subject: Re: [Ironruby-core] Code Review (External): NumericalFixes Ping! From: Peter Bacon Darwin [mailto:[EMAIL PROTECTED] Sent: Tuesday,07 October 07, 2008 14:38 To: '[email protected]' 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.)
_______________________________________________ Ironruby-core mailing list [email protected] http://rubyforge.org/mailman/listinfo/ironruby-core
