[crossfire] [PATCH 1/2] Keybindings: multiple changes

[Kevin Zheng]

On 11/03/2013 11:15, Arvid Brodin wrote:
> I think this is a problem with my code. 
> The default Nethack-style bindings look like this (from common/def-keys):
> 
> Logically that means they should work if no modifier keys are pressed, or with
> Run pressed. (Are you saying they work with Fire pressed as well?)

I've cleared out my old key bindings and have the client running from
defaults. The NetHack style keys don't work in "normal" mode, but do
when CTRL is pressed.

> Anyway, this needs to be expressed with two keybindings in my code: one with no
> modifiers (called "None" in the keybindings dialog), and one with the "Run" 
> modifier. My code ignores the 'N' here since it means "no flags" and only looks 
> at the 'R' - which of course breaks things. I will figure out a solution.

I'll try to break them down into two bindings for each command. But this
seems cumbersome and bloated, and deserves to be fixed.

Thanks,
Kevin Zheng