On Sat, Aug 10, 2002 at 07:12:52PM -0400, Dan Sugalski wrote:
> At 10:49 PM +0000 8/10/02, Nicholas Clark (via RT) wrote:
> >Appended patch makes the assembler faster. The speedup was 1.8% on on machine
> >I tested on, 1.3% on another. (Cumulative time to loop round all the .pasm
> >files created by the test suite and assemble them)
> >I feel there' still a way to go with speedups.
>
> Applied, thanks. D'you have the message number for the patch in
> limbo, perchance?
[perl #16024]
But it won't apply cleanly as some bugger called "Nicholas Clark (via RT)"
:-) has just sent a patch that reworked the context line
$line=~s/(^\s+|\s+$)//g; # Remove leading and trailing whitespace
as two separate regexps without the alternation.
(that was the only non ///o bit of the other patch. Not benchmarked, but
I seem to remember reading that two regexps for that is faster than
alternation. And why that ///g?
Nicholas Clark
