Skip Menu |

This queue is for tickets about the Object-InsideOut CPAN distribution.

Report information
The Basics
Id: 106661
Status: resolved
Priority: 0/
Queue: Object-InsideOut
People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 3.99
Fixed in: (no value)

History Show all quoted text
  Wed Aug 26 03:20:08 2015 ppisar [...] redhat.com - Ticket created
Subject: Forgotten print() in Object::InsideOut::create_accessors()?
3.99 version adds a print() call in Object::InsideOut::create_accessors(): --- Object-InsideOut-3.98/lib/Object/InsideOut.pm 2013-10-03 21:34:54.000000000 +0200 +++ Object-InsideOut-3.99/lib/Object/InsideOut.pm 2015-08-22 02:48:44.000000000 +0200 @@ -2780,6 +2780,7 @@ } } + print($code); # Compile the subroutine(s) in the smallest possible lexical scope my @errs; local $SIG{'__WARN__'} = sub { push(@errs, @_); }; It looks like a forgotten debugging code to me. Should the line be removed?
  Wed Aug 26 07:21:12 2015 jdhedden [...] cpan.org - Correspondence added
D'oh! 4.01 uploaded. Sorry about that. On 2015-08-26 03:20:08, ppisar wrote: Show quoted text
> 3.99 version adds a print() call in > Object::InsideOut::create_accessors(): > > --- Object-InsideOut-3.98/lib/Object/InsideOut.pm 2013-10-03 > 21:34:54.000000000 +0200 > +++ Object-InsideOut-3.99/lib/Object/InsideOut.pm 2015-08-22 > 02:48:44.000000000 +0200 > @@ -2780,6 +2780,7 @@ > } > } > > + print($code); > # Compile the subroutine(s) in the smallest possible lexical > scope > my @errs; > local $SIG{'__WARN__'} = sub { push(@errs, @_); }; > > It looks like a forgotten debugging code to me. Should the line be > removed?
  Wed Aug 26 07:21:12 2015 The RT System itself - Status changed from 'new' to 'open'
  Wed Aug 26 07:21:13 2015 jdhedden [...] cpan.org - Status changed from 'open' to 'resolved'
  Wed Aug 26 09:57:35 2015 ppisar [...] redhat.com - Correspondence added
From: ppisar [...] redhat.com
Dne St 26.srp.2015 07:21:12, JDHEDDEN napsal(a): Show quoted text
> D'oh! 4.01 uploaded. Sorry about that. >
I'm sorry, but I still can see the line in 4.01. Moreover the most recent time stamp in Changes has changed. Entry for 4.01 is missing there. -- Petr
  Wed Aug 26 09:57:42 2015 ppisar [...] redhat.com - Status changed from 'resolved' to 'open'
  Wed Aug 26 21:01:10 2015 jdhedden [...] cpan.org - Correspondence added
That's what I get for trying to do a hurried job just before going to work. I've uploaded 4.02 which should be okay. On 2015-08-26 09:57:35, ppisar wrote: Show quoted text
> Dne St 26.srp.2015 07:21:12, JDHEDDEN napsal(a):
> > D'oh! 4.01 uploaded. Sorry about that. > >
> I'm sorry, but I still can see the line in 4.01. Moreover the most > recent time stamp in Changes has changed. Entry for 4.01 is missing > there. > > -- Petr
  Wed Aug 26 21:01:11 2015 jdhedden [...] cpan.org - Status changed from 'open' to 'resolved'