Here is an updated version of the patch, with the Temporal.t bug fixed, and Rat.perl made more sensible. Several other test failures I missed the first time have been patched, and several tests added, included a TODO test which demonstrates how this patch is not fully correct for Int div Int.
On Mon, Aug 31, 2009 at 7:11 PM, perl6 via RT<perl6-bugs-follo...@perl.org> wrote: > Greetings, > > This message has been automatically generated in response to the > creation of a trouble ticket regarding: > "[PATCH] Swap meanings of Int / Int and div", > a summary of which appears below. > > There is no need to reply to this message right now. Your ticket has been > assigned an ID of [perl #68898]. > > Please include the string: > > [perl #68898] > > in the subject line of all future correspondence about this issue. To do so, > you may reply to this message. > > Thank you, > perl6-bugs-follo...@perl.org > > ------------------------------------------------------------------------- > There are two -- well, three issues with this patch that I know of: > > 1. div merely has the code for the old Int / Int, but S03 suggests the > edge cases should be a bit different. > 2. Something is broken in temporal.t, presumably indicating something > is broken in Temporal.pm. > 3. Rat.perl is probably a bit silly, considering that "N/M" should work. > > -- > Solomon Foster: colo...@gmail.com > HarmonyWare, Inc: http://www.harmonyware.com > > -- Solomon Foster: colo...@gmail.com HarmonyWare, Inc: http://www.harmonyware.com
0001-Fix-Rat.perl-to-return-N-M-rather-than-Rat.new-N.patch
Description: Binary data
0001-Changes-needed-to-swap-the-meanings-of-div-and-as.patch
Description: Binary data
spectest-changes-needed-to-swap-the-meanings-of-div-and-as.patch
Description: Binary data
