fixup: Formatting and wording changes

Author: AustinBenoit

analytics/src/FirebaseAnalytics.cs

@@ -254,7 +254,7 @@ public static void LogEvent(string name, IEnumerable<Parameter> parameters) {
   ///
   /// @param[in] parameters A parameter array of `Parameter` instances.
   public static void SetDefaultEventParameters(params Parameter[] parameters){
-      SetDefaultEventParameters((IEnumerable<Parameter>)parameters);
+    SetDefaultEventParameters((IEnumerable<Parameter>)parameters);
   }
 
   /// @brief Adds parameters that will be set on every event logged from the SDK.
@@ -269,15 +269,15 @@ public static void SetDefaultEventParameters(params Parameter[] parameters){
   ///
   /// @param[in] parameters An enumerable list of `Parameter` instances.
   public static void SetDefaultEventParameters(IEnumerable<Parameter> parameters){
-      StringList parameterNames = new StringList();
-      VariantList parameterValues = new VariantList();
-      if (parameters != null) {
-        foreach (Parameter p in parameters) {
-          parameterNames.Add(p.Name);
-          parameterValues.Add(Firebase.Variant.FromObject(p.Value));
-        }
+    StringList parameterNames = new StringList();
+    VariantList parameterValues = new VariantList();
+    if (parameters != null) {
+      foreach (Parameter p in parameters) {
+        parameterNames.Add(p.Name);
+        parameterValues.Add(Firebase.Variant.FromObject(p.Value));
       }
-      FirebaseAnalyticsInternal.SetDefaultEventParameters(parameterNames, parameterValues);
+    }
+    FirebaseAnalyticsInternal.SetDefaultEventParameters(parameterNames, parameterValues);
   }
 
   /// Clears all analytics data for this app from the device and resets the app

analytics/src/swig/analytics.i

@@ -78,9 +78,9 @@ void LogEvent(const char* name, std::vector<std::string> parameter_names,
   delete[] parameters;
 }
 
-// Internal version of SetDefaultEventParameters that teakes in two vectors of
-// of know types and converts them into C++ parameters to pass them along to
-// the public SetDefaulteventparameters
+// Internal version of SetDefaultEventParameters that takes in two vectors
+// of known types and converts them into C++ parameters to pass them along to
+// the public SetDefaultEventParameters
 void SetDefaultEventParameters(std::vector<std::string> parameter_names,
               std::vector<firebase::Variant> parameter_values) {
   if (parameter_names.size() != parameter_values.size()) {
@@ -91,10 +91,11 @@ void SetDefaultEventParameters(std::vector<std::string> parameter_names,
   }
   std::vector<Parameter> parameters;
 
-    for(size_t i = 0; i < parameter_names.size(); ++i) {
-      parameters.push_back(Parameter(parameter_names[i].c_str(), parameter_values[i]));
-    }
-    SetDefaultEventParameters(parameters);
+  for(size_t i = 0; i < parameter_names.size(); ++i) {
+    parameters.push_back(Parameter(parameter_names[i].c_str(), parameter_values[i]));
+  }
+
+  SetDefaultEventParameters(parameters);
 }
 
 // Converts from a generic int, int map to the C++ Consent enums

docs/readme.md

@@ -111,8 +111,10 @@ Release Notes
 -------------
 ### Upcoming
 -   Changes
-    - Analytics: Added `SetDefaultEventParameters()` which allows developers to specify a dictionary of parameters that will be set on every event logged.
-    - Analytics: `LogEvent()` now takes a vector of `Parameter` objects, instead of a pointer to an array and a count.
+    - Analytics: Added `SetDefaultEventParameters()` which allows developers to
+      specify a list  of parameters that will be set on every event logged.
+    - Analytics: `LogEvent()` now takes a vector of `Parameter` objects, instead
+	  of a pointer to an array and a count.
 
 ### 13.5.0
 -   Changes