Follow-up Comment #4, bug #20249 (project wesnoth):
I'm curious about two spots that don't seem to be doing the right thing in
src/hotkeys.cpp:
Line 477:
if (character < 64 && ctrl) {
Shouldn't that be:
if (0 < character && character < 64 && ctrl) {
The variable "character" can be -1.
==============================
The other spot is around line 764 where it merges the results from testing
KMOD_ALT and KMOD_RMETA inside the #idfef. Shouldn't it be merging the
results from testing KMOD_META and KMOD_RMETA? At the very least it should be
commented if it's supposed to be doing something weird on purpose.
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?20249>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Wesnoth-bugs mailing list
[email protected]
https://mail.gna.org/listinfo/wesnoth-bugs
