commit f8b240e897ed68b90c116ce4c2f8814e71e5aa97
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Wed Mar 17 15:39:53 2021 +1000

    xkbcomp 1.4.5
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit da810f10d732a312ccee88652141c1d165efc114
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:24:00 2021 +1000

    keycodes: compress the high keycode warning
    
    No point spending 3 lines on this, especially given that this warning is
    always triggered.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 80398db098b2a2214c5bef28d09cefd58beac43d
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:20:40 2021 +1000

    keycodes: downgrade the >255 keycode warning to an info
    
    This warning will be triggered all the time now that xkeyboard-config has
    started adding keycodes > 255. Downgrade to an info, there's nothing the user
    can do about this warning anyway.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 1955a2645670eeea83d6d163653749249cd839f2
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:16:53 2021 +1000

    Downgrade the warning for missing symbols to info
    
    In the interest of maintainability, it's easier to include as many keycodes as
    possible and then have the symbols mapping specific to the layout. This is
    particularly true for evdev where the kernel takes care of device-specifics
    and every keyboard has the same set of keycodes anyway.
    
    So let's downgrade this from a warning to a mere info, virtually every
    keyboard right now triggers this warning for a number of keys.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 9510dedad875099c32993246188050ea73ab1a9f
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:08:25 2021 +1000

    Replace WARN[1-9], ERROR[1-9], etc. with their unnumbered version
    
    Those macros date back to when varargs weren't a thing but they've
    been #defined to the same value for 17 years now.
    
    Patch generated with:
            for action in WARN INFO ERROR ACTION FATAL WSGO; do
            sed -i "s/${action}[1-9]/${action}/g" `git ls-files`
            done
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit ff6a19ed8898ee8c79b2b2a2f87806353d89f5d4
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:46:01 2021 +1000

    gitlab CI: add basic build test
    
    Copied from xorgproto
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 3b3d25dd32ba48fd6d15ca98baf7109af21e1d97
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jan 21 13:24:51 2021 +1000

    Remove trailing whitespaces
    
    Let's clean this up so I don't have to fight vim and git in removing them.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 790b998647ec171133bf196a3d84c5153cd64840
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Nov 5 14:24:24 2020 +1000

    xkbcomp 1.4.4
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit a0a4f112c5858ac2b932247dba2d85061bb59145
Author: Miroslav Koškár <mk@mkoskar.com>
Date:   Mon Jul 6 07:48:48 2020 +0200

    Fix lockdevbtn to be XkbSA_LockDeviceBtn action

commit 101d5f2bce9ee13521fe9cb47b7cfd00d01f5914
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Jul 23 10:25:08 2020 -0700

    Fix spelling/wording issues
    
    Found by using:
      codespell --builtin clear,rare,usage,informal,code,names
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit c8cfca25ab2ed65013448f7502bae933c66390ae
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Fri Jun 5 14:23:09 2020 +1000

    Don't pretend unresolved symbols are an error
    
    Whenever xkeyboard-config is newer than xorgproto, or libX11 just hasn't yet
    been rebuilt against the protocol we end up with some unresolved symbols.
    That's not an error, it just happens. Let's downgrade this to a warning
    instead.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit dcaa947c488b4cb458946ab9c19579f7c866cbd4
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Wed Jul 17 15:07:14 2019 +1000

    For -R and after chdir, add the current directory to the path
    
    This restores the behavior pre 1aecdffaa0db7bbf85bc0aae9043e9437b25c30d, at
    least when -R is given. For a normal invocation without -R we don't add the
    current directory, because 1aecdff isn't totally wrong about whether that's a
    good idea.
    
    Fixes https://gitlab.freedesktop.org/xorg/app/xkbcomp/issues/3

commit d373c9b434c603bf6a5eb442b6427992d23ef874
Author: Matt Turner <mattst88@gmail.com>
Date:   Thu Feb 20 09:47:30 2020 -0800

    xkbcomp 1.4.3
    
    Signed-off-by: Matt Turner <mattst88@gmail.com>

commit e200d0d41ef3158ea717206c1490e499a0c07f1b
Author: Dimitry Andric <dimitry@andric.com>
Date:   Wed Feb 19 21:24:59 2020 +0100

    Don't compare with string literals
    
    xkbcomp.c:228:37: error: result of comparison against a string literal is unspecified (use an explicit string comparison function instead) [-Werror,-Wstring-compare]
            if ((argv[i][0] != '-') || (uStringEqual(argv[i], "-")))
                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~
    ./utils.h:195:30: note: expanded from macro 'uStringEqual'
                                     ^~~~~~~~~~~~~~~~~~~~~
    ./utils.h:198:38: note: expanded from macro 'uStringCompare'
                                     (s1)!=(s2):strcmp(s1,s2))
                                         ^ ~~~~
    
    Don't attempt to do this macro trickery, and simply use strcmp instead,
    where it applies.
    
    Bug: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=244235

commit 29fda8e50e4a4a127348e63fcf9f47600beab93c
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jun 13 12:29:19 2019 +1000

    Error out if we have no default path
    
    The path is set through configure.ac/Makefile.am and always defined anyway.
    Let's not re-define it here with a different value than our default.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit ff0e59084d6b9f2e7085fc88ba68916150085afb
Author: Andreas Wettstein <wettstein509@solnet.ch>
Date:   Sat Feb 15 17:34:41 2014 +0100

    xkbcomp Fix missing support for "affect" and incorrect modifier handling for ISOLock
    
    Add missing support for "affect" flag to selectively affect locking or
    unlocking for for modifier locking, control locking, and ISOLock.
    Fix some incorrect masking and modifier handling for ISOLock.
    
    Signed-off-by: Andreas Wettstein <wettstein509@solnet.ch>
    Reviewed-By: Ran Benita <ran234@gmail.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 9edd61b91a02bda31fc6edd2c32c553e872b6ea2
Author: Miroslav Koškár <mk@mkoskar.com>
Date:   Fri Jan 18 20:01:01 2019 +0100

    Fix invalid error report on F_Accel field
    
    F_Accel is valid MovePtr field yet following error is reported:
    
        > Error:            Field accel is not defined for an action of type MovePtr
        >                   Action definition ignored
    
    Signed-off-by: Miroslav Koškár <mk@mkoskar.com>

commit 7ffec6583ae92fe81402870eb89dea21ca34e933
Author: Andreas Boll <andreas.boll.dev@gmail.com>
Date:   Mon Feb 11 21:44:51 2019 +0100

    configure: Remove unused AC_SUBST([REQUIRED_MODULES])
    
    v2: Add square brackets back.
    
    Signed-off-by: Andreas Boll <andreas.boll.dev@gmail.com>

commit 4683410b6b072bd55dbdc661c76a5ba30bae608e
Author: Andreas Boll <andreas.boll.dev@gmail.com>
Date:   Mon Feb 11 21:31:19 2019 +0100

    pkgconfig: Remove unneeded Requires.private
    
    Fixes "pkg-config --exists xkbcomp" without installed pkg-config files
    for x11, xkbfile, and xproto >= 7.0.17.
    
    xkbcomp works fine without those pkg-config files.
    
    https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=913359
    
    Signed-off-by: Andreas Boll <andreas.boll.dev@gmail.com>

commit 1fdbacbe34742328b2a95742982b6017f60a1342
Author: Adam Jackson <ajax@redhat.com>
Date:   Wed Oct 17 11:44:37 2018 -0400

    Suppress high-keycode warnings at the default warning level
    
    We expect evdev to have keycodes above 255, there's no reason to warn
    about this.
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit 32d8d53fc1b622d97b3b8eb7b8b5e8354eda60a5
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Nov 21 17:11:16 2018 -0800

    Update configure.ac bug URL for gitlab migration
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 2abe23d23d3755335c10ff573e4e1f93b682e9d9
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Fri Jun 8 09:10:03 2018 +1000

    xkbcomp 1.4.2
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 5bc2849273a224a2df593b7e8df31f0bde666482
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jun 7 11:25:01 2018 +1000

    Ignore xkb_keycodes.maximum of > 255
    
    Continuation from 7fdfabd75 "keycodes: Ignore high keycodes"
    
    A keymap with a key > 255 will have a xkb_keycodes.maximum of that keycode.
    Let's not throw a fatal error on that, just crop it back to the maximum of
    255. This doesn't set the "high_keycode_warned" on purpose so we get this for
    the first key that actually matters.
    
    Reproducible with xkeyboard-config 2.24 which has a maximum of 374.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1587998
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 34842845b191082fd1cfff3b7038332cdcadb825
Author: Adam Jackson <ajax@redhat.com>
Date:   Thu Mar 1 11:15:18 2018 -0500

    xkbcomp 1.4.1
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit 993d4916a92b56b5043cf0b90f550568c817ad2b
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Feb 13 13:48:46 2018 +1000

    pkgconfig: add the exec_prefix substitution
    
    bindir gets replaced with:
        bindir=${exec_prefix}/bin
    but exec_prefix isn't set so we end up with an invalid bindir.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Jon Turney <jon.turney@dronecode.org.uk>

commit 3e2a6ad4edfbf21c3f76f8319f0039b7f589944f
Author: Adam Jackson <ajax@redhat.com>
Date:   Mon Nov 6 12:54:41 2017 -0500

    pkgconfig: Add our bindir to xkbcomp.pc
    
    Reviewed-by: Daniel Stone <daniels@collabora.com>
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit 3fcd5f4490adee266eb4c0110d058c7f4e515e36
Author: Daniel Stone <daniels@collabora.com>
Date:   Mon May 1 15:59:28 2017 +0100

    xkbcomp 1.4.0
    
    Signed-off-by: Daniel Stone <daniels@collabora.com>

commit 7fdfabd753a39dbdcfcfa65ec33d0192448119d8
Author: Daniel Stone <daniels@collabora.com>
Date:   Thu Apr 6 15:52:10 2017 +0100

    keycodes: Ignore high keycodes
    
    Rather than throwing a fatal error when a keycode definition exceeds the
    declared maximum (i.e. 255), just ignore the definition and continue.
    
    This allows xkeyboard-config to start shipping datasets including high
    keycodes, which will work in xkbcommon as it ignores explicit range
    declarations.
    
    Signed-off-by: Daniel Stone <daniels@collabora.com>
    Reviewed-by: Ran Benita <ran234@gmail.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reported-by: Christian Kellner <gicmo@gnome.org>

commit 9f1e3e8c8914b349b9b52501bb4acadedb31081d
Author: Mihail Konev <k.mvc@ya.ru>
Date:   Thu Jan 26 14:00:21 2017 +1000

    autogen: add default patch prefix
    
    Signed-off-by: Mihail Konev <k.mvc@ya.ru>

commit 6f16a2e38667bea33e0cb566734a87c2b92c9391
Author: Emil Velikov <emil.l.velikov@gmail.com>
Date:   Mon Mar 9 12:00:52 2015 +0000

    autogen.sh: use quoted string variables
    
    Place quotes around the $srcdir, $ORIGDIR and $0 variables to prevent
    fall-outs, when they contain space.
    
    Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit bbc55549ee44830747eccfadd61df3ed38a99202
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Jan 24 10:32:07 2017 +1000

    autogen.sh: use exec instead of waiting for configure to finish
    
    Syncs the invocation of configure with the one from the server.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Emil Velikov <emil.velikov@collabora.com>

commit 8995ac58e15595de9158c2d8df9e02b0c925e8f4
Author: Ran Benita <ran234@gmail.com>
Date:   Wed Jan 6 21:32:52 2016 +0100

    When a file contains several maps, look for a default map
    
    Rather than always taking the first one in the file. This is exactly the
    intended use of the 'default' flag.
    
    Note that pretty much the same code is duplicated in xkbcomp.c when
    compiling a single file from the command line, but there it is
    implemented correctly (look for XkbLC_Default).
    
    https://bugs.freedesktop.org/show_bug.cgi?id=69950
    
    Signed-off-by: Ran Benita <ran234@gmail.com>
    Tested-by: Benno Schulenberg <bensberg@justemail.net>
    Acked-by: Sergey Udaltsov <sergey.udaltsov@gmail.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit bf4d7d059daf5c6f81d70f8d3e5e2a87265381b7
Author: Benno Schulenberg <bensberg@justemail.net>
Date:   Thu Sep 11 22:04:21 2014 +0200

    When overriding a key, adjust also its number of levels (#57242).
    
    Specifying an explicit key type when overriding a key should adjust
    the number of levels to that of the specified type. This gets rid of
    the age-old warning of the right Alt key being ONE_LEVEL but having
    two symbols assigned.
    
    Fixes bug #57242 <http://bugs.freedesktop.org/show_bug.cgi?id=57242>.
    
    Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
    Reviewed-by: Ran Benita <ran234@gmail.com>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit f909259b1da70bc145f19e5ce595c8e945a2a808
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Nov 5 12:19:22 2015 +1000

    xkbcomp 1.3.1
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 26d1c57a938ce5c0d4fd9d645ace25308e6a7070
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Nov 3 09:01:19 2015 +1000

    symbols: increase the warning level for shortening a key type
    
    Any use of the german keyboard layout or anything else using
    level3(ralt_switch) produces the warning
        Type "ONE_LEVEL" has 1 levels, but <RALT> has 2 symbols
        Ignoring extra symbols
    This warning is meaningless, the default definition for RALT comes from
    pc(pc105) including altwin(meta_alt) which defines it as TWO_LEVEL
    and { Alt_R, Meta_R }. Including level3(ralt_switch) correctly sets it as
    ONE_LEVEL but now we get the warning. That makes users file bug reports and we
    can't be having that, can we?
    
    Up the warning level for this case to one past the default. The warning is
    only useful when you're testing a new keyboard layout or trying to fix an
    actual bug with missing symbols. And then you can run xkbcomp with -w10.
    No need to spam the log on every server startup.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Hans de Goede <hdegoede@redhat.com>

commit 511adde23a0518db0379b444580b63edcb9138a8
Author: Matthew Green <mrg@NetBSD.org>
Date:   Tue Jul 28 17:53:48 2015 +0200

    Adjust a NUL-string assignment to avoid const issues.
    
    Signed-off-by: Thomas Klausner <wiz@NetBSD.org>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 1ae525b3d236b59e6437b2b5433d460e18370973
Author: Jon TURNEY <jon.turney@dronecode.org.uk>
Date:   Tue Mar 17 15:19:10 2015 +0000

    Define WIN32_LEAN_AND_MEAN when building on Win32
    
    w32api/rpcnndr.h (included by windows.h) uses TypeString as the name of a structure member.
    
    xkbcomp.h defines it as a macro for a constant.
    
    Define WIN32_LEAN_AND_MEAN to avoid this clash.
    
    Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
    Reviewed-by: David Macek <david.macek.0@gmail.com>

commit fc3e6ddb2c8e922ea80f2dc5cbc1df2102e30d99
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Mar 3 16:24:50 2015 -0800

    Stop including <X11/Xalloca.h> in parseutils.c
    
    I can not find any point in our git history at which alloca() or the
    Xalloca.h macros ALLOCATE_LOCAL() & DEALLOCATE_LOCAL() were actually
    used in this source file.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 1e8ee9d0aad072f04186df84752f5636340574e0
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Fri Nov 21 08:18:41 2014 +1000

    xkbcomp 1.3.0
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 978998a1fe0a78829452f5432efe1b74c5049de4
Author: Vincent Lefevre <vincent@vinc17.net>
Date:   Mon Oct 27 10:50:30 2014 +0100

    xkbcomp: Improved -w option parsing
    
    This patch improves -w option parsing even further, for cases like
    "xkbcomp -w6 4.xkb out.xkb" (which were not handled by the fix of
    the warning level is still optional (set to 0 if not present), and
    errors like "xkbcomp -wfoo in out" are detected and reported.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=66344
    
    Signed-off-by: Vincent Lefevre <vincent@vinc17.net>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit d4e02a09258063c6d024c3ccd42d6b22212e6e18
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Mar 8 19:04:24 2014 -0800

    Initialize nMatch even if WIN32 is defined
    
    Flagged by cppcheck 1.64:
    Checking app/xkbcomp/listing.c: WIN32...
    [app/xkbcomp/listing.c:335]: (error) Uninitialized variable: nMatch
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 7606a46b922dfdb0627f1af6ab1432ae88c79d66
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Mar 8 19:01:12 2014 -0800

    Remove useless assignment to 'outline' variable
    
    Flagged by cppcheck 1.64:
    [app/xkbcomp/geometry.c:2426] -> [app/xkbcomp/geometry.c:2427]:
     (performance) Variable 'outline' is reassigned a value before
                   the old one has been used.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 6c2d3b0891f7be6ef4895be75188aa1c6f530bfc
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Mar 8 18:50:22 2014 -0800

    Don't dereference the pointer whose allocation failed
    
    Flagged by cppcheck 1.64:
    [app/xkbcomp/keycodes.c:264] -> [app/xkbcomp/keycodes.c:262]:
     (warning) Possible null pointer dereference: new - otherwise it
               is redundant to check it against null.
    [app/xkbcomp/keytypes.c:600] -> [app/xkbcomp/keytypes.c:597]:
     (warning) Possible null pointer dereference: old - otherwise it
               is redundant to check it against null.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 50ebe95a32dc59b1de899ef08f72462055b04065
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sat Mar 8 18:46:11 2014 -0800

    Remove useless checks for NULL before free in OverlayKeyCreate()
    
    There is no need to ensure the pointers passed to free are not NULL,
    especially right after passing them to strncpy without checking for
    NULL.
    
    Flagged by cppcheck 1.64:
    [app/xkbcomp/parseutils.c:557] -> [app/xkbcomp/parseutils.c:559]:
     (warning) Possible null pointer dereference: over - otherwise it
               is redundant to check it against null.
    [app/xkbcomp/parseutils.c:558] -> [app/xkbcomp/parseutils.c:561]:
     (warning) Possible null pointer dereference: under - otherwise it
               is redundant to check it against null.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 31b90ee4ffc774e0da540277907fc5540c0b012c
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Mon Feb 3 07:24:22 2014 +1000

    man: replace default include directory with the one from configure
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Gaetan Nadon <memsize@videotron.ca>

commit e3e6e938535532bfad175c1635256ab7fb3ac943
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Oct 20 12:36:31 2013 -0700

    Fix many const char * warnings from gcc
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>

commit 2458580ac95c550217b3376c46eecb2cca646241
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Oct 20 11:52:43 2013 -0700

    Convert remaining sprintf calls to snprintf
    
    Most were fixed length or length checked anyway, this just saves time
    doublechecking that.   (A few could be replaced by asprintf, but we
    don't have a copy guaranteed to be reachable from this program yet.)
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>

commit 3ed68e06cb45fb526b09e4c7b7c3d60de552b2b3
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Oct 20 12:49:31 2013 -0700

    Remove unused uASSERT macro
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>

commit c936bd4315a7dc78de074ac89c5a4d12813421be
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Sun Oct 20 11:58:56 2013 -0700

    Remove unused function entry/exit tracking framework
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Jasper St. Pierre <jstpierre@mecheye.net>

commit 839ccda42d8b088d94324cd77c4be954859914d3
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Wed Aug 28 15:20:56 2013 +1000

    compat: don't warn about redefinition when nothing is defined yet
    
    info->groupCompat[i] is bzero'd on init, define is 0. Don't warn when that
    is the case, otherwise any "group 2 = ..." statement will cause an error.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 24d18e0a844041ef82441adb16aa18cc4b4814ae
Author: Benno Schulenberg <bensberg@justemail.net>
Date:   Wed Aug 28 20:03:30 2013 +0200

    Making sure that a copied string is always null-terminated (#66345).
    
    A more minimalistic and formally correct solution.
    This amends and extends the previous fix for bug #66345,
    fixing not just yyGetKeyName() but also yyGetString().
    
    Signed-off-by: Benno Schulenberg <bensberg@justemail.net>
    
    Fixes a typo from cdcd552 (should be sizeof - 1, not sizeof -i).
    Code flows that i is at most sizeof(scanBuf) - 1, so last is not needed.
    
    Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 0ebdf47fd4bc434ac3d2339544c022a869510738
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jul 11 13:53:31 2013 +1000

    Parse -w1 flag correctly (#66344)
    
    if the flag is "-w", then argv[i][1] is 'w' and unlikely to be a digit.
    Access [2] instead, which is either \0 or an actual digit.
    
    X.Org Bug 66344 <http://bugs.freedesktop.org/show_bug.cgi?id=66344>
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Reviewed-by: Matt Dew <marcoz@osource.org>

commit cdcd552041fc1325a2a81e3374fadb0dd15950dc
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Thu Jul 11 13:26:18 2013 +1000

    Always terminate the scanBuf string (#66345)
    
    If a key name exceeds 4 characters, the content of scanBuf is not
    null-terminated, giving error messages like
    
    syntax error: line 7 of test.xkb
    last scanned symbol is: FOOBARm
    Errors encountered in test.xkb; not compiled.
    
    (last character of the preceding 'maximum' statement in this case)
    
    X.Org Bug 66345 <http://bugs.freedesktop.org/show_bug.cgi?id=66345>
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit 1cd5c50c54b06de2238d6d7675d0a3c65a21414d
Author: Laura <lmemsm@gmail.com>
Date:   Fri Mar 15 11:21:26 2013 +0000

    Add #include <stdlib.h> to xkbscan.c
    
    This fixes an issue with _exit not declared when building xkbcomp
    version 1.2.4 natively on Windows using MinGW32 (version 4.7.2)
    from www.mingw.org.
    
    https://bugs.freedesktop.org/show_bug.cgi?id=62365
    
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit ed582f4fccd4e23abcfba8b3b03649fea6414f44
Author: Thomas Klausner <wiz@NetBSD.org>
Date:   Sun Jun 2 22:22:36 2013 +0200

    Protect config.h like usual.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

commit c66de126ad10a7220ab74e52fb7cc81b6d8b0993
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue May 21 15:19:35 2013 +1000

    man: document -help/-?, -em1, -emp, -eml
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit a4ff0f0eee046be846cc0f46edaba48b765e50d3
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Apr 9 11:29:15 2013 +1000

    Use DEBUG, not DEBUG_ON to determine whether debugging is enabled
    
    Nothing else looks at DEBUG_ON.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 705b9bbb426410f9510601c7010da51184919b36
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Tue Jan 22 22:39:19 2013 -0800

    config: Add missing AC_CONFIG_SRCDIR
    
    Regroup AC statements under the Autoconf initialization section.
    Regroup AM statements under the Automake initialization section.
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit ed1b90216e59e6b3ac6ac5fd2faca8e056c76caa
Author: Colin Walters <walters@verbum.org>
Date:   Wed Jan 16 13:02:57 2013 -0500

    autogen.sh: Honor NOCONFIGURE=1
    
    See http://people.gnome.org/~walters/docs/build-api.txt
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit 0522fab51eec0aa3e623aa17c5b7ea3aa6dfec45
Author: Adam Jackson <ajax@redhat.com>
Date:   Wed Jan 16 13:01:21 2013 -0500

    configure: Drop AM_MAINTAINER_MODE
    
    Signed-off-by: Adam Jackson <ajax@redhat.com>

commit e4f767913338052527538d429931e8abd8c3fb88
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Fri Jan 4 18:37:45 2013 -0800

    unifdef -U__UNIXOS2__
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit bd1103ef3cfef9cfed645566f944a69e7ca568b4
Author: Ryan Pavlik <rpavlik@iastate.edu>
Date:   Wed Oct 26 12:17:44 2011 -0500

    Include Xwindows.h rather than windows.h
    
    Include the specially wrapped windows.h in Xwindows.h, rather than
    including it directly to avoid compilation errors due to clashing
    definitions
    
    Signed-off-by: Ryan Pavlik <rpavlik@iastate.edu>
    Reviewed-by: Jon TURNEY <jon.turney@dronecode.org.uk>
    Reviewed-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>

commit a68c5b7b29eb1433b1be73b50c4248e10eab8e64
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Thu Nov 1 16:37:03 2012 +1100

    Reset scan state when opening a new file
    
    When we opened a new file after encountering a parse error, we weren't
    resetting our scan state, and continued to read out of the old buffer
    rather than that of the new file.
    
    Fixes a regression introduced in 9887842e ('Use fread() instead of
    getc()').
    
    Test case:
        xkbcomp -lfhlpR "path/to/xkeyboard-config.git/symbols/*"
    The Makefiles in that tree would cause parse errors, the resulting directory
    file is incomplete (down from ~12000 to ~230 lines).
    
    Reported-by: Peter Hutterer <peter.hutterer@who-t.net>
    Signed-off-by: Daniel Stone <daniel@fooishbar.org>

commit e119cbec7e750ffc4d4bd08b577db2c697035a30
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Wed Oct 31 12:40:49 2012 +1100

    Add parsing support for multiple keysyms per level
    
    Parse, but ignore, level definitions in the form of:
    key <FOO> { [ normal_sym, { M, U, L, T, I, S, Y, M } ] };
    
    normal_sym will be carried, but the latter will be turned into NoSymbol.
    
    http://bugs.freedesktop.org/show_bug.cgi?id=25025
    
    Signed-off-by: Daniel Stone <daniel@fooishbar.org>

commit f2eff337176551f643e35c54c1e1ca0a78df69c6
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Thu Mar 22 22:02:56 2012 -0700

    xkbcomp 1.2.4
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit b4fafc4fa7edc4c04c3c0affae0d7f79a63e8d2f
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Dec 28 20:57:55 2011 -0800

    Delete redundant redeclarations of extern Atoms in symbols.c
    
    Fixes gcc warnings of:
    
    symbols.c:43: warning: redundant redeclaration of 'tok_ONE_LEVEL'
    tokens.h:99: warning: previous declaration of 'tok_ONE_LEVEL' was here
    symbols.c:44: warning: redundant redeclaration of 'tok_TWO_LEVEL'
    tokens.h:100: warning: previous declaration of 'tok_TWO_LEVEL' was here
    symbols.c:45: warning: redundant redeclaration of 'tok_KEYPAD'
    tokens.h:102: warning: previous declaration of 'tok_KEYPAD' was here
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 3835cae3cb1ad1073cbb2711f938beb878b4986c
Author: Alan Coopersmith <alan.coopersmith@oracle.com>
Date:   Wed Dec 28 20:53:45 2011 -0800

    Make sure to leave room for trailing nil byte in yyGetNumber
    
    ...though really, by the time you've added 1023 digits to the number
    you want to parse, you've got much bigger problems than an off-by-one
    error in your buffer count.
    
    Fixes parfait warnings:
       Buffer overflow (CWE 120): In array dereference of (*buf)[nInBuf] with index 'nInBuf'
          Array size is 1024 bytes, nInBuf >= 1 and nInBuf <= 1024
            at line 625 of xkbscan.c in function 'yyGetNumber'.
       Buffer overflow (CWE 120): In array dereference of (*buf)[nInBuf] with index 'nInBuf'
          Array size is 1024 bytes, nInBuf <= 1025
            at line 632 of xkbscan.c in function 'yyGetNumber'.
    
    [ This bug was found by the Parfait 0.4.2 bug checking tool.
      For more information see http://labs.oracle.com/projects/parfait/ ]
    
    Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit 1447071942dbbbfc37b08417c74c8a1d302c1626
Author: Andreas Wettstein <wettstein509@solnet.ch>
Date:   Fri Dec 2 20:10:06 2011 +0000

    include resets group compatibility modifiers #43091
    
    This change makes sure that include does not overwrite previous
    compatibility modifier settings when the included files does not
    explicitly specify them.
    
    Signed-off-by: Andreas Wettstein <wettstein509@solnet.ch>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>

commit 5fd14db5b2cc0c2c43f7a1ddefd6f9991eb1dcc2
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:18:43 2011 -0800

    Silence -Wparentheses-equality
    
    warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 6099655a4bbe1fd3e26b8afd2b91888ad2766086
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:17:38 2011 -0800

    Silence -Wshadow warnings
    
    warning: declaration shadows a local variable [-Wshadow]
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit b7e88debd937be147d3581b9c8720c0bfe354052
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:13:42 2011 -0800

    Correct our zeroing out of the row in InitRowInfo
    
    geometry.c:369:20: warning: argument to 'sizeof' in '__builtin___memset_chk' call is the same
          pointer type 'RowInfo *' (aka 'struct _RowInfo *') as the destination; expected 'RowInfo'
          (aka 'struct _RowInfo') or an explicit length [-Wsizeof-pointer-memaccess]
            bzero(row, sizeof(RowInfo *));
            ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 33b5c9413463613d341d15184b34a4f4c4e70e56
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:12:22 2011 -0800

    Fix -Wformat warnings
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit b9f6482d5fc413222334c2370ef8b9693f4b50fd
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:07:30 2011 -0800

    Add missing _X_NORETURN to uFatalError
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 8ea06128c70b07c265a217222f2a245aad4bfce4
Author: Jeremy Huddleston <jeremyhu@apple.com>
Date:   Fri Nov 11 11:06:02 2011 -0800

    Include strings.h for strcasecmp and string.h for strdup
    
    Our minimum requirement for X11 is currently Unix98.  Unix98 provides
    strcasecmp in <strings.h>.  This commit fixes implicit declarations
    of this function on systems that closely adhere to the standard.
    
    Signed-off-by: Jeremy Huddleston <jeremyhu@apple.com>

commit 89c8317ad84386f69d31f38dcddbc7ff0abffb31
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Tue Jul 5 08:23:37 2011 +1000

    Add .pc file for version export
    
    Exports xkbcomp version and the xkb base directory.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Daniel Stone <daniel@fooishbar.org>
    Reviewed-by: Gaetan Nadon <memsize@videotron.ca>

commit cdb8677f370b34a3a523a32ae0023c7e8c0c965c
Author: Peter Hutterer <peter.hutterer@who-t.net>
Date:   Wed Jun 22 10:41:33 2011 +1000

    Print version number on -version.
    
    This commit isn't playing the WARN/M/M1 macro game. Version numbers to to
    stdout, full stop.
    
    Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
    Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>

commit cef4ba4b8fdc9a5439f71437c08cf690a750bd6b
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Tue Jun 21 16:04:45 2011 +0100

    Bump to 1.2.3
    
    Signed-off-by: Daniel Stone <daniel@fooishbar.org>

commit 3caab5aa37decb7b5dc1642a0452efc3e1f5100e
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Tue Jun 21 15:55:59 2011 +0100

    Interp: Allow explicit Any/NoSymbol mappings
    
    Brown paper bag in full effect.
    
    The previous fix, while crushing the previous problem where an unknown
    keysym for an interp def would lead to every key matching it, also
    ignored explicit Any+AnyOfOrNone(All) mappings.
    
    Such as the one xkeyboard-config relied on for Control to actually
    update the modifier state.
    
    Fix this by allowing mappings explicitly declared as Any/NoSymbol, while
    ignoring only those with failed keysym lookups.  Unfortunately, due to
    the structure of the parser, it's a deeply inelegant fix.
    
    Verified with a quick check of all layouts (albeit using default
    variants only) in xkeyboard-config that this results in no changes to
    the output at all, compared to xkbcomp 1.1.1.
    
    Signed-off-by: Daniel Stone <daniel@fooishbar.org>

commit b34af8b0aec3a1dfc58f9732996274cbf2646a53
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Tue Jun 21 15:55:28 2011 +0100

    Constify LookupKeysym input argument
    
    Signed-off-by: Daniel Stone <daniel@fooishbar.org>

commit e68a8ef1067f98056237d8067052e2a0d9896133
Author: Daniel Stone <daniel@fooishbar.org>
Date:   Tue Jun 21 14:06:05 2011 +0100

    Bump to 1.2.2
    
    Emergency bugfix for the syminterp declarations.  They've been broken
    since the beginning of time, but I guess we just haven't added new
    interpreted symbols since around then.
    
