Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the CGI CPAN distribution.

Report information
The Basics
Id: 9023
Status: resolved
Priority: 0/
Queue: CGI
People
Owner: Nobody in particular
Requestors: tshinnic [...] io.com
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sun Dec 19 18:59:55 2004 Guest - Ticket created
Subject: Duplicated code entered in 3.01 - no error, no symptoms
Seeking understanding of new()'s order of operations (again) I got cross-eyed in routine init(), lines 511 thru 539 in version 3.05. After getting another opinion, it appears that indeed this code block is duplicated since 3.01. Perhaps part of a patch that had extraneous code? Because it is duplicated and ends with a "last METHOD;", the same initial conditions never cause a second execution of the code. Thus no symptoms. Code begins at line 511 and again at line 526 with if (defined($fh) && ($fh ne '')) { (I only wish I were less confused now than before (sigh))
  Thu Apr 16 23:25:04 2009 MARKSTOS [...] cpan.org - Correspondence added
Hello, I'm just reviewing the state of bugs on this bug queue. As you are likely already aware, your bug report was confirmed with 3.05 and has sense been addressed-- the duplicate code in init() has been removed. This bug report can be considered resolved. Thanks for the report! Mark On Sun Dec 19 18:59:55 2004, guest wrote: Show quoted text
> Seeking understanding of new()'s order of operations (again) I got > cross-eyed in routine init(), lines 511 thru 539 in version 3.05. > After getting another opinion, it appears that indeed this code > block is duplicated since 3.01. Perhaps part of a patch that had > extraneous code? > > Because it is duplicated and ends with a "last METHOD;", the same > initial conditions never cause a second execution of the code. > Thus no symptoms. > > Code begins at line 511 and again at line 526 with > if (defined($fh) && ($fh ne '')) { > > (I only wish I were less confused now than before (sigh))
  Thu Apr 16 23:25:04 2009 The RT System itself - Status changed from 'new' to 'open'
  Sun Jul 19 21:10:50 2009 MARKSTOS [...] cpan.org - Status changed from 'open' to 'resolved'
  Fri May 23 14:28:25 2014 The RT System itself - Queue changed from CGI.pm to CGI