always-cautious/ortools-clone
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix logging timestamps

Browse Source 
The old code was incorrectly casting a double milliseconds value to
the integer seconds time_t type, which caused incorrect month/day and
timestamp values in log output.

Instead use absl::ToTimespec() to access the tv_sec and tv_nsec values
directly.
This commit is contained in:
Andrew Scherkus
2021-07-06 13:16:53 -04:00
parent f5ffe9af50
commit 06010a46b6
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
ortools/base/logging.cc
View File

@@ -1140,10 +1140,10 @@ void LogMessage::Init(const char* file, int line, LogSeverity severity,
data_->send_method_ = send_method;
data_->sink_ = NULL;
data_->outvec_ = NULL;
double now = ToUDate(absl::Now());
data_->timestamp_ = static_cast<time_t>(now);
timespec now_ts = absl::ToTimespec(absl::Now());
data_->timestamp_ = now_ts.tv_sec;
localtime_r(&data_->timestamp_, &data_->tm_time_);
int usecs = static_cast<int>((now - data_->timestamp_) * 1000000);
int usecs = now_ts.tv_nsec / 1000;
data_->num_chars_to_log_ = 0;
data_->num_chars_to_syslog_ = 0;