use default arguments instead of duplicated functions

Author: dorin-ga

include/GameAnalytics/GameAnalytics.h

@@ -138,11 +138,9 @@ namespace gameanalytics
          static bool isThreadEnding();
 
      private:
-         static bool _endThread;
 
-        static bool isSdkReady(bool needsInitialized);
-        static bool isSdkReady(bool needsInitialized, bool warn);
-        static bool isSdkReady(bool needsInitialized, bool warn, std::string const& message);
+        static bool _endThread;
+        static bool isSdkReady(bool needsInitialized, bool warn = true, std::string const& message = "");
     };
 
 } // namespace gameanalytics

source/gameanalytics/GameAnalytics.cpp

@@ -956,18 +956,6 @@ namespace gameanalytics
         return _endThread || threading::GAThreading::isThreadFinished();
     }
 
-    // --------------PRIVATE HELPERS -------------- //
-
-    bool GameAnalytics::isSdkReady(bool needsInitialized)
-    {
-        return isSdkReady(needsInitialized, true);
-    }
-
-    bool GameAnalytics::isSdkReady(bool needsInitialized, bool warn)
-    {
-        return isSdkReady(needsInitialized, warn, "");
-    }
-
     bool GameAnalytics::isSdkReady(bool needsInitialized, bool warn, std::string const& message)
     {
         constexpr std::size_t maxMsgLen = 64u;