Skip Menu |

This queue is for tickets about the Tangram CPAN distribution.

Report information
The Basics
Id: 24580
Status: patched
Worked: 10 min
Priority: 0/
Queue: Tangram

People
Owner: SAMV [...] cpan.org
Requestors: pt [...] tchorbadjiev.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Tangram::Storage patch, removing bogus sub ref in BEGIN
Date: Fri, 26 Jan 2007 00:45:30 +0200
To: bug-Tangram [...] rt.cpan.org
From: Assen Tchorbadjiev <pt [...] tchorbadjiev.com>
Hello, attached is a minor (cg-diff) patch against the latest HEAD rev. There seems to be a stale alias in the BEGIN block inside Tangram::Storage, to the Tangram::Core::pretty sub, which got moved inside the Tangram::Util package. There is a problem only when the TRACE is being enabled. Regards, Assen
diff --git a/lib/Tangram/Storage.pm b/lib/Tangram/Storage.pm index 842451b..b44711a 100644 --- a/lib/Tangram/Storage.pm +++ b/lib/Tangram/Storage.pm @@ -12,10 +12,6 @@ use Scalar::Util qw(weaken refaddr); use vars qw( %storage_class ); -BEGIN { - *pretty = *Tangram::Core::pretty; -} - sub new { my $pkg = shift;
From: SAMV [...] cpan.org
Thanks, applied as c4f06adaf7e8. Sam. On Thu Jan 25 17:47:59 2007, pt@tchorbadjiev.com wrote: Show quoted text
> > Hello, > > attached is a minor (cg-diff) patch against the latest HEAD rev. > There seems to be a stale alias in the BEGIN block inside > Tangram::Storage, to the Tangram::Core::pretty sub, which got moved > inside the Tangram::Util package. > > There is a problem only when the TRACE is being enabled. > > Regards, > Assen >
Make that 7efbb50e5d78. It's there, anyway :)
Sam has already applied this.