diff options
author | Ran Benita <ran234@gmail.com> | 2012-09-30 10:50:38 +0200 |
---|---|---|
committer | Ran Benita <ran234@gmail.com> | 2012-09-30 14:20:16 +0200 |
commit | 22684cd1dd1fd651803f27f3595e02d16b5df0e8 (patch) | |
tree | 8b3aa5b10bb1638635a352646cd57de0e25738c5 | |
parent | 3b5ada233be2de7e068d27b827b9e1a320791fbe (diff) |
parser: remove XkbCompMapList rule
This rule allows you to put several xkb_keymaps in one file.
This doesn't make any sense: only the default/first can ever be used,
yet the others are fully parsed as well.
Different keymaps should just be put in different files.
Signed-off-by: Ran Benita <ran234@gmail.com>
-rw-r--r-- | src/xkbcomp/parser.y | 10 | ||||
-rw-r--r-- | test/data/keymaps/default.xkb | 15 | ||||
-rw-r--r-- | test/filecomp.c | 2 |
3 files changed, 2 insertions, 25 deletions
diff --git a/src/xkbcomp/parser.y b/src/xkbcomp/parser.y index 5ffb38b..491e96c 100644 --- a/src/xkbcomp/parser.y +++ b/src/xkbcomp/parser.y @@ -172,22 +172,16 @@ _xkbcommon_error(struct YYLTYPE *loc, struct parser_param *param, const char *ms %type <geom> Keys Key OverlayDecl OverlayKeyList OverlayKey OutlineList OutlineInList %type <geom> DoodadDecl %type <file> XkbFile XkbMapConfigList XkbMapConfig -%type <file> XkbCompositeMap XkbCompMapList +%type <file> XkbCompositeMap %% -XkbFile : XkbCompMapList +XkbFile : XkbCompositeMap { $$ = param->rtrn = $1; } | XkbMapConfigList { $$ = param->rtrn = $1; } ; -XkbCompMapList : XkbCompMapList XkbCompositeMap - { $$ = (XkbFile *)AppendStmt(&$1->common, &$2->common); } - | XkbCompositeMap - { $$ = $1; } - ; - XkbCompositeMap : OptFlags XkbCompositeType OptMapName OBRACE XkbMapConfigList CBRACE SEMI diff --git a/test/data/keymaps/default.xkb b/test/data/keymaps/default.xkb deleted file mode 100644 index c40ceb9..0000000 --- a/test/data/keymaps/default.xkb +++ /dev/null @@ -1,15 +0,0 @@ -xkb_keymap { - xkb_keycodes { include "evdev+aliases(qwerty)" }; - xkb_types { include "complete" }; - xkb_compat { include "complete" }; - xkb_symbols { include "pc+us" }; - xkb_geometry { include "pc(pc105)" }; -}; - -default xkb_keymap { - xkb_keycodes { include "evdev+aliases(qwertz)" }; - xkb_types { include "complete" }; - xkb_compat { include "complete" }; - xkb_symbols { include "pc+de" }; - xkb_geometry { include "pc(pc105)" }; -}; diff --git a/test/filecomp.c b/test/filecomp.c index 1d7a127..0c1111a 100644 --- a/test/filecomp.c +++ b/test/filecomp.c @@ -41,8 +41,6 @@ main(void) struct xkb_context *ctx = test_get_context(); assert(test_file(ctx, "keymaps/basic.xkb")); - /* XXX check we actually get qwertz here ... */ - assert(test_file(ctx, "keymaps/default.xkb")); assert(test_file(ctx, "keymaps/comprehensive-plus-geom.xkb")); assert(test_file(ctx, "keymaps/no-types.xkb")); |