Hoist version string to namespace level

src/ripple/protocol/BuildInfo.h

@@ -57,9 +57,6 @@ getCurrentProtocol();
 /** The oldest protocol version we will accept. */
 ProtocolVersion const& getMinimumProtocol ();
 
-char const*
-getRawVersionString();
-
 } // BuildInfo (DEPRECATED)
 
 std::string

src/ripple/protocol/impl/BuildInfo.cpp

@@ -27,22 +27,15 @@ namespace ripple {
 
 namespace BuildInfo {
 
-char const* getRawVersionString ()
+//------------------------------------------------------------------------------
-{
+char const* const versionString =
-    static char const* const rawText =
 
     //--------------------------------------------------------------------------
-    //
+    //  The build version number. You must edit this for each release
-    //  The build version number (edit this for each release)
+    //  and follow the format described here at http://semver.org/
     //
         "0.32.1"
 
-    //
-    //  Must follow the format described here:
-    //
-    //  http://semver.org/
-    //
-
 #if defined(DEBUG) || defined(SANITIZER)
        "+"
 #ifdef DEBUG
@@ -60,9 +53,6 @@ char const* getRawVersionString ()
     //--------------------------------------------------------------------------
     ;
 
-    return rawText;
-}
-
 ProtocolVersion const&
 getCurrentProtocol ()
 {
@@ -108,18 +98,19 @@ std::string const&
 getVersionString ()
 {
     static std::string const value = [] {
-        std::string const versionString = getRawVersionString ();
+        std::string const s = versionString;
         beast::SemanticVersion v;
-        if (!v.parse (versionString) || v.print () != versionString)
+        if (!v.parse (s) || v.print () != s)
-            LogicError (versionString + ": Bad server version string");
+            LogicError (s + ": Bad server version string");
-        return versionString;
+        return s;
     }();
     return value;
 }
 
 std::string const& getFullVersionString ()
 {
-    static std::string const value = "rippled-" + getVersionString ();
+    static std::string const value =
+        "rippled-" + getVersionString();
     return value;
 }
 

src/ripple/protocol/tests/BuildInfo.test.cpp

@@ -27,16 +27,6 @@ namespace ripple {
 class BuildInfo_test : public beast::unit_test::suite
 {
 public:
-    void testVersion ()
-    {
-        testcase ("version");
-
-        beast::SemanticVersion v;
-
-        BEAST_EXPECT(v.parse (BuildInfo::getRawVersionString ()));
-    }
-
-
     ProtocolVersion
     from_version (std::uint16_t major, std::uint16_t minor)
     {
@@ -97,7 +87,6 @@ public:
 
     void run ()
     {
-        testVersion ();
         testValues ();
         testStringVersion ();
         testVersionPacking ();