commit ae4d05384bebad96510969e859dc2af1bd6b81c4
author Prabir Pradhan <prabirmsp@google.com> Mon Oct 31 17:50:04 2022 +0000
committer Prabir Pradhan <prabirmsp@google.com> Mon Oct 31 21:55:37 2022 +0000
tree 3d24bdf9c9dfc59be9f0ebb49b33a5bc8e193346
parent 15c01e27967cbf5df7dd0c6adb3059063c4d05be

Use std::array and default copy assignment for PointerCoords

Bug: 245989146
Test: atest inputflinger_tests
Change-Id: I5e008d03184204a2f34f369e2d5958f6cd4de952

services/inputflinger/InputCommonConverter.cpp

diff --git a/services/inputflinger/InputCommonConverter.cpp b/services/inputflinger/InputCommonConverter.cpp
index 6db89d4..628ce6f 100644
--- a/services/inputflinger/InputCommonConverter.cpp
+++ b/services/inputflinger/InputCommonConverter.cpp
@@ -304,8 +304,8 @@
         common::PointerCoords coords;
         // OK to copy bits because we have static_assert for pointerCoords axes
         coords.bits = args.pointerCoords[i].bits;
-        coords.values = std::vector<float>(args.pointerCoords[i].values,
-                                           args.pointerCoords[i].values +
+        coords.values = std::vector<float>(args.pointerCoords[i].values.cbegin(),
+                                           args.pointerCoords[i].values.cbegin() +
                                                    BitSet64::count(args.pointerCoords[i].bits));
         outPointerCoords.push_back(coords);
     }