Skip Menu |

This queue is for tickets about the XML-Twig CPAN distribution.

Report information
The Basics
Id: 40369
Status: rejected
Priority: 0/
Queue: XML-Twig

People
Owner: Nobody in particular
Requestors: mixmaster [...] remailer.privacy.at
Cc:
AdminCc:

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



Subject: suggestion for possible performance enhancement
Date: Sat, 25 Oct 2008 10:43:41 +0200 (CEST)
To: bug-XML-Twig [...] rt.cpan.org
From: "Anonymous Remailer (austria)" <mixmaster [...] remailer.privacy.at>
after seeing the speedup document, i think minor performance enhancement might be achieved by using Data::Alias in subs to access the arguments, or at the very least replace the overly misused idiom 'my $self = shift' with 'my ($self) = @_' to avoid unecessary modifications to the parameter array (@_).
Subject: Re: [rt.cpan.org #40369] suggestion for possible performance enhancement
Date: Wed, 29 Oct 2008 11:34:02 +0100
To: bug-XML-Twig [...] rt.cpan.org
From: mirod <xmltwig [...] gmail.com>
Anonymous Remailer (austria) via RT wrote: Show quoted text
> Sat Oct 25 04:43:59 2008: Request 40369 was acted upon. > Transaction: Ticket created by mixmaster@remailer.privacy.at > Queue: XML-Twig > Subject: suggestion for possible performance enhancement > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: mixmaster@remailer.privacy.at > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=40369 > > > > > after seeing the speedup document, i think minor performance > enhancement might be achieved by using Data::Alias in subs > to access the arguments, or at the very least replace the > overly misused idiom 'my $self = shift' with 'my ($self) = @_' > to avoid unecessary modifications to the parameter array (@_).
Data::Alias requires 5.8.9 or later on Win32 (and AIX), so I don't want to use it. Is my ($self) = @_; really faster than my $self = shift; ? I haven't seen any benchmark that shows that. I'll have a look at it. Thanks. -- mirod