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.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/util/statistics_registry.h b/src/util/statistics_registry.h
index 1206a3e4a..bd7f87b44 100644
--- a/src/util/statistics_registry.h
+++ b/src/util/statistics_registry.h
@@ -137,7 +137,7 @@ public:
    */
   virtual void safeFlushStat(int fd) const {
     if (__CVC4_USE_STATISTICS) {
-      safe_print(fd, d_name.c_str());
+      safe_print(fd, d_name);
       safe_print(fd, ", ");
       safeFlushInformation(fd);
     }