diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2016-10-06 20:57:16 +0200 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@kdab.com> | 2017-01-26 10:33:51 +0000 |
| commit | 9ba16336c51c5843db7c39df30853449972f8ead (patch) | |
| tree | 205bf6e824e1d4bba9a172ccb8819f08b650e14a /doc | |
| parent | 940d667eb41958de120ee759323c67bd7385af0d (diff) | |
QGuiApplication: fix misleading code
The comment says "don't send event if nothing changed", but the
condition it is attached to reads if (velocityOnly), which is
totally opaque.
Change the variable name to stationaryTouchPointChangedVelocity
and drop the then-branch of the if, because it is duplicated in
the following default case of the switch.
The comment makes sense again, and GCC 7 is happy, too, because
this whole issue was pointed out by its -Wimplicit-fallthrough,
which is why I added the fall-through attribute, too.
Change-Id: I0631a381095f8897c55d9440304f6aefbc021a9a
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Reviewed-by: Sérgio Martins <sergio.martins@kdab.com>
Diffstat (limited to 'doc')
0 files changed, 0 insertions, 0 deletions