commit 00fca7c4db1f24701b17b05321abb9701b695de4
author Siarhei Vishniakou <svv@google.com> Tue Oct 29 13:05:57 2019 -0700
committer Siarhei Vishniakou <svv@google.com> Thu Oct 31 23:06:10 2019 +0000
tree 91ff309356d2bc09247a18ff185892fdb97dec6a
parent dcc57af9e27cf5899510e4c9c38cb6170234cb1e

Return vector of cancellation events

Instead of passing a vector to a void function, return the vector from
the function to make ownership explicit.
In the future, we could return a vector of unique_ptr.

Bug: none
Test: none
Change-Id: I9e50e37fff53fd888149f8686c6671a93e4781b1

services/inputflinger/dispatcher/InputState.h

diff --git a/services/inputflinger/dispatcher/InputState.h b/services/inputflinger/dispatcher/InputState.h
index cc72152..a93f486 100644
--- a/services/inputflinger/dispatcher/InputState.h
+++ b/services/inputflinger/dispatcher/InputState.h
@@ -49,8 +49,8 @@
     bool trackMotion(const MotionEntry& entry, int32_t action, int32_t flags);
 
     // Synthesizes cancelation events for the current state and resets the tracked state.
-    void synthesizeCancelationEvents(nsecs_t currentTime, std::vector<EventEntry*>& outEvents,
-                                     const CancelationOptions& options);
+    std::vector<EventEntry*> synthesizeCancelationEvents(nsecs_t currentTime,
+                                                         const CancelationOptions& options);
 
     // Clears the current state.
     void clear();