commit 024e18f06032583c3e9ba36a376a444ef745080c
author Bart Van Assche <bvanassche@google.com> Thu Feb 24 21:22:07 2022
committer Bart Van Assche <bvanassche@google.com> Thu Mar 03 18:46:12 2022
tree a8a7c63272da34f10890c24d1645998b5f3ec72c
parent 25b2a8de532b632892e6ea70cf8e53277f4a0015

healthd: Make initHealthInfo() easier to read

Use aggregate initialization instead of memberwise initialization. This
patch does not change any functionality.

Test: Compile-tested only.
Change-Id: I9c7831936e449aec8a48336b1ee76d26638e4f72
Signed-off-by: Bart Van Assche <bvanassche@google.com>

healthd/BatteryMonitor.cpp

diff --git a/healthd/BatteryMonitor.cpp b/healthd/BatteryMonitor.cpp
index 4bc9c1c..45882a6 100644
--- a/healthd/BatteryMonitor.cpp
+++ b/healthd/BatteryMonitor.cpp
@@ -121,14 +121,13 @@
 }
 
 static void initHealthInfo(HealthInfo* health_info) {
-    *health_info = HealthInfo{};
-
-    // Enum values may be zero initialized, so they need to be initialized properly.
-    health_info->batteryCapacityLevel = BatteryCapacityLevel::UNSUPPORTED;
-    health_info->batteryChargeTimeToFullNowSeconds =
-            (int64_t)HealthInfo::BATTERY_CHARGE_TIME_TO_FULL_NOW_SECONDS_UNSUPPORTED;
-    health_info->batteryStatus = BatteryStatus::UNKNOWN;
-    health_info->batteryHealth = BatteryHealth::UNKNOWN;
+    *health_info = {
+            .batteryCapacityLevel = BatteryCapacityLevel::UNSUPPORTED,
+            .batteryChargeTimeToFullNowSeconds =
+                    (int64_t)HealthInfo::BATTERY_CHARGE_TIME_TO_FULL_NOW_SECONDS_UNSUPPORTED,
+            .batteryStatus = BatteryStatus::UNKNOWN,
+            .batteryHealth = BatteryHealth::UNKNOWN,
+    };
 }
 
 BatteryMonitor::BatteryMonitor()