Fix key modifiers

[crondog]

I end up receiving the event modifier events starting from 1024. Masking fixes this. Is there a nicer way of doing this?

[m910q]

Thanks, this fixes the non-working shortcuts in Ubuntu.

I guess it would be better to pre-calculate the value, instead of doing it in each if-statement.

[crondog]

So i found a nice way of doing it. I ended up creating Mod vars to make things easier to read. From what i have seen elsewhere doing it inside the if is common practice.

Could a mac person please test this as i changed the mac modifier code

[spolu]

I'll test it on both OSX and Linux on Monday!

[spolu]

Looks not bad, can you comply with the coding style (no camelcase).
Maybe we could define a function instead of a bunch of variable names, to test the the masks?

[spolu]

Works well on OSX! :)

[crondog]

Ok I have fixed the var names to be lower case. Also I'm not really sure how we would use a function here instead the the variable names because wouldn't we still need to pass in and/or test that the event key modifier is what we want.

[spolu]

It could take the event as imput, an array of modifiers and a key?