DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13947>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=13947

DailyRollingFileAppender.rollover() uses default access

           Summary: DailyRollingFileAppender.rollover() uses default access
           Product: Log4j
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Enhancement
          Priority: Other
         Component: Appender
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


I'd like to override the rollover method to combine the functionality of
RollingLogAppender and DailyRollingFileAppender (i.e. roll daily, but only keep
a limited number of backups). This should be fairly straightforward, create a
new class, override rollover as follows:

public void rollover() throws IOException
{
    super.rollover();
    deleteBackupFilesBeyondLimits();
}

however, rollover is defined with default access, so I'm stuck putting my class
in your package or patching log4j... My suggested fix would be to make the
rollover() method protected as opposed to default.

Thanks!

--
To unsubscribe, e-mail:   <mailto:log4j-dev-unsubscribe@;jakarta.apache.org>
For additional commands, e-mail: <mailto:log4j-dev-help@;jakarta.apache.org>

Reply via email to