ACE_TAO icon indicating copy to clipboard operation
ACE_TAO copied to clipboard

fix a bug in the ACE log mechanism when using a '%#T' format string t…

Open esohns opened this issue 1 month ago • 2 comments

…o print time periods.

These logging calls invoke ACE::timestamp, which in turn (erroneously) invokes localtime_r (instead of gmtime_r). Localtime(_r) always adds the timezone offset (if any) to the hour field, which results in a wrong result string (when you live anywhere outside of the GMT time zone)

Summary by CodeRabbit

  • New Features

    • Added duration formatting and a new log format specifier (%Y) to print durations in logs.
  • Documentation

    • Clarified timestamp text to "hour:minute:second.microsecond" and documented %Y usage and overflow behavior for long durations.
  • Behavior

    • Log formatting now supports the %Y specifier to render durations and updates output handling accordingly.

✏️ Tip: You can customize this high-level summary in your review settings.

esohns avatar Nov 22 '25 11:11 esohns