summaryrefslogtreecommitdiff
path: root/src/xkb/XKB.c
AgeCommit message (Collapse)AuthorFilesLines
2024-02-20unifdef __sgiAlan Coopersmith1-4/+0
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2022-04-23XkbOpenDisplay: Add _Xconst qualifier to display name argumentAlan Coopersmith1-1/+1
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2020-04-21reduce gcc-normal warnings using casts (no object change)Thomas E. Dickey1-1/+1
Signed-off-by: Thomas E. Dickey <dickey@invisible-island.net>
2014-07-18Fix validation of ctrls parameter to XkbGetPerClientControls()Alan Coopersmith1-7/+4
Nothing in the XKB spec states that the memory pointed to by ctrls has to be initialized to any given value when passed to the function, only that it is set by the function to the values returned by the X server: http://www.x.org/releases/X11R7.7/doc/libX11/XKB/xkblib.html#The_Miscellaneous_Per_client_Controls The check for the incoming value seems to be copied from XkbSetPerClientControls without explanation. Instead change it to checking if ctrls is non-NULL, since there's no point asking the X server to return a value the caller won't even see. Found while investigating report from cppcheck-1.65: [src/xkb/XKB.c:699] -> [src/xkb/XKB.c:719]: (warning) Possible null pointer dereference: ctrls - otherwise it is redundant to check it against null. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-12-26XkbSelectEventDetails: remove unnecessary assignmentsAlan Coopersmith1-3/+0
clear & selectAll are set to 0 already a few lines earlier, affectWhich is set to XkbMapNotifyMask a few lines later. None are used between the other assignments and the removed ones. Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2013-08-20Reindent XKB code to X.Org standard styleAlan Coopersmith1-376/+390
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2011-01-31Variable "entry" tracked as NULL was dereferenced.Erkki Seppälä1-1/+1
Check entry for non-nullness before dereferencing it Reviewed-by: Dirk Wallenstein <halsmit@t-online.de> Reviewed-by: Ander Conselvan de Oliveira <ander.conselvan-de-oliveira@nokia.com> Signed-off-by: Erkki Seppälä <erkki.seppala@vincit.fi>
2010-07-07Purge macros NEED_EVENTS and NEED_REPLIESFernando Carrijo1-2/+0
Signed-off-by: Fernando Carrijo <fcarrijo@yahoo.com.br> Acked-by: Tiago Vignatti <tiago.vignatti@nokia.com> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
2010-01-14Purge CVS/RCS id tagsAlan Coopersmith1-2/+0
Signed-off-by: Alan Coopersmith <alan.coopersmith@sun.com>
2008-06-17Strip whitespace from end of lines in source filesAlan Coopersmith1-12/+12
2005-05-13- For now put xtrans in X11/Xtrans/X11, since libX11 is looking for it in ↵XORG-6_8_99_7Søren Sandmann Pedersen1-0/+3
<X11/...> - For Xcomposite and Xdamage, don't link the build system out of the xc tree - Link the public X11 headers into their own directory - Add links to XKeysymDB and XErrorDB - Add links to all the Xlib man pages - Add links to the lcUniConv subdirectory - Conditionally include config.h in Xlib source
2004-04-23Merging XORG-CURRENT into trunkEgbert Eich1-171/+13
2003-11-14R6.6 is the Xorg base-lineXORG-MAINKaleb Keithley1-0/+926