Ironruby core - Aug 2010 - Review: more miscellaneous fixes

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.

-- 
Daniele Alessandri
http://clorophilla.net/
http://twitter.com/JoL1hAHN
-------------- next part --------------
A non-text attachment was scrubbed...
Name: corelib19_20100822A.diff
Type: application/octet-stream
Size: 70957 bytes
Desc: not available
URL:
<http://rubyforge.org/pipermail/ironruby-core/attachments/20100822/8bbce2f5/attachment-0001.obj>

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 at
gmail.com> wrote:
> 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.
>
> --
> Daniele Alessandri
> http://clorophilla.net/
> http://twitter.com/JoL1hAHN
>


-- 
Daniele Alessandri
http://clorophilla.net/
http://twitter.com/JoL1hAHN