logging-log4j-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ralph Goers <ralph.go...@dslextreme.com>
Subject Re: Config element name confusion
Date Tue, 08 Jul 2014 14:47:49 GMT

On Jul 8, 2014, at 7:32 AM, Gary Gregory <garydgregory@gmail.com> wrote:

> On Mon, Jul 7, 2014 at 1:44 PM, Ralph Goers <ralph.goers@dslextreme.com> wrote:
> Well, If there is only one Filter then it would just be declared. If there is more than
one then it should be:
> 
> <Filters>
>   <Filter>…</Filter>
>   <Filter>…</Filter>
> <Filters>
> 
> Even in this case since there is only the CompositeFilter (which contains the other Filters)
there should only be one Filter and it should be declared as PluginElement(“Filter”).
 The only place PluginElement(“Fitlers”) should be used is on CompositeFilter.  FileAppender
would be wrong to have “Filters” specified.
> 
> Double checking: 
> 
> - The PluginElement on CompositeFilter typed as Filter[] should be named "Filters" (plural).
This is the only Filter[] PluginElement. This does not need fixing.
> - All PluginElement usage typed as Filter should be named "Filter" (singular). This means
fixing some call sites.

call sites?  Do you mean tests?

Ralph



Mime
View raw message