]> asedeno.scripts.mit.edu Git - PuTTY.git/commit
Remove a redundant ?: in the nethack_keypad code.
authorSimon Tatham <anakin@pobox.com>
Wed, 15 Feb 2017 05:21:13 +0000 (05:21 +0000)
committerSimon Tatham <anakin@pobox.com>
Wed, 15 Feb 2017 05:47:16 +0000 (05:47 +0000)
commit54720b2c5a36648864d7bfc6547ef86168db1865
treeccbe84f15e116b91b9e27bdf6285f6ed9a8edaeb
parentefdbe568e2d0ab58aa0330fda392c4c12f92dc1c
Remove a redundant ?: in the nethack_keypad code.

I think all of the cases in this switch must have originally said
(shift_state ? 'this' : 'that'), and in all but the VK_NUMPAD5 case
the two options were different, and I left VK_NUMPAD5 containing a
redundant ?: just to make it line up in a nice table with the others.
But now the others all have more options than that because I had to
support Ctrl as well as Shift modifiers, so there's no reason to have
that silly ?: lingering around (and it annoys Coverity).
windows/window.c