lib/base: Make to_hex() easier to use

src/lib/base/String.cpp

@@ -185,8 +185,7 @@ removeFileExt(std::string filename)
     return filename.substr(0, dot);
 }
 
-void
-toHex(std::string& subject, int width, const char fill)
+std::string to_hex(const std::string& subject, int width, const char fill)
 {
     std::stringstream ss;
     ss << std::hex;
@@ -194,7 +193,7 @@ toHex(std::string& subject, int width, const char fill)
         ss << std::setw(width) << std::setfill(fill) << (int)(unsigned char)subject[i];
     }
 
-    subject = ss.str();
+    return ss.str();
 }
 
 void

src/lib/base/String.h

@@ -75,7 +75,7 @@ std::string removeFileExt(std::string filename);
 /*!
 Convert each character in \c subject into hexdecimal form with \c width
 */
-void toHex(std::string& subject, int width, const char fill = '0');
+std::string to_hex(const std::string& subject, int width, const char fill = '0');
 
 //! Convert to all uppercase
 /*!

src/lib/net/SecureUtils.cpp

@@ -23,7 +23,7 @@ std::string format_ssl_fingerprint(const std::string& fingerprint, bool hex, boo
     std::string result = fingerprint;
     if (hex) {
         // to hexadecimal
-        barrier::string::toHex(result, 2);
+        result = barrier::string::to_hex(result, 2);
     }
 
     // all uppercase
@@ -36,4 +36,5 @@ std::string format_ssl_fingerprint(const std::string& fingerprint, bool hex, boo
             result.insert(i * 3 - 1, ":");
         }
     }
+    return result;
 }

src/test/unittests/base/StringTests.cpp

@@ -59,9 +59,7 @@ TEST(StringTests, toHex_plaintext_hexString)
     String subject = "foobar";
     int width = 2;
 
-    string::toHex(subject, width);
-
-    EXPECT_EQ("666f6f626172", subject);
+    EXPECT_EQ("666f6f626172", string::to_hex(subject, width));
 }
 
 TEST(StringTests, uppercase_lowercaseInput_uppercaseOutput)