[log4perl-devel] triggers, subs-in-subs

[Kevin M. G. <cp...@go...>]

Hey, Mike, per the note I just sent to Robert Jacobson, it looks like
the implementation of triggers doesn't match the docs.  Something like
this actually gets compiled and run when the config is loaded, but we
just want it compiled, it shouldn't be run until the trigger is called:

 log4perl.appender.Buffer.trigger = sub {return 99};

That results in Buffer.trigger being assigned a value of 99, instead of
a CODE(x123123) ref.

Attached is a quick fix that will produce the desired behavior.  I
haven't fixed the docs in Buffer.pm.  And is there a better way to do
priority comparisons than what I put in the test?  Sorry, I'm in a
hurry, I'll try to look at it more tonight.

What do you think?

-- 
Happy Trails . . .

Kevin M. Goess