Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl-Sensitive Sunglasses

Re^2: Support for attributes:get in Attribute::Handlers?

by Thilosophy (Curate)
on Feb 15, 2009 at 22:22 UTC ( #743964=note: print w/replies, xml ) Need Help??

in reply to Re: Support for attributes:get in Attribute::Handlers?
in thread Support for attributes:get in Attribute::Handlers?

shmem++ for this, although this is basically hand-rolling the support, without any help from Attribute::Handlers. It is also rather intrusive, in that you have to amend the original attribute handlers, which could be a problem if those are located in someone else's modules.

I suppose this is how it is with attributes...

  • Comment on Re^2: Support for attributes:get in Attribute::Handlers?

Replies are listed 'Best First'.
Re^3: Support for attributes:get in Attribute::Handlers?
by shmem (Chancellor) on Feb 16, 2009 at 02:34 UTC

    Thank you. - yes, this is hand-rolling, a workaround, and IMHO Attribute::Handlers should shoehorn FETCH_<type>_ATTRIBUTES methods into the caller's namespace, if not already there yet. Best thing to do would be patching Attribute::Handlers, but since I haven't done much with attributes yet, and haven' got (yet) a full understanding neither of what's going on inside Attribute::Handlers nor why it is written as it is; and since the POD states

    There are undoubtedly serious bugs lurking somewhere in code this funky :-) Bug reports and other feedback are most welcome.

    it is quite easy for me to abstain from adding bugs - and patching code of TheDamian's make is a terrible thing to do... ;-)

    You could send TheDamian "other feedback", though.

Log In?

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://743964]
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others chanting in the Monastery: (3)
As of 2022-05-22 14:38 GMT
Find Nodes?
    Voting Booth?
    Do you prefer to work remotely?

    Results (80 votes). Check out past polls.