commit 1f2db4f98af16aca68216ab99c783b4cb6c2626f
author Michael Wright <michaelwr@google.com> Thu Nov 24 16:47:38 2022 +0000
committer Michael Wright <michaelwr@google.com> Thu Nov 24 21:51:09 2022 +0000
tree b1346972f271b26aed99f3138c27740c9c7e4582
parent ebd913b1555b0fdc094a7d6f74a2415f3651e9b0

Add explicit copy assignment operator for NotifyMotionArgs.

Right now we rely on an implicit definition of this operator, which is
deprecated since we have an explicit copy constructor. See
-Wdeprecated-copy-with-user-provided-copy for details on why this isn't
desirable. Fortunately, a default implementation of the copy assignment
operator is sufficient, so there's no actual bug in the current usages.

Test: atest NotifyArgs_Test.cpp
Change-Id: I58a913ad1c37414e86167557fdd4deb29499f432

services/inputflinger/tests/Android.bp

diff --git a/services/inputflinger/tests/Android.bp b/services/inputflinger/tests/Android.bp
index e55e121..cf2a96a 100644
--- a/services/inputflinger/tests/Android.bp
+++ b/services/inputflinger/tests/Android.bp
@@ -46,6 +46,7 @@
         "InputDispatcher_test.cpp",
         "InputReader_test.cpp",
         "LatencyTracker_test.cpp",
+        "NotifyArgs_test.cpp",
         "PreferStylusOverTouch_test.cpp",
         "TestInputListener.cpp",
         "UinputDevice.cpp",