Branch :
| Author | Commit | Date | CI | Message |
|---|---|---|---|---|
| 322cd856 | 2019-11-12 20:34:31 | parser: fix merge mode only applied to first vmod in a virtual_modifiers statement Given augment virtual_modifiers NumLock,Alt,LevelThree Previously it was expanded (directly in the parser) to augment virtual_modifiers NumLock; virtual_modifiers Alt; virtual_modifiers LevelThree; Now it expands to augment virtual_modifiers NumLock; augment virtual_modifiers Alt; augment virtual_modifiers LevelThree; Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 400cc849 | 2019-11-12 20:04:13 | ast: use a separate expr struct for action list Currently it's under UnaryExpr, which just doesn't make sense. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 8c62d48c | 2019-11-12 19:16:08 | ast-build: get rid of unhelpful macro Straightforward code is better here. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 4849bc19 | 2019-11-09 22:07:15 | atom: a string is greater than its prefix Bug accidentally introduced in 9a92b46. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| c79c8033 | 2019-11-09 21:25:01 | atom: combine atom_intern() and atom_lookup() Use an "add" bool parameter instead. This simplifies the code a bit. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| adbd9c6f | 2019-11-09 13:47:16 | atom: correct iteration count in hash function Fixup of ccab349 - unlike the commit message, hash a byte twice instead of zero times, which is probably better. This is how it was before. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 9ebf97d7 | 2019-11-09 13:12:02 | atom: describe how this odd data structure works Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| ccab349c | 2019-11-09 12:43:04 | atom: use a better hash function FNV-1a instead of the djb2-like one from before. Keep the unrolling since it seems quite beneficial, even though it loses one byte if the length is odd... Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 9a92b464 | 2019-11-09 11:49:25 | atom: style changes Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 1fe1b653 | 2019-11-09 11:39:17 | atom: remove handling of garbage input Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| a5f95c2b | 2019-11-09 11:33:45 | atom: use explicit size for fingerprint Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 8ea4a001 | 2019-11-09 00:20:45 | atom: replace an avoidable strlen Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 6f8bb5ee | 2019-11-09 00:05:59 | atom: remove redundant field The field is redundant. Due to alignment, this will only save memory on 32bit architectures. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 2a615593 | 2019-11-08 22:40:13 | test/atom: increase iteration count and print random seed on failure Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 2af474e8 | 2019-11-02 13:31:44 | parser: get rid of "stealing" atoms This requires (well, at least implemented by) casting away `const` which is undefined behavior, and clang started to warn about it. The micro optimization didn't save too many allocations, anyway. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 31e561fc | 2019-11-05 13:33:11 | test: remove a superfluous string-is-null check A few lines above we check path_rel[0], so any null pointer will blow up before we get here. Found by coverity Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 96ef14ac | 2019-11-05 13:22:49 | test: fix a potential memory leak Found by coverity Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 3515ba19 | 2019-11-01 10:45:43 | test: xkeyboard-config: bring back the progress bar Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 8f93e22a | 2019-11-01 18:41:16 | test: xkeyboard-config: invoke the python3 command (#120) python3 is always python3, but python could be python2 in some cases. Or just missing (e.g. RHEL8). Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 0609073c | 2019-11-01 11:09:16 | test: xkeyboard-config: add missing variant tests Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| f4a0f738 | 2019-11-01 09:54:29 | test: xkeyboard-config: use universal_newlines instead of decode This way stdin/stdout of the process are opened in text mode and we don't need manually decode. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 7832cc72 | 2019-10-30 12:03:48 | test: xkeyboard-config: flake8 fixes Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| cd5a24aa | 2019-10-30 11:22:49 | test: xkeyboard-config: handle keyboard interrupts correctly In python multiprocessing, each process needs to handle (and ignore) the KeyboardInterrupt to avoid exception logging. This is a separate patch for easier reviewing, the first hunks merely re-indent all of the xkbcommontool/xkbcomp functions into a try/except KeyboardInterrupt block. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 9fc0cb87 | 2019-10-30 10:53:58 | test: xkeyboard-config: print to stderr on failure, stdout otherwise This is a change in behavior and requires any automated callers to adjust accordingly. Still, much easier to get the errors that way rather than it being mixed into a thousands-of-lines output file. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| f5383847 | 2019-10-29 16:06:10 | test: xkeyboard-config: add a multiprocessing.Pool() to speed up the test Collect all options into a dictionary, then process that as async actions through a process pool. This of course requires collecting the various print statements to avoid mangled output. This dropped the time to completion from around 14 min to 8 min on my local machine (unscientific single run only for the original timing). Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 1e131906 | 2019-10-30 11:15:49 | test: xkeyboard-config: use argparse for the path and the tool selection Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 90497b84 | 2019-10-31 21:21:35 | scripts/makeheader: slight simplification Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| f0c0cb80 | 2019-10-31 17:04:49 | scripts/makeheader: allow overriding the prefix path of the X11 headers with X11_HEADERS_PREFIX Signed-off-by: Sebastian Wick <sebastian@sebastianwick.net> | ||
| 255200fa | 2019-10-28 11:40:56 | test: add test for the various default include paths All tests create a temporary directory, set up the environment for that directory and then check the include paths for the presence of that directory, ideally in the right position of the list. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| e23f1061 | 2019-10-25 14:36:16 | Use XDG_CONFIG_HOME as first XKB search path Use $XDG_CONFIG_HOME/xkb as the primary lookup path for XKB rules. Same motivation as in 3a91788d9254b, however the XDG directories are more standard and recommended these days than application-specific dotfiles. The XDG spec says to fall back to $HOME/.config where XDG_CONFIG_HOME is not set so we implement that behavior as well. Fixes #112 Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 6d83838c | 2019-10-20 23:07:52 | Bump version to 0.9.1 Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 4b378398 | 2019-10-20 19:06:19 | context: Don't fail to create the context if HOME isn't available E.g. when Mutter has CAP_SYS_NICE and thus secure_getenv returns NULL. Fixes https://bugs.archlinux.org/task/64191 [ran: changed to ignore error] Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| aa118b4b | 2019-10-19 00:55:21 | README: update my email address Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| a88a0710 | 2019-10-19 00:33:09 | Bump version to 0.9.0 Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| e41dc19a | 2019-10-19 00:41:16 | build: remove leftover autotools file Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 45496c33 | 2019-10-19 00:37:48 | test: fix printf("%s", NULL) in error path ../test/common.c: In function ‘test_get_path’: ../test/common.c:171:9: warning: ‘%s’ directive argument is null [-Wformat-overflow=] 171 | fprintf(stderr, "Failed to allocate path (%d chars) for %s\n", | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 172 | (int) path_len, path); | ~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 076047b2 | 2019-10-16 10:32:19 | keymap-dump: use consistent capitalization for "Group<N>" It's used capitalized everywhere except a couple places. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| a6ed0304 | 2019-10-16 10:27:12 | keymap-dump: fix invalid names used for levels above 8 xkbcomp only accepts the "Level" prefix for a level name for levels 1 to 8, but the keymap dumping code added it always, e.g. "Level15". The plain integer, e.g. "8", "15" is always accepted, so just use that. Fixes https://github.com/xkbcommon/libxkbcommon/issues/113 Signed-off-by: Ran Benita <ran@unusedvar.com> Reported-by: progandy | ||
| 3a91788d | 2019-10-03 17:27:00 | context: move ~/.xkb to before XKB_CONFIG_ROOT in the default include path Previously, the default include path was XKB_CONFIG_ROOT:~/.xkb. The ~/.xkb include path is intended to allow the local user to customize their keymaps without having to modify system paths. But usually, the user only wants to customize specific parts. When XKB_CONFIG_ROOT is first, the user can only customize through the "entry point" (the RMLVO). When ~/.xkb is first, the user can drop in a file and it will override the system one. The impetus for this change is the rules file. "evdev" is hard-coded everywhere, so it not often not possible to change to something else. And the rules files determines how the rest of the RMLVO is interpreted. So, to enable customization, we have these options: A: System includes user. B: User includes system. C: Library goes over both in one or the other order. Option A is problematic due to backward compatibility and is also unnatural. Option B gives the user control and is backward compatible, so that's what we choose. This is also how Compose files are handled, and that seems to work fine in the wild. Option C is actually less flexible than B, and more complicated. (The rules file format doesn't have an include statement yet, but it's planned). Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| 934d5741 | 2019-09-09 17:56:42 | build: Solaris needs __EXTENSIONS__ instead of _GNU_SOURCE Fix meson build on Solaris by using __EXTENSIONS__ where Linux & other platforms use _GNU_SOURCE. Without this the build fails due to missing prototypes for functions like strdup & getopt not defined in the C99 standard. (In autoconf, this was handled by AC_USE_SYSTEM_EXTENSIONS.) Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> | ||
| 25cd67da | 2019-08-10 14:11:36 | build: remove autotools build We introduced meson 2 years ago, hopefully that's was enough time for everyone to convert. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| e7bb7045 | 2019-08-10 14:04:29 | ci: remove travis CI We moved to Azure Pipelines. Signed-off-by: Ran Benita <ran@unusedvar.com> | ||
| d486b542 | 2019-08-09 23:07:28 | Set up CI with Azure Pipelines [skip ci] | ||
| cc0e3617 | 2019-08-06 23:55:22 | CI: Install graphviz using APT This adds the missing graphviz dependency, from which Doxygen uses "dot" to create diagrams for the documentation. Also, the meson dependency is removed as it is installed using Pip, and ninja-build added. | ||
| c408adc2 | 2019-08-06 18:59:10 | CI: Publish test results from Meson | ||
| f796bbb8 | 2019-08-06 17:25:25 | CI: Try builds both with Clang and GCC | ||
| 510cbdd2 | 2019-08-06 16:55:41 | CI: Enable a macOS job | ||
| af22f9dc | 2019-08-06 16:24:43 | CI: Make Autotools and Meson builds parallel This splits the steps for installing dependencies into a new template file, as well as the build steps for Autotools. Both gets used to define two jobs which can run in parallel for both build systems. | ||
| 789bd05c | 2019-08-06 16:16:21 | CI: Always try to gather test results Resulting test logs should be always attached to the task, regardless of whether the test suite ran successfully or not. | ||
| ef74ee1b | 2019-08-06 15:34:37 | CI: Split Meson build steps into a separate YAML file | ||
| 7413c6f5 | 2019-08-06 12:25:29 | CI: Save the Meson logs as downloadable files This uploads the files saves to meson-build/meson-logs/ and associates them to the pipeline task, which makes them availabl for download in the Azure Pipelines web interface. | ||
| 69651bf7 | 2019-08-06 11:53:02 | CI: Initial set up to use Azure Pipelines For the moment, configure only basic builds with Ubuntu 16.04, more to be added as follow-ups. [skip ci] | ||
| ab4b4b7f | 2019-07-25 10:12:53 | travis: improve configuration and add macOS Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 7407d311 | 2019-07-25 13:49:41 | test/symbols-leak-test: fix sed regex on macOS Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| c68a8e29 | 2019-07-25 13:34:49 | build: fixup autotools build after 97f41fe Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 97f41fe4 | 2019-07-25 13:24:00 | test/symbols-leak-test: make it work with macOS diff The <() stuff fails with an error: diff: extra operand `/dev/fd/61' Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 6728ebca | 2019-07-25 11:59:07 | test/rmlvo-to-keymap: drop basename usage It wants some libgen.h include which is POSIX only, let's just remove it as it's hardly important. Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 32d178b5 | 2019-07-19 02:56:41 | test/rmlvo-to-keymap.c: fix compilation on Darwin (#101) program_invocation_short_name isn't portable. | ||
| 909cc04d | 2019-07-02 13:48:32 | interactive-wayland: Port to stable xdg-shell (#100) xdg_shell v6 was pretty close to the finalised stable version of xdg-shell. We can now just use the stable version, which is supported everywhere (Enlightenment, KWin, Mutter, Weston, wlroots). This requires bumping the wayland-protocols dependency. Signed-off-by: Daniel Stone <daniels@collabora.com> | ||
| 06a80bee | 2019-06-11 19:59:25 | Merge pull request #98 from michaelforney/popcount Use bitwise test for power-of-2 instead of popcount | ||
| 9d58bbd4 | 2019-06-04 14:01:02 | Use bitwise test instead of popcount to check if one bit is set We don't need to determine the total number of bits set to determine if exactly one is set. Additionally, on x86_64 without any -march=* flag, __builtin_popcount will get compiled to a function call to the compiler runtime (on gcc), or a long sequence of bit operations (on clang). Signed-off-by: Michael Forney <mforney@mforney.org> | ||
| db33ec0d | 2019-06-05 09:49:04 | Merge pull request #96 from Jjagg/patch-1 Fix a few doc typos | ||
| 8129f3b2 | 2019-06-04 23:44:42 | Fix some doc typos | ||
| db7e79e7 | 2019-03-24 14:05:17 | Merge pull request #95 from Hi-Angel/fix-lgtm-warnings Fix some of LGTM warnings | ||
| 75d1110c | 2019-03-23 23:29:29 | symbols: add a comment to suppress warning from code analyzers Signed-off-by: Konstantin Kharlamov <Hi-Angel@yandex.ru> | ||
| 83cfbf88 | 2019-03-12 19:39:23 | Merge pull request #94 from milloni/milloni/m4 Makefile.am: add include dir for AC_CONFIG_MACRO_DIR to work | ||
| 10311c95 | 2019-03-12 16:22:15 | Makefile.am: add include dir for AC_CONFIG_MACRO_DIR to work With older versions of autotools, one needs to add this line to Makefile.am. From the autoconf docs: "Note that if you use aclocal from Automake to generate aclocal.m4, you must also set ACLOCAL_AMFLAGS = -I dir in your top-level Makefile.am". [1] I couldn't build with autoconf 2.68 without this. [1] https://www.gnu.org/software/autoconf/manual/autoconf-2.65/html_node/Input.html Signed-off-by: milloni <milloni@preemptable.org> | ||
| 255047f7 | 2019-02-26 18:27:53 | README: usage questions are alright Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 6f0f9534 | 2019-02-26 18:26:01 | README: stop pointing at bugs.freedesktop.org No-one filed bugs in there for a while, let's consolidate on the more popular one. Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 9badb4e4 | 2019-02-23 05:55:00 | meson.build: use program from build machine not host or target. We can't always execute binaries from the host or target machine, as is the case in cross compilation. closes #89 | ||
| d40b368b | 2019-02-22 22:26:49 | Bump version to 0.8.4 Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 2d9e1751 | 2019-02-22 22:26:11 | Update NEWS Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| df28d5e8 | 2019-02-22 15:37:27 | build: fix meson build from tarball generated by autotools These files are used by the meson build only. Previously, trying to build with meson using the tarball generated by distcheck would fail. Fixes https://github.com/xkbcommon/libxkbcommon/issues/87. Reported-by: manesm52 Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 3f509533 | 2019-02-11 09:19:28 | meson.build: link the sources directly into libxkbcommon-x11 Similar to 75ce741ab97e3d17a0c9b06dd4bdf57c00d5538e, just for the -x11 sublibrary. This works around meson bug 3937, 'link_whole' arguments don't get added into the final static library and we end up with a virtually empty 8-byte libxkbcommon-x11.a file, see https://github.com/mesonbuild/meson/issues/3937 The internal lib is still built for the one test case that requires it. Fixes #86 Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 9f93ebcf | 2019-02-08 12:39:01 | Bump version to 0.8.3 Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 53e7a315 | 2019-02-08 12:38:02 | Update NEWS Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| caddfdb0 | 2019-02-08 12:29:27 | meson: make comment make sense now Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 327fd9a1 | 2019-02-08 12:18:00 | Merge pull request #85 from whot/wip/static-lib-sources meson.build: manually link all sources into the library | ||
| 75ce741a | 2019-02-08 12:15:48 | meson.build: manually link all sources into the library This works around meson bug 3937, 'link_whole' arguments don't get added into the final static library and we end up with a virtually empty 8-byte libxkbcommon.a file, see https://github.com/mesonbuild/meson/issues/3937 Workaround is simply to add all sources to both libraries we need them in. This obviously compiles them twice but this year's winter was cold and bit of extra warmth will be appreciated. Fixes #84 Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 73794e1e | 2019-01-28 16:29:09 | docs: fix a doxygen reference warning | ||
| 828cec7b | 2019-01-25 12:50:55 | Merge pull request #81 from whot/master keysyms: fix comment for XKB_KEY_OCARON | ||
| e08d589f | 2019-01-25 09:26:58 | keysyms: fix comment for XKB_KEY_OCARON Reported-by: Keve Müller Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| bc04af63 | 2019-01-22 09:48:23 | Merge pull request #79 from jwrdegoede/add-2-new-keysyms Sync Keysyms with recent xproto additions | ||
| 9b85d96d | 2019-01-22 08:31:43 | Sync Keysyms with recent xproto additions xproto recently has been extended with 2 new keysyms: XF86XK_MonBrightnessCycle XF86XK_RotationLockToggle This commit is the result of running "scripts/update-keysyms" on a system with the updated xproto installed. Signed-off-by: Hans de Goede <hdegoede@redhat.com> | ||
| 293c704c | 2018-10-01 10:24:45 | Merge pull request #69 from alanc/master Fix off-by-one error in index check in xkb_file_type_to_string | ||
| 31f1f355 | 2018-09-30 16:04:29 | Fix off-by-one error in index check in xkb_file_type_to_string Found by Oracle's Parfait 2.2 static analyzer: Error: Buffer overrun Read outside array bounds [read-outside-array-bounds] (CWE 125): In array dereference of xkb_file_type_strings[type] with index type Array size is 56 bytes, index <= 56 at line 734 of src/xkbcomp/ast-build.c in function 'xkb_file_type_to_string'. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com> | ||
| c9a499c9 | 2018-08-24 09:14:14 | darray: fix unprotected macro argument Reported-by: @msmeissn Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| 87b0765f | 2018-08-21 09:05:09 | Merge pull request #68 from whot/wip/xkeyboard-config-tester xkeyboard-config combination tester | ||
| 878bc085 | 2018-08-20 16:46:19 | test: allow for absolute paths to be resolved This makes it possible to check a keymap sitting elsewhere than in the test directory. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| 16c84cdd | 2018-08-20 15:50:35 | test: drop the rmlvo ability from print-compiled-keymap This is now handled by the rmlvo-to-keymap tool Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| d1cb8ad4 | 2018-08-14 11:16:30 | test: add a tool to test-compile all LVO combinations from xkeyboard-config This test contains of two parts: - a simple program to convert RMLVO commandline arguments into a keymap (and print that keymap if requested). - a python script that runs through rules/evdev.xml, and tries to compile a keymap for sort-of every layout/variant/option combination. Sort-of, because we can have multiple options and it really only does one per layout(variant) combination. Same thing can be done using xkbcomp, but right now it doesn't take that as argument, it's hard-coded. This takes quite a while, installing python-tqdm is recommended to see fancy progress bars instead of just miles of dumps. Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net> | ||
| d7891d09 | 2018-08-18 15:12:15 | build: turn off strict aliasing The benchmarks don't show any effect, so turn it off to have one less thing to worry about. The parser does a lot of casting between AST nodes. Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| a9ace75f | 2018-08-18 14:28:15 | x11: fix undefined behavior when copying the coordinates of ptr movements actions Left shift of a negative integer. For some reason the protocol representation here got really botched (in the spec it is just a nice and simple INT16). Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| f8134c85 | 2018-08-05 08:51:30 | Bump version to 0.8.2 Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| e3f9c1b4 | 2018-08-05 08:49:36 | Update NEWS Signed-off-by: Ran Benita <ran234@gmail.com> | ||
| a3e110d6 | 2018-08-05 08:45:19 | Merge pull request #67 from fooishbar/fix-more-fuzz Fix more fuzz-testing fallout | ||
| bb4909d2 | 2017-10-30 11:21:55 | Fail expression lookup on invalid atoms If we fail atom lookup, then we should not claim that we successfully looked up the expression. Signed-off-by: Daniel Stone <daniels@collabora.com> | ||
| 5440aaa5 | 2017-06-26 21:52:27 | Fix signed vs. unsigned confusion in name sanitisation Don't try to divide through a signed char when indexing an array, lest ye try to index off the start of it. Signed-off-by: Daniel Stone <daniels@collabora.com> | ||
| 4fcbc470 | 2017-06-26 21:49:49 | darray: Don't call memcpy() on NULL The only time we could ever hit this was with count == 0, which seems unnecessarily pedantic. But OK. Signed-off-by: Daniel Stone <daniels@collabora.com> | ||
| ae7856db | 2017-06-26 21:38:52 | text: NULL-terminate SI mask names The list should have a NULL sentry. Add one. testcase: 'interpret KP_Delete+AnyOfOrNaneo(ll)' Signed-off-by: Daniel Stone <daniels@collabora.com> |