commit 0fb1a0e904a10bd24fbbaa3324c94c26833e3925
author Siarhei Vishniakou <svv@google.com> Tue Oct 22 11:23:36 2019 -0700
committer Siarhei Vishniakou <svv@google.com> Tue Oct 22 14:00:48 2019 -0700
tree fc76d28987d7e9437c66377e2d90d2338b08d462
parent 524027761e00c014fbe7e52741ca82bf6e6f1dca

Use enum class for DropReason

Using enum class provides type safety and helps ensure there are no
missed switch/case statements at compile time.

Bug: 70668286
Test: presubmit
Change-Id: Ia3d761d2aec0694edcf8ce24643f17e25cd79a1b

services/inputflinger/dispatcher/InputDispatcher.h

diff --git a/services/inputflinger/dispatcher/InputDispatcher.h b/services/inputflinger/dispatcher/InputDispatcher.h
index 1f906e4..9d22e2c 100644
--- a/services/inputflinger/dispatcher/InputDispatcher.h
+++ b/services/inputflinger/dispatcher/InputDispatcher.h
@@ -113,14 +113,13 @@
     virtual status_t pilferPointers(const sp<IBinder>& token) override;
 
 private:
-
-    enum DropReason {
-        DROP_REASON_NOT_DROPPED = 0,
-        DROP_REASON_POLICY = 1,
-        DROP_REASON_APP_SWITCH = 2,
-        DROP_REASON_DISABLED = 3,
-        DROP_REASON_BLOCKED = 4,
-        DROP_REASON_STALE = 5,
+    enum class DropReason {
+        NOT_DROPPED,
+        POLICY,
+        APP_SWITCH,
+        DISABLED,
+        BLOCKED,
+        STALE,
     };
 
     sp<InputDispatcherPolicyInterface> mPolicy;