action: disallow setting an action_field default for all types
authorRan Benita <ran234@gmail.com>
Sun, 2 Sep 2012 13:16:57 +0000 (16:16 +0300)
committerRan Benita <ran234@gmail.com>
Mon, 3 Sep 2012 07:25:20 +0000 (10:25 +0300)
commit4ca85c7b3b46fb8d54d884a13c29cc78967f2313
treeb711f4d1a3b666d093b2087789f6949bb9e3ede3
parentba9f66bda37cbd1ed8826f401bf06b51ae83b050
action: disallow setting an action_field default for all types

Currently where it is possible to write:
    setMods.clearLocks = True;
It's also possible to write:
    action.clearLocks = True;
This will set the default value for the clearLocks action field for
*all* action types, as opposed to just setMods in this case. If
subsequently an action is used for which this field does not make sense,
it will error out.
This doesn't make any sense, because any given field is only possible by at
most 3 or 4 action types... which you might as well write explicitly and
avoid the side effect mentioned above.
Needless to say this is one of xkbcomp's "hidden features" and is not
used anywhere; remove it.

Signed-off-by: Ran Benita <ran234@gmail.com>
src/xkbcomp/action.c