Re: [log4perl-devel] Could we have a token in PatternLayout for pid

[<msc...@ao...>]

In a message dated 11/8/2002 12:34:05 PM Eastern Standard Time, ke...@go... writes:

>What do you think?  Once I feel a consensus I need to add error 
>checking, unit tests, and maybe some documentation.

If subclassing hurts performance, it's probably not the right way to go. Also, it might be difficult to subclass PatternLayout. Adding it to PatternLayout should be fine.

One more thought: Is adding Perl code to the conf file the right approach or should this be a perl API along the same lines as Eric's create_custom_level() which has to be called before init()? Typical placeholders probably don't need to be changed as often as the log4perl configuration, but might be more system-specific. Something like a class method as in Log::Log4perl::Layout::PatternLayout->extend("%K", sub {...}) perhaps? Just a thought ...

-- 
-- Mike

############################
# Mike Schilli             #
# log...@pe... #
# http://perlmeister.com   #
# log4perl.sourceforge.net #
############################