http://www.perlmonks.org?node_id=565711


in reply to Re^4: defining methods on the fly
in thread defining methods on the fly

The following modification does not assign to a glob.

Apologies if I'm misunderstanding something here, but isn't the following code assigning to a glob?

*{$package . "::" . $name} = sub :lvalue {

Update: Thanks for the reply -- I was indeed "misunderstanding something here."

Replies are listed 'Best First'.
Re^6: defining methods on the fly
by jdhedden (Deacon) on Aug 05, 2006 at 01:57 UTC
    Apologies if I'm misunderstanding something here, but isn't the following code assigning to a glob?
    *{$package . "::" . $name} = sub :lvalue {
    True, but that wasn't what I was referring to. The above is only executed once. The concern was with the following line in the original code that was executed every time the method was called:
    local *call = sub { $self->{$name}->(@_); };
    By removing that line, my suggested code should not suffer from the method cache reset performance penalty that occurs each time the method in the original code is called.

    Remember: There's always one more bug.