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

2007-10-08 Thread Joshua Juran
On Oct 7, 2007, at 12:43 PM, Paul Cochrane (via RT) wrote: # 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 t

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

2007-10-08 Thread Paul Cochrane
On 08/10/2007, Joshua Juran <[EMAIL PROTECTED]> wrote: > On Oct 7, 2007, at 12:43 PM, Paul Cochrane (via RT) wrote: > > > # 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.p

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

2007-10-08 Thread Joshua Juran
On Oct 8, 2007, at 4:36 AM, Paul Cochrane wrote: On 08/10/2007, Joshua Juran <[EMAIL PROTECTED]> wrote: On Oct 7, 2007, at 12:43 PM, Paul Cochrane (via RT) wrote: # New Ticket Created by Paul Cochrane # Please include the string: [perl #46223] # in the subject line of all future corresponde

Parrot Bug Summary

2007-10-08 Thread Parrot Bug Summary
Parrot Bug Summary http://rt.perl.org/rt3/NoAuth/parrot/Overview.html Generated at Mon Oct 8 13:00:02 2007 GMT --- * Numbers * New Issues * Overview of Open Issues * Ticket Status By Version * Requestors with mo

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

2007-10-08 Thread Klaas-Jan Stol
On 10/8/07, Allison Randal <[EMAIL PROTECTED]> wrote: > > 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

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

2007-10-08 Thread Klaas-Jan Stol
On 10/8/07, Klaas-Jan Stol <[EMAIL PROTECTED]> wrote: > > > > On 10/8/07, Allison Randal <[EMAIL PROTECTED]> wrote: > > > > Klaas-Jan Stol wrote: > > > I think it should be something like this: > > > > > > /* RT46099 Check we don't already have this parent. */ > > > > > > /* If we ha

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

2007-10-08 Thread Paul Cochrane
On 08/10/2007, Joshua Juran <[EMAIL PROTECTED]> wrote: > On Oct 8, 2007, at 4:36 AM, Paul Cochrane wrote: > > > On 08/10/2007, Joshua Juran <[EMAIL PROTECTED]> wrote: > >> On Oct 7, 2007, at 12:43 PM, Paul Cochrane (via RT) wrote: > >> > >>> # New Ticket Created by Paul Cochrane > >>> # Please inc

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

2007-10-08 Thread Leopold Toetsch
Am Montag, 8. Oktober 2007 19:05 schrieb Paul Cochrane: > So, the patch is right (however, for my wrong reasoning)?  Is everyone > happy if I apply it then? $ svn ann src/pmc/pair.pmc 8374leo A Pair PMC represents one key => value mapping like a one element hash. I actually can't remem

Re: wrapping up the OO implementation

2007-10-08 Thread Allison Randal
Bob Rogers wrote: I could only find a few find_type hits in t/**/*.t; these are now fixed. (I assume this is because I'm late to the party.) Great contributions all around. Thanks everybody! All of the rest (besides the implementation of the op itself) seems to be in documentation. Attat

[perl #46249] [PATCH] Add missing macro parenthesis in pobj.h

2007-10-08 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46249] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46249 > Hi, in the process of tracking down another issue I noticed (with the help of vim's sy

[perl #46253] [PATCH] Remove reverse inull in imcc/instructions.c (Coverity CID 130)

2007-10-08 Thread via RT
# New Ticket Created by Paul Cochrane # Please include the string: [perl #46253] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=46253 > Hi, Coverity Prevent tells us in CID 130 that the 'next' variable is checked for nulln

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

2007-10-08 Thread James Keenan via RT
No complaints; no smoke failures; resolving ticket.