[Ironruby-core] [patch] Fixes for Array#=
Shri.Borde at microsoft.com
Tue Apr 14 17:18:33 EDT 2009
Changes look good! Thanks for the patch, Daniele!
Note that the tags should be deleted once the bug is fixed. That way, running irtests.bat will run the specs you have just enabled and ensure that you really have fixed the issue. Deleting the tags should be in the same commit as the source code fix.
I have deleted the tags and pushed it to
From: ironruby-core-bounces at rubyforge.org [mailto:ironruby-core-bounces at rubyforge.org] On Behalf Of Daniele Alessandri
Sent: Tuesday, April 14, 2009 7:46 AM
To: ironruby-core at rubyforge.org
Subject: [Ironruby-core] [patch] Fixes for Array#=
I've just committed my first patch to IronRuby on my forked
repository, this is the URL for the diff:
This patch fixes 4 failures in core/array/element_set_spec.rb, namely:
Array#= inserts the given elements with [range] which the range is zero-width
Array#= inserts the given elements with [start, length] which length is zero
Array#= inserts the given elements with [range] which the range has
Array#= with [index, count] inserts other section in place defined by idx
The patch itself is nothing special, but it is needed to run
ironruby-hpricot out of the box on IronRuby.
Ironruby-core mailing list
Ironruby-core at rubyforge.org
More information about the Ironruby-core