Be consistent and hold mutex in any case when returning from condition_t::timedwait().

[loba.git] / options.cpp

diff --git a/options.cpp b/options.cpp
index 0ed04acfde62e493404a795f5d45920f8eadfeea..b31888f90f13b0b2b96e99c12687ffdbfae923e0 100644 (file)
--- a/options.cpp
+++ b/options.cpp
@@ -166,25 +166,24 @@ const char* opt_helper::on_off(bool b)
 
 const char* opt_helper::descr(const char* str)
 {
-    std::string& res = descr_str;
-    res = str;
-    res.resize(DATA_DESCR_WIDTH, '.');
-    return res.c_str();
+    descr_str = str;
+    descr_str.push_back(' ');
+    descr_str.resize(DATA_DESCR_WIDTH, '.');
+    return descr_str.c_str();
 }
 
 template <typename T>
 const char* opt_helper::val_or_string(const T& val, const char* str,
                                       const T& deflt)
 {
-    std::string& res = val_or_string_str;
     if (val != deflt) {
         std::ostringstream oss;
         oss << val;
-        res = oss.str();
+        val_or_string_str = oss.str();
     } else {
-        res = str;
+        val_or_string_str = str;
     }
-    return res.c_str();
+    return val_or_string_str.c_str();
 }
 
 template <typename T>