fixup from comments

Author: AustinBenoit

analytics/src/analytics_desktop.cc

@@ -436,7 +436,11 @@ void SetLogCallback(const LogCallback& callback) {
     std::lock_guard<std::mutex> lock(g_log_callback_mutex);
     g_log_callback = callback;
   }
-  GoogleAnalytics_SetLogCallback(GoogleAnalyticsWrapperLogCallback);
+  if (callback) {
+    GoogleAnalytics_SetLogCallback(GoogleAnalyticsWrapperLogCallback);
+  } else {
+    GoogleAnalytics_SetLogCallback(nullptr);
+  }
 }
 
 // Notify the Analytics SDK about the current state of the app's lifecycle.

analytics/windows/include/public/analytics.h

@@ -155,9 +155,10 @@ class Analytics {
     google_analytics_options->analytics_collection_enabled_at_first_launch =
         options.analytics_collection_enabled_at_first_launch;
     google_analytics_options->app_data_directory =
-        options.app_data_directory.value_or("").empty()
-            ? nullptr
-            : options.app_data_directory.value().c_str();
+        (options.app_data_directory.has_value() &&
+         !options.app_data_directory->empty())
+            ? options.app_data_directory->c_str()
+            : nullptr;
     return GoogleAnalytics_Initialize(google_analytics_options);
   }