[Ironruby-core] More fixes for core/array specs
suppakilla at gmail.com
Sat Apr 18 18:38:59 EDT 2009
just a self-reply to point out that the changes in ArrayOps.ToArray
introduced a regression for Array#to_ary (reported by
core/array/to_ary) due to the different behaviour of Array#to_a and
Array#to_ary on subclasses of Array. I have already fixed this on my
local repository, I'm just waiting to have a few more commits to
rebase and push the next set of changes on github.
On Sat, Apr 18, 2009 at 13:31, Daniele Alessandri <suppakilla at gmail.com> wrote:
> This commit resolves the following failing specs:
> Array#<=> properly handles recursive arrays (critical)
> Array#initialize with (size, object=nil) uses the block value instead
> of using the default value
> Array#to_a does not return subclass instance on Array subclasses
> Array#uniq! raises a TypeError on a frozen array if modification would
> take place
> Array#values_at returns an array of elements in the ranges when passes ranges
> From the commit message:
> * Removed unused allocateStorage argument from IListOps.First and
> IListOps.Last; Removed the manual protocol conversion bits in
> * Fix IoOps.ToPrintedString to output a double converted to string
> using an invariant culture (Issue #597 NumericLiterals1 test fails
> under french culture)
> * IListOps.ValuesAt adds a null value to the result if the begin value
> of the specified range instance(s) is <= the length of the array and
> the end value is >= the length of the array.
> * Array#uniq! raises a TypeError on a frozen array if modification
> would take place.
> * Fixed ArrayOps.ToArray to return a new RubyArray instance with the
> elements of self if self is a RubyArray.Subclass.
> * Array#initialize with (size, object=nil) uses the block value
> instead of using the default value
> * Fixed IListOps.Compare to work with recursive arrays.
> A diff is attached to this mail.
> PS: This completes my previous commit by removing unused
> allocateStorage arguments from IListOps.First and IListOps.Last.
> Daniele Alessandri
More information about the Ironruby-core