diff options
| author | 2022-10-14 15:17:30 +0000 | |
|---|---|---|
| committer | 2022-10-24 15:40:55 +0000 | |
| commit | bea6ce5a7fceb854f80e40b113c2a7559860639e (patch) | |
| tree | 8974798262857d2e5f620ea4988e95783aa7143b /libs/gui/SurfaceComposerClient.cpp | |
| parent | 7b022c606948d80b29b84c666c639963530847c4 (diff) | |
Split up TouchInputMapper::preparePointerGestures
This monster 700-line method was quite difficult to understand. It still
is, but hopefully splitting it up into a few more methods helps.
Also, activeTouchId was basically just being used as a shorter name for
mPointerGesture.activeTouchId (with both being assigned to at once),
leaving space for a bug to come in if an assignment was made to one but
not the other. Making it a const reference removes that possibility, and
makes it a bit clearer what's going on.
This CL should not result in any behaviour changes.
Bug: none
Test: atest inputflinger_tests
Test: compare touchpad gesture behaviour before and after
Change-Id: Ia85bf5dd80d87bd8fb83332f50dc25ad8769c32e
Diffstat (limited to 'libs/gui/SurfaceComposerClient.cpp')
0 files changed, 0 insertions, 0 deletions