commit ff716cdd33b75e628c3abeed3f79ac6547b51ee5
author Amin Hassani <ahassani@chromium.org> Tue Aug 11 14:37:10 2020 -0700
committer Commit Bot <commit-bot@chromium.org> Thu Aug 13 01:11:30 2020 +0000
tree 9944532fab5d37b57356ba1b3982f5d78cf0b544
parent b57c16e8e593af2984e28cc53ec8cf5750aa8051

update_engine: Fix a lint issue

For some reason the linter is thinking this line is doing C-Style
casting and gives this error:

/mnt/host/source/src/aosp/system/update_engine/update_manager/real_device_policy_provider.cc:108:  Using C-style cast.  Use reinterpret_cast<T*>(...) instead  [readability/casting] [4]

BUG=None
TEST=repo upload doesn't fail anymore.

Change-Id: I5d1e0d96a21fdf1080262e8e04237ccfb809c96f
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2350579
Tested-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Jae Hoon Kim <kimjae@chromium.org>
Commit-Queue: Amin Hassani <ahassani@chromium.org>

update_manager/real_device_policy_provider.cc

diff --git a/update_manager/real_device_policy_provider.cc b/update_manager/real_device_policy_provider.cc
index 781e2ac..55232f3 100644
--- a/update_manager/real_device_policy_provider.cc
+++ b/update_manager/real_device_policy_provider.cc
@@ -104,9 +104,10 @@
 }
 
 template <typename T>
-void RealDevicePolicyProvider::UpdateVariable(AsyncCopyVariable<T>* var,
-                                              bool (DevicePolicy::*getter)(T*)
-                                                  const) {
+void RealDevicePolicyProvider::UpdateVariable(
+    AsyncCopyVariable<T>* var,
+    // NOLINTNEXTLINE(readability/casting)
+    bool (DevicePolicy::*getter)(T*) const) {
   T new_value;
   if (policy_provider_->device_policy_is_loaded() &&
       (policy_provider_->GetDevicePolicy().*getter)(&new_value)) {