Re: [log4perl-devel] "Filter" attribute to standard appenders

[Mike S. <log...@pe...>]

On Wed, 16 Jul 2003, Alexey Mahotkin wrote:

>         log4perl.appender.DefaultAppender = Log::Log4perl::Appender::Screen
>         # WARNING: here "Filter" should be in title case, not in lowercase
>         log4perl.appender.DefaultAppender.Filter = RelevanceFilter
>
> Unfortunately, first time I've typed the "Filter" in lowercase, and it
> did not attach the filter to screen appender.  No error message, no
> warning.  Very counter-intuitive, especially because filter definition
> itself uses lower-case:
>
>        log4perl.filter.RelevanceFilter = PR::Logger::RelevanceFilter

I agree that it's somewhat confusing -- reason for the strange uppercasing
convention in "log4perl.appender.DefaultAppender.Filter" is that "Filter"
is a special keyword, not a specific appender property, like "file"
or "mode" for a Log::Log4perl::Appender::File appender.

It's probably too harsh to issue a warning in this case, because
theoretically, you could have a specific appender property called
"filter" (although that's not recommended).

It's a compromise.

-- Mike

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