Re: [perl #46099] [TODO] Check for existing parent classes in add_parent()

2007-10-07 Thread Allison Randal
Klaas-Jan Stol wrote: I think it should be something like this: /* RT46099 Check we don't already have this parent. */ /* If we have already added a method with this name... */ if (VTABLE_exists_keyed_str(interp, _class->all_parents, VTABLE_name(interp, parent))) {

Re: [svn:parrot] r21891 - branches/pdd15oo/src/pmc

2007-10-07 Thread Allison Randal
chromatic wrote: On Friday 05 October 2007 14:02:32 [EMAIL PROTECTED] wrote: Modified: branches/pdd15oo/src/pmc/namespace.pmc === === --- branches/pdd15oo/src/pmc/namespace.pmc (original) +++ branches/pdd15oo/src/pmc/namesp

Re: [perl #44353] [BUG] Configure.pl: verbose-step option not workingwith named step

2007-10-07 Thread Allison Randal
James E Keenan wrote: Allison Randal wrote: So, how about we make both --fatal-step and --verbose-step accept either a step number or step name. Didn't I hear someone recently remark, "Simplification is a good idea"? ;-) This is doable, but it will take me at least 9 separate test files

wrapping up the OO implementation

2007-10-07 Thread Bob Rogers
From: Allison Randal <[EMAIL PROTECTED]> Date: Mon, 01 Oct 2007 15:58:24 -0700 My goal was to wrap up the pdd15oo branch by the end of September. We're quite close. The two remaining big things are PGE and multiple dispatch, and then test cleanup. Thanks to chromatic and particle

[perl #45737] [PATCH] [NEW] Change interface to all configuration step runstep() methods

2007-10-07 Thread James Keenan via RT
Applied to trunk in r21956.

[perl #46227] [PATCH] Remove passing negative index to array in compilers/imcc/optimizer.c (Coverity CID 20

2007-10-07 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46227] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46227 > This patch is brought to you by the RT command line utility because parrotbug doesn't s

[perl #46225] [PATCH] Remove passing possible negative index to array in src/pmc.c (Coverity CID 22)

2007-10-07 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46225] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46225 > In Coverity CID 22 there is a warning about potentially passing a negative index to an

Re: [perl #46213] [PGE] Bug in longest token matching "||" ?

2007-10-07 Thread Patrick R. Michaud
On Sun, Oct 07, 2007 at 08:52:26AM -0700, Klaas-Jan Stol wrote: > hi, > > while converting lang/PIR into the new Perl6Grammar syntax, I came across > the following. > > token relational_operator { > || '==' > || '!=' > || '<=' > || '<' > || '>' > || '>=' > } > > At first,

[perl #46223] [PATCH] Remove dead code in src/pmc/pair.pmc (Coverity CID 5)

2007-10-07 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46223] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46223 > Coverity Prevent mentions that the code after the C comparison in src/pmc/pair.pmc can

PDD17 - PMCs

2007-10-07 Thread Leopold Toetsch
Hi, some days ago near #parrot: 17:13 if you get a chance, allison has modified pdd17 (pmc). i'm reviewing it, but i think you'd be a better judge of its implications on gc, etc. 17:14 I'll have a look at it - thx Well, xx days after, I had a very brief look at it: - it's mostly a summary o

[perl #46221] [PATCH] Resource leak fix in imcc.y (Coverity CID 30)

2007-10-07 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46221] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46221 > Coverity Prevent has managed to throw up some difficult-to-fix issues in Parrot, howeve

[perl #46213] [PGE] Bug in longest token matching "||" ?

2007-10-07 Thread via RT
# New Ticket Created by Klaas-Jan Stol # Please include the string: [perl #46213] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46213 > hi, while converting lang/PIR into the new Perl6Grammar syntax, I came across the fol