Forum: IronRuby Review: more miscellaneous fixes

Posted by Daniele Alessandri (Guest)
on 2010-08-22 12:23
Attachment: corelib19_20100822A.diff (69,3 KB)
(Received via mailing list)
Next bunch of fixes:
http://github.com/nrk/ironruby/compare/793effd...e7492dc (see also the
attached diff):

e7492dc Make a minor style change.
35d6011 Set taint and trust flags on the resulting packed string in 
Array#pack.
2e9e125 Fix: Array#pop raises a RuntimeError on a frozen array even
when it is empty.
437e5bd Implement support for format specifier options in Time#strftime.
c7ca6f4 Fix some minor failing specs for Time.
f39ca06 Fix failing specs for Comparable.
e9b5ff9 Fix all the failing specs for Bignum.
84bcdb4 Update the specs for Bignum#&, Bignum#| and Bignum#^ (RubySpec
Git eb98e84).

I updated the specs for Bignum#&, Bignum#| and Bignum#^ just because
there was a bug in them (already fixed in RubySpec's repository) when
running on 1.9.
Posted by Daniele Alessandri (Guest)
on 2010-08-22 13:26
(Received via mailing list)
Whoops, forgot to post a question related to the following commit:
http://github.com/nrk/ironruby/commit/437e5bd9104d648a0002017d8240f848e90b5589

Should we just remove the various checks for RubyCompatibility when
encountering them in a case like this one?

Thanks

On Sun, Aug 22, 2010 at 12:11, Daniele Alessandri <suppakilla@gmail.com> 
wrote:
> f39ca06 Fix failing specs for Comparable.
> http://clorophilla.net/
> http://twitter.com/JoL1hAHN
>



--
Daniele Alessandri
http://clorophilla.net/
http://twitter.com/JoL1hAHN
Please log in before posting. Registration is free and takes only a minute.
Existing account (Switch to SSL-encrypted connection)
NEW: Do you have a Google/GoogleMail or Yahoo account? No registration required!
Log in with Google account | Log in with Yahoo account
No account? Register here.