commit | fd45e7404b46acd77c5e2befec7c390462fd9826 | [log] [tgz] |
---|---|---|
author | Lakshman Annadorai <lakshmana@google.com> | Fri Sep 25 05:24:55 2020 +0000 |
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Fri Sep 25 05:24:55 2020 +0000 |
tree | 4b24f8135ff1f5d1f4fa4a7ae5a552f51fac086b | |
parent | 591ad7c7052297ee6606f24245ac0db2dd9bee09 [diff] | |
parent | bfaf79c9c6f0ea02656d2dab14f4d74949afe70e [diff] |
Remove MIN and MAX checking for hidl_enum in UserHalHelper native library. am: bfaf79c9c6 Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/12663093 Change-Id: Icf2fa91a0bf466ce93c4b8365773801c8d7c9755
diff --git a/automotive/vehicle/2.0/utils/UserHalHelper.cpp b/automotive/vehicle/2.0/utils/UserHalHelper.cpp index fcfe4bf..abf59b7 100644 --- a/automotive/vehicle/2.0/utils/UserHalHelper.cpp +++ b/automotive/vehicle/2.0/utils/UserHalHelper.cpp
@@ -141,11 +141,6 @@ template <typename T> Result<T> verifyAndCast(int32_t value) { T castValue = static_cast<T>(value); - const auto iter = hidl_enum_range<T>(); - if (castValue < *iter.begin() || castValue > *std::prev(iter.end())) { - return Error() << "Value " << value << " not in range [" << toString(*iter.begin()) << ", " - << toString(*std::prev(iter.end())) << "]"; - } for (const auto& v : hidl_enum_range<T>()) { if (castValue == v) { return castValue;