ACE_TAO
ACE_TAO copied to clipboard
fix a bug in the ACE log mechanism when using a '%#T' format string t…
…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.