Skip Menu |

This queue is for tickets about the Text-Balanced CPAN distribution.

Report information
The Basics
Id: 85572
Status: resolved
Priority: 0/
Queue: Text-Balanced

People
Owner: Nobody in particular
Requestors: nick [...] ccl4.org
Cc:
AdminCc:

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



Subject: Text::Balanced has C<use SelfLoader>, but no longer needs it.
Text::Balanced has C<use SelfLoader>, but no longer needs it. Version 1.51 of Text::Balanced first added the use of SelfLoader, to enable gen_extract_tagged() to be conditionally loaded. Version 1.76 removed the use of SelfLoader (and the __DATA__ token), but did not remove the C<use SelfLoader> line. The line is superfluous. It's a minor waste of resources. It also happens to be the only use of SelfLoader in the core distribution. Please could it go. Trivial patch attached.
Subject: TBnoSL.patch
diff --git a/cpan/Text-Balanced/lib/Text/Balanced.pm b/cpan/Text-Balanced/lib/Text/Balanced.pm index 07d9567..280a1f5 100644 --- a/cpan/Text-Balanced/lib/Text/Balanced.pm +++ b/cpan/Text-Balanced/lib/Text/Balanced.pm @@ -6,7 +6,6 @@ package Text::Balanced; use 5.005; use strict; use Exporter (); -use SelfLoader; use vars qw { $VERSION @ISA %EXPORT_TAGS }; BEGIN {
Subject: Re: [rt.cpan.org #85572] Text::Balanced has C<use SelfLoader>, but no longer needs it.
Date: Wed, 2 Sep 2015 11:41:12 +0100
To: Steve Hay via RT <bug-Text-Balanced [...] rt.cpan.org>
From: Nicholas Clark <nick [...] ccl4.org>
On Wed, Sep 02, 2015 at 04:02:06AM -0400, Steve Hay via RT wrote: Show quoted text
Thanks. Nicholas Clark