On Sun, Oct 20, 2013 at 9:51 AM, Roberto E. Vargas Caballero <k...@shike2.com> wrote: > I think this patch is correct, because XK_NO_MOD is 0, so if there is > some modifier then 'state == mask' will be false. After the simplification, > match is too short and does nothing, so I think it should be > merged into kmap.
Note that match() is used in four places, so I can't merge it into kmap(). On Sat, Oct 19, 2013 at 12:43 PM, Alexander S. <alex0pla...@gmail.com> wrote: > However, I do not like using != as a logical operator. I'd suggest > using (IS_SET(MODE_APPKEYPAD) ? kp->keypad < 0 : kp->keypad > 0), I've done this, but I'm not sure it reads as well. The existing version first checks the *kp fields because if they are zero, the is no need to peek at term.mode. -Mark
0001-Simplify-logic-in-match.patch
Description: Binary data
0002-Simplify-logic-in-kmap.patch
Description: Binary data