I'm using a current SVN snapshot (Revision 332025) and think that there
are some issues with the RollingFileAppender:

- There are test cases that seem to work, but they configure the
  appender programmatically
- I'm using a property configuration file and have observed that
  the triggeringPolicy is ManualTriggeringPolicy instead of the expected
  SizeBasedTriggeringPolicy
- Reason for this (correct me if I didn't understand the code correctly):
  The MaxFileSize and MaxBackupIndex properties are
  ignored, probably because RollingFileAppender::setOption() is missing.
- There is code to interpret MaxFileSize in
  sizebasedtriggeringpolicy.cpp, but when I stepped through the parser,
  this code wasn't reached
- According to rollingfileappender.h RollingFileAppender should be
  derived from FileAppender, but the code says Appender:

>  /** RollingFileAppender extends FileAppender to backup the log files when 
> they reach a certain size. */
>  class LOG4CXX_EXPORT RollingFileAppender : public 
> log4cxx::helpers::ObjectImpl, public Appender

  Which one is correct?

  
Regards, Peter
-- 
    _   _        Peter Steiner <[EMAIL PROTECTED]>
  / /_/ /        Hug-Witschi AG <http://www.hugwi.ch/>
 /  _  /         Electronic Engineering
/_/ /_/  _   _   Auriedstrasse 10
   / / / / / /   CH-3178 Boesingen
  / /_/ /_/ /    Tel  +41 31 740 44 44
 /_ _ _ _ _/     Fax  +41 31 740 44 45

Reply via email to