develooper Front page | perl.perl5.porters | Postings from October 2006

RE: FW: Attribute::Handlers

Thread Previous | Thread Next
From:
David Feldman
Date:
October 27, 2006 07:03
Subject:
RE: FW: Attribute::Handlers
Message ID:
BA9AB162DD5CED46AC03DC5904B19C5B3736CD@tudor.com
1.  I don't see a reason, looking at the code, that this shouldn't work
universally.  The only thing I can think of is that the stack is
different somehow for CODE versus SCALAR handlers invocations, so that
the use of "caller" isn't working for SCALARs.  However, I can't
reproduce this problem.

2.  No, I didn't update the docs because you might not have been
interested in the patch.  Also, since we don't know what's up with #1
above, we should delay on the docs until we're sure my patch actually
works properly.

Maybe you can send me the test program you wrote along with environment
info. like Perl version.  We're using Perl 5.6.1 here since it's the
version that's shipping with Solaris 9 and matches our version of
ActiveState on Windows.

=-Dave-F->

 

-----Original Message-----
From: Rafael Garcia-Suarez [mailto:rgarciasuarez@gmail.com] 
Sent: Friday, October 27, 2006 5:23 AM
To: David Feldman
Cc: perl5-porters@perl.org
Subject: Re: FW: Attribute::Handlers

On 26/10/06, David Feldman <David.Feldman@tudor.com> wrote:
> Rafael,
>
> See the attached output of a context diff against version 0.78.

Thanks.
I had a quick look at your patch, and I have two small remarks :

1. for attributes on scalars (my $foo :Bar), $filename and $linenum
aren't set. Is it possible to handle this case too ? (if not, I'll apply
this patch anyway, only documenting that it works only for subroutine
attributes)

2. You didn't update the docs for the module.

Thread Previous | Thread Next


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