Get rid of unwanted whitespace before punctuation by moving them to the
lines with the tags, instead of before & after.
perl -i -0 -p -e 's{\>\s*\n([\.,;:])}{>\1\n}g' *xml
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Get rid of unwanted whitespace inside parens by moving them to the
lines with the tags, instead of before & after.
perl -i -0 -p \
-e 's{(?<!--) \(\s*\n\<}{\n(<}g;' \
-e 's{\>\s*\n\)([\.,;]?)(?! [^\n]*--)}{>)\1\n}g' *xml
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Also add <indexterm> entries for most of them, to make their definitions
or introductions easy to find from the index.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Handles typos that caused the scripts to miss matches, functions like
malloc & free from other libraries, function name patterns, etc.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Most were caught by applying libX11 lists, but BadKeyboard & XKB*_Bad*
are XKB-specific. (Plus some were badly split across tag boundaries.)
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
All the places marked by these have been turned into the appropriate
link, xref, or olink tags now.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
- Stop placing <emphasis> on empty space, commas, and periods.
- Move periods & commas after closing </emphasis> tag
- move <emphasis> open & close tags to same line, instead of mirroring
nroff layout.
Simplifies automating further transformations of these tags.
Performed via:
perl -i -0 -p \
-e 's{<emphasis>(\s*)</emphasis>}{}msg;' \
-e 's{<emphasis>([\s\.,]*)</emphasis>\s*}{\1}msg;' \
-e 's{\n([\.,])\s*}{\1\n}msg;' \
-e 's{([^\.])([\.,])\s*</emphasis>}{\1</emphasis>\2}msg;' \
-e 's{\s*<emphasis>\n\s*}{\n<emphasis>}msg;' *xml
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
- Merge some functionargdecl entries incorrectly split across rows
- Add missing parameter name markup to some functionargdecls
- Add missing function prototype markup to a functiondecl
- Remove stray emphasis tags in a functiondecl
Allows them to correctly convert to funcsynopsis markup in next step.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Xutil.h has always had a value of 3 for IconicState, since 2 was
previously used for the long-obsolete ZoomState, so make the spec
match what programs have used for decades.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matthieu Herrb <matthieu@herrb.eu>
Forces compiler to zero-fill unset fields in the struct (fixing bug 81236)
and allows optimizer to order field initialization to best fit cache layout
or other considerations.
Before & after output of gcc -S on AMD64 shows insertion of "rep stosq"
instructions to rapidly zero-fill structs.
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matthieu Herrb <matthieu@herrb.eu>
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>
Check is intended to ensure we allocate at least XkbNumRequiredTypes
in map, but was accidentally marked with a ! causing the wrong check.
Reported-by: Harms <wharms@bfs,de>
Reviewed-by: Matthieu Herrb <matthieu@herrb.eu>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
"unsigned int width, unsigned int height", not a single parameter "height"
of type "unsignedintwidth,".
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
* Do not use variables before checked for NULL.
* remove some superfluid spaces (Mark Kettenis)
Signed-off-by: Harms <wharms@bfs,de>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
