diff options
| author | Amin Hassani <ahassani@chromium.org> | 2020-08-11 14:37:10 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-08-13 01:11:30 +0000 |
| commit | ff716cdd33b75e628c3abeed3f79ac6547b51ee5 (patch) | |
| tree | 9944532fab5d37b57356ba1b3982f5d78cf0b544 /update_manager | |
| parent | b57c16e8e593af2984e28cc53ec8cf5750aa8051 (diff) | |
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>
Diffstat (limited to 'update_manager')
| -rw-r--r-- | update_manager/real_device_policy_provider.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/update_manager/real_device_policy_provider.cc b/update_manager/real_device_policy_provider.cc index 781e2acb..55232f3e 100644 --- a/update_manager/real_device_policy_provider.cc +++ b/update_manager/real_device_policy_provider.cc @@ -104,9 +104,10 @@ void RealDevicePolicyProvider::RefreshDevicePolicyAndReschedule() { } 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)) { |