Skip Menu |

This queue is for tickets about the podlators CPAN distribution.

Report information
The Basics
Id: 83253
Status: resolved
Priority: 0/
Queue: podlators

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

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



Subject: Pod::Man speedups
Date: Sun, 10 Feb 2013 16:54:36 +0100
To: bug-podlators [...] rt.cpan.org
From: ilmari [...] ilmari.org (Dagfinn Ilmari Mannsåker)
Hi, Inspired by RJBS' suggestion to profile the core's installman script (https://rt.perl.org/rt3/Public/Bug/Display.html?id=116485), I've come up with a few speed improvements to Pod::Man. They're available in a branch in my Github clone of the podlators repo: git://github.com/ilmari/podlators.git speedup-pod-man https://github.com/ilmari/podlators/commits/speedup-pod-man They're all pretty trivial and uncontroversial, except the inlining of ->formatting(), which you can leave out if that's meant to be overridable by subclasses. All together, these changes reduce the time spent in Pod::Man during an installman run by about 10% NYTProf. -- "I use RMS as a guide in the same way that a boat captain would use a lighthouse. It's good to know where it is, but you generally don't want to find yourself in the same spot." - Tollef Fog Heen
On 2013-02-10 07:54:51, ilmari@ilmari.org wrote: Show quoted text
> Hi, > > Inspired by RJBS' suggestion to profile the core's installman script > (https://rt.perl.org/rt3/Public/Bug/Display.html?id=116485), I've come > up with a few speed improvements to Pod::Man. They're available in a > branch in my Github clone of the podlators repo:
Thanks! These have been applied for the next release.
Fixed in 2.5.2. Thanks!