remove old configs from annotations

Author: Gogoshika-ga

source/gameanalytics/GAState.cpp

@@ -19,6 +19,7 @@
 #include <climits>
 #include <string.h>
 #include <stdio.h>
+#include <shared_mutex>
 
 
 namespace gameanalytics
@@ -1160,18 +1161,36 @@ namespace gameanalytics
 
         json GAState::getRemoteConfigAnnotations()
         {
-            json configs;
-            for(json& obj : _configurations)
-            {
-                json cfg;
-                cfg["vsn"] = utilities::getOptionalValue<int>(obj, "vsn", 0);
-                cfg["key"] = utilities::getOptionalValue<std::string>(obj, "key", "");
-                cfg["id"] = utilities::getOptionalValue<std::string>(obj, "id", "");
+            static json cachedConfigsAnnotation;
+            static json lastConfigurations;
+            static std::shared_mutex configsMutex;
 
-                configs.push_back(cfg);
+            {
+                std::shared_lock lock(configsMutex);
+                if (_configurations == lastConfigurations) {
+                    return cachedConfigsAnnotation;
+                }
             }
 
-            return configs;
+            {
+                std::unique_lock lock(configsMutex);
+                if (_configurations != lastConfigurations) {
+                    cachedConfigsAnnotation = json::array();
+                    for (json& obj : _configurations) {
+                        json cfg;
+                        cfg["vsn"] = utilities::getOptionalValue<int>(obj, "vsn", 0);
+
+                        if (cfg["vsn"] == 0) continue;
+
+                        cfg["key"] = utilities::getOptionalValue<std::string>(obj, "key", "");
+                        cfg["id"]  = utilities::getOptionalValue<std::string>(obj, "id", "");
+
+                        cachedConfigsAnnotation.push_back(cfg);
+                    }
+                    lastConfigurations = _configurations;
+                }
+                return cachedConfigsAnnotation;
+            }
         }
     }
 }