develooper Front page | perl.perl5.porters | Postings from September 2012

[perl #74400] [PATCH] Combine Exporter::Heavy back into Exporter.pm, and enable strict (sans refs) for Exporter

Thread Previous
From:
James E Keenan via RT
Date:
September 21, 2012 19:35
Subject:
[perl #74400] [PATCH] Combine Exporter::Heavy back into Exporter.pm, and enable strict (sans refs) for Exporter
Message ID:
rt-3.6.HEAD-11172-1348281347-370.74400-15-0@perl.org
On Wed Jul 04 15:23:41 2012, autarch wrote:
> On Wed, 4 Jul 2012, Jesse Luehrs via RT wrote:
> 
> > On Wed, Jul 04, 2012 at 01:43:54AM -0500, Dave Rolsky wrote:
> >> On Tue, 3 Jul 2012, Father Chrysostomos via RT wrote:
> >>
> >>> On Tue Jul 03 21:47:20 2012, doy wrote:
> >>>> I'd like to merge this, but it seems to have bitrotted. Think you
could
> >>>> rebase this on top of current blead?
> >>>
> >>> Is it so necessary to merge Exporter::Heavy into Exporter.pm?  For
Carp,
> >>> the whole purpose was to avoid having to load modules after an error,
> >>> which doesn’t always work.
> >>>
> >>> For Exporter, the most common code is in Exporter.pm, and the least
> >>> common code--rarely used in fact--is in Heavy.pm.
> >>
> >> Yeah, I think that was the objection to this work way back when I
did it.
> >
> > What about the "use strict" aspect?
> 
> That part might be salvageable, although at this point it might be
just as 
> easy to do it over from scratch.
> 
> 
> -dave

Discussion appears to have petered out on this.

Dave, do you want to pursue this cleanup?  If so, can we close this
ticket and have you pursue a new patch against blead?

Thank you very much.
Jim Keenan


---
via perlbug:  queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=74400

Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About