Curt,

Just looked at the code. It answered my question regarding the setOption1, ..., setOptionN. Just use an aggregated data type instead of a simple one.

I propose the following clean up would make the code in PatternConverter simpler.

- Remove the setOption method
- Change PatternConverter.setOptions to be implemented as {}, i.e. NOP.
- Change sub-classes of PatternConverter dealing with options override the NOP implementaton in PatternConverter.setOptions.


OK?


-- Ceki G�lc�

  The complete log4j manual: http://qos.ch/log4j/



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to