b4n commented on this pull request.


>               mods |= GEANY_PRIMARY_MOD_MASK;
+               mods &= ~GDK_MOD2_MASK;

Unless I'm mistaken, in the expression above `^` has priority over `|` so it's 
the same as `(mods ^ GDK_MOD2_MASK) | GEANY_PRIMARY_MOD_MASK`.
But the simple fact you said what you did makes your version better, as it's 
then less confusing.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1636#discussion_r144423385

Reply via email to