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

2008-03-16 Thread James Keenan via RT
Resolved per discussion with ptc.

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

2008-03-16 Thread Paul Cochrane
On 16/03/2008, James Keenan via RT <[EMAIL PROTECTED]> wrote: > Paul, > > I see that you touched src/pmc/pair.pmc after your last post in this thread. > > > r22490 | paultcochrane | 2007-10-25 16:10:19 -0400 (Thu, 25 Oct 20

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

2007-10-12 Thread Paul Cochrane via RT
On Mon Oct 08 12:14:07 2007, leo wrote: > 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 map

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: [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 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

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 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

[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