[all-commits] [llvm/llvm-project] f219cd: [lldb] Fix printf formatting of std::time_t second...
Jason Molenda via All-commits
all-commits at lists.llvm.org
Thu Feb 8 09:16:24 PST 2024
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: f219cda7bd43696792ca4668ca5a9fbf55a9f09f
https://github.com/llvm/llvm-project/commit/f219cda7bd43696792ca4668ca5a9fbf55a9f09f
Author: Jason Molenda <jmolenda at apple.com>
Date: 2024-02-08 (Thu, 08 Feb 2024)
Changed paths:
M lldb/source/Plugins/Language/CPlusPlus/LibCxx.cpp
M lldb/test/API/functionalities/data-formatter/data-formatter-stl/libcxx/chrono/TestDataFormatterLibcxxChrono.py
Log Message:
-----------
[lldb] Fix printf formatting of std::time_t seconds (#81078)
This formatter
https://github.com/llvm/llvm-project/pull/78609
was originally passing the signed seconds (which can refer to times in
the past) with an unsigned printf formatter, and had tests that expected
to see negative values from the printf which always failed on macOS. I'm
not clear how they ever passed on any platform.
Fix the printf to print seconds as a signed value, and re-enable the
tests.
More information about the All-commits
mailing list