diff options
author | Andres Noetzli <noetzli@stanford.edu> | 2017-05-15 08:39:16 -0700 |
---|---|---|
committer | Andres Noetzli <noetzli@stanford.edu> | 2017-05-15 14:53:38 -0700 |
commit | 41460b380a9538e2bd06c42c25ccf20f0644f600 (patch) | |
tree | 682070ddc170cfd4aa9ebfcce08475b949c5d9e0 /src/util/statistics.cpp | |
parent | 31681c7ff2a1469f5efc325fc1b3a406e3a85949 (diff) |
Minor fix in safe_print function
This commit fixes two issues reported by Coverity:
- Fixes the check whether the buffer is full in safe_print_hex
- Removes dead code in safe_print for floating-point values
Additionally, it fixes an issue reported by Andy where the names of the
statistics were printed as "<unsupported>" due to calling the const char*
version instead of the std::string version of safe_print.
Finally, this fixes an issue where --segv-spin would not print the program name
because it was a const char*. The program name is now stored as a string.
NOTE: As a side effect, the last part also fixes Coverity issue 1362944, which
has been in CVC4 for a long time.
Diffstat (limited to 'src/util/statistics.cpp')
-rw-r--r-- | src/util/statistics.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/util/statistics.cpp b/src/util/statistics.cpp index 73ea5b1b1..9423c9379 100644 --- a/src/util/statistics.cpp +++ b/src/util/statistics.cpp @@ -122,8 +122,8 @@ void StatisticsBase::safeFlushInformation(int fd) const { for (StatSet::iterator i = d_stats.begin(); i != d_stats.end(); ++i) { Stat* s = *i; if (d_prefix.size() != 0) { - safe_print(fd, d_prefix.c_str()); - safe_print(fd, s_regDelim.c_str()); + safe_print(fd, d_prefix); + safe_print(fd, s_regDelim); } s->safeFlushStat(fd); safe_print(fd, "\n"); |