SourceForge has been redesigned. Learn more.
Close

#200 Print some kind of warning when maxFileSIze is too low

v1.1.3
closed
5
2013-11-29
2010-06-14
Psychon
No

Hi,

The FileAppender does this:

this->maxFileSize = (std::max)(maxFileSize_, MINIMUM_ROLLING_LOG_SIZE);

and the user sits there wondering while the file wont be rotated after 1k of output. It would be nice if there could be some kind of message displayed when this is done. Anything to make it more obvious while the maxFileSize from the config file is seemingly ignored.

Thanks.
Uli

Discussion

  • Václav Haisman

    Václav Haisman - 2010-06-14
    • assigned_to: nobody --> wilx
    • milestone: --> Next_Release
     
  • Václav Haisman

    Václav Haisman - 2010-06-14

    I think this is reasonable request.

     
  • Václav Haisman

    Václav Haisman - 2010-06-15

    I have committed a change to PRODUCTION_1_0_x branch in revision 1383 that adds the requested warning.

     
  • Václav Haisman

    Václav Haisman - 2010-06-15
    • status: open --> closed
     
  • Václav Haisman

    Václav Haisman - 2013-07-14

    Ticket moved from /p/log4cplus/feature-requests/24/

     
  • Václav Haisman

    Václav Haisman - 2013-08-14
    • labels: --> enhancement
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -2,7 +2,9 @@
    
     The FileAppender does this:
    
    -this->maxFileSize = \(std::max\)\(maxFileSize\_, MINIMUM\_ROLLING\_LOG\_SIZE\);
    +~~~~~
    +this->maxFileSize = (std::max)(maxFileSize_, MINIMUM_ROLLING_LOG_SIZE);
    +~~~~~
    
     and the user sits there wondering while the file wont be rotated after 1k of output. It would be nice if there could be some kind of message displayed when this is done. Anything to make it more obvious while the maxFileSize from the config file is seemingly ignored.
    
    • status: closed --> open
    • Group: Next_Release --> v1.1.x
     
  • Václav Haisman

    Václav Haisman - 2013-11-29
    • labels: enhancement --> enhancement, fixed
    • status: open --> closed
    • Group: v1.1.x --> v1.1.3
     

Log in to post a comment.