commit 9d5a61d7c71beb655e67eaf5c685e7add8f7768d
author Andrew <andrewlassalle@chromium.org> Thu Mar 26 13:40:37 2020 -0700
committer Commit Bot <commit-bot@chromium.org> Sat Mar 28 17:20:31 2020 +0000
tree 153e89c077acd3d77b638c8fb89bc2a091daa911
parent 48ec4c85c00fc7be778dd2d1ce37d8f2908f8f17

update_engine: Fix issues reported by clang-tidy

Fix uninitialized variables and some other minor issues reported by
clang-tidy.

BUG=chromium:982837
TEST=cros_workon_make update_engine --test

Change-Id: I305dedb058c9b0787ba2f68feff42afe6810a276
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2122683
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Tested-by: Andrew Lassalle <andrewlassalle@chromium.org>
Commit-Queue: Andrew Lassalle <andrewlassalle@chromium.org>

payload_consumer/delta_performer.cc

diff --git a/payload_consumer/delta_performer.cc b/payload_consumer/delta_performer.cc
index 3263ff7..11cf006 100644
--- a/payload_consumer/delta_performer.cc
+++ b/payload_consumer/delta_performer.cc
@@ -533,7 +533,7 @@
 #define OP_DURATION_HISTOGRAM(_op_name, _start_time)                         \
   LOCAL_HISTOGRAM_CUSTOM_TIMES(                                              \
       "UpdateEngine.DownloadAction.InstallOperation::" _op_name ".Duration", \
-      base::TimeTicks::Now() - _start_time,                                  \
+      (base::TimeTicks::Now() - _start_time),                                \
       base::TimeDelta::FromMilliseconds(10),                                 \
       base::TimeDelta::FromMinutes(5),                                       \
       20);