Send locking assertion backtraces to stderr.  Improve the heading on the backtrace.

Print backtraces in case an assert fails inside xlib/xcb.

As you know there are some nasty libs / apps doing locking
incorrectly. In order to improve the information given to the user
when he encounters such a situation (people don't run apps in gdb
normally) I created the patch attached.
It's very non-intrusive (and affects only xlib/xcb, Josh told me on
irc that it could be useful for other areas too, personally I don't
think that it's really needed at other places ...).

Some same outputs and the discussion of them:

    lxuser@pdln:/tmp$ ./main
    Got a backtrace:
    #0 /tmp/usr/lib/libxcb-xlib.so.0 [0xb7f9d728]
    #1 /tmp/usr/lib/libxcb-xlib.so.0(xcb_xlib_unlock+0x31) [0xb7f9d861]
    #2 ./test.so(function_a+0x11) [0xb7f9f3fd]
    #3 ./test.so(function_b+0x11) [0xb7f9f410]
    #4 ./main [0x80484a7]
    #5 /lib/libc.so.6(__libc_start_main+0xdc) [0xb7e60ebc]
    #6 ./main [0x80483f1]
    main: xcb_xlib.c:82: xcb_xlib_unlock: Assertion `c->xlib.lock' failed.
    Aborted

That's kinda the normal situation.

    lxuser@pdln:/tmp$ ./main
    Got a backtrace:
    #0 /tmp/usr/lib/libxcb-xlib.so.0 [0xb7f90728]
    #1 /tmp/usr/lib/libxcb-xlib.so.0(xcb_xlib_unlock+0x31) [0xb7f90861]
    #2 /tmp/test.so [0xb7f923cd]
    #3 /tmp/test.so(function_b+0x11) [0xb7f923e0]
    #4 ./main [0x80484ab]
    #5 /lib/libc.so.6(__libc_start_main+0xdc) [0xb7e53ebc]
    #6 ./main [0x80483f1]
    main: xcb_xlib.c:82: xcb_xlib_unlock: Assertion `c->xlib.lock' failed.
    Aborted

There are two possible reasons that the name doesn't appear in #2:
a) a hidden symbol or a symbol with statical linkage in a library
b) a symbol in an app not compiled with -rdynamic.
But in both cases you still know _where_ the caller is.

Note that in this example test.so was compiled with
-fomit-frame-pointer; this isn't an issue as _one_ (= the caller)
stack trace is still valid (as long as you don't have the insane idea
to compile xcb with -fo-f-p).

Another issue that may appear is "tail call elimination" (some entries
are mysteriously missing; this is quite ugly, but you still get enough
information so that you can do something useful with the issue e.g. by
disassembling the relevant parts with gdb).

Signed-off-by: Jamey Sharp <jamey@minilop.net>

xcb_poll_for_event: Return already-read events before read(2)ing again.

Merge branch 'master' of git://anongit.freedesktop.org/git/xcb/libxcb

Generate error constants as XCB_BAD_*, similar to Xlib.

The previous constants remain for compatibility, but should be deprecated.

Merge branch 'master' of git+ssh://git.freedesktop.org/git/xcb/libxcb

remove the 7th bit of the response_type for the event loops

Update autogen.sh to one that does objdir != srcdir

Modify new attribute from previous patch so that it is necessary only on
extensions with split names.  Tested with diff and found no difference with
previous stylesheet header-file output.

Replaces special-casing in c-client stylesheet with support
for new attribute.  Tested with diff and found no difference with
previous stylesheet header-file output.

add the first step toward the documentation of the request/reply functions. The arguments of the requests are not found yet. Josh, can you look at it ?

and make the html code valid...

font part

no more xid or id fields

add the complete cursor example. Make the html code valid

add doxygen doc for the *_end functions

fix all the occurence where a drawable where considered as a union

Add autogen.sh to EXTRA_DIST.

Add tools/* to EXTRA_DIST.

add doc tag for the _next functions. It creates doxygen doc in the header files for these functions

Merge branch 'master' of git+ssh://git.freedesktop.org/git/xcb/libxcb

add some output informations to configure script

If enable_build_docs is "no", we don't even try to look for doxygen.

Also set BUILD_DOCS in that branch. Now the disable-build-docs works
as expected.

Fixed evaluation of the disable-build-docs argument.

Now at least enable_build_docs is set correctly.

Fix unit tests for FreeBSD

putenv() string must contain '='
environment failure test is invalid if argument is NULL

Use substitition variables in xcb-xinerama.pc.in, not instances of their values

xcb-xinerama.pc.in looked more like a generated .pc file; replace specific
instances of values provided by an invocation of configure with the general
substitution variables configure replaces.

Bug #9119: test xcb_popcount

libXau didn't have a correct pkg-config file until 0.99.2: fail if an older version is found.

NetBSD doesn't have AI_ADDRCONFIG: use it only if it's available.

replace all the _new functions with xcb_generate_id. repalce X11/XCB with xcb. Fix a description of the default background of a window

Release libxcb 1.0

Avoid race condition when using multiple make jobs

Avoid race condition when symlinking XML files.
When declaring a rule with many files as target, the rule is called
when any of them is requested, resulting in multiple for loops happening
during a make process using more than one job.
Also, use '$(LN_S) -f' rather than removing and recreating a file,
that one should be as supported as 'rm -f' and requires one less command.

Rewrite automake's data installation rules, because they suck.

Specifically, they didn't handle installing data from both srcdir and builddir.
We have the tutorial in the srcdir, and build the manual in the builddir.

Also, stop rebuilding the manual for each make target in the doc directory, and
every time any of those targets get called.  This change now makes the manual
never rebuild once built; we plan to fix that later, by rewriting the makefiles
to avoid recursive make, and then making the manual depend on the source files.

Commit by Jamey Sharp and Josh Triplett.

Rework doxygen build and install to work with srcdir != builddir

The documentation generation with doxygen now works when built out of tree,
with srcdir != builddir.  xcb.doxygen now gets generated from xcb.doxygen.in,
so that it can use top_builddir and top_srcdir to find source and to output
documentation.  Also fill in PROJECT_NUMBER from @VERSION@, now that we have
it readily available via autoconf.

Remove --with-opt and --with-debug options from configure.ac; use CFLAGS instead

configure supports using custom CFLAGS, so remove the --with-opt and
--with-debug options from configure.ac, and the corresponding usage of
COPTFLAGS and CDEBUGFLAGS in src/Makefile.am.

Check for getaddrinfo rather than gethostbyname in configure.ac

Since the addition of IPv6 support, we now use getaddrinfo rather than
gethostbyname; update configure.ac accordingly.

xcb-proto has no libraries or headers, so don't use XCBPROTO_CFLAGS or XCBPROTO_LIBS.

Use pthread-stubs as needed, and list xdmcp in Requires.private when XCB is built to use it.

Replace uses of "long" with uint32_t, and similar; fixes 64-bit bugs

Fix IP6 work for FreeBSD/Mac.

Support XDM-AUTHORIZATION-1 on IPv6

Commit by Jamey Sharp and Josh Triplett.

Refactor auth code to get display number from xcb_connect

Change xcb_connect to pass the display number to _xcb_get_auth_info, which
passes it to get_authptr.  This allows get_authptr to stop hacking the display
number out of the sockaddrs of various address families, such as
port - X_TCP_PORT, or the number after the last X in the UNIX socket path. This
also removes a portability bug introduced during the IPv6 changes: relying on
'\0'-termination of the UNIX socket path in a sockaddr_un.

Commit by Jamey Sharp and Josh Triplett.

Only use AI_NUMERICSERV if defined.

Support authentication for IPv6 connections

Support AF_INET6 in get_authptr, and refactor to use common code for IPv4 and
v4-mapped IPv6 addresses.

Commit by Jamey Sharp and Josh Triplett.

Support displays with IPv6 addresses or hosts which resolve to IPv6 addresses

xcb_parse_display already correctly handled IPv6 displays.  Now, _xcb_open_tcp
uses getaddrinfo, and correctly connects to IPv6 displays.  Displays can use
bare IPv6 addresses, square-bracketed IPv6 addresses, or hostnames which
resolve to IPv6 addresses.

Since IPv6 addresses may include colons, including at the end, revise the
DECnet display parsing code to avoid triggering on IPv6 addresses.

Authorization may not work with IPv6 connections yet.

This commit brought to you by the (display) number ::1:1.1, the letter X,
the Gobby collaborative editor, Josh Triplett, and Jamey Sharp.

Add new xcb_parse_display test cases, most related to IPv6 addresses

Bug #5958: Also zero out the implicit pad byte in empty requests.

Remove support for the <localfield> tag: nothing needs it.

Only one use of <localfield> remained, for a list length expression in
xv.xml. List length parameters that don't actually appear in the
protocol should be left implicit: if no length expression is given, then
a localfield will be automatically created by c-client.xsl.

Bug #5958: zero out padding bytes in requests.

htmldir was introduced in autoconf 2.59c: set a reasonable value for earlier versions.

Provide xcb_prefetch_maximum_request_length counterpart to xcb_get_maximum_request_length.

Fix Doxygen warnings.

Specifically, fixes these two warnings which were emitted for every
generated source file:

* Warning: end of file while inside a group
* Warning: group XCB_BigRequests_API already documented. Skipping documentation.

Make doc installation work and simplify doc/Makefile.am.

Don't override the default htmldir with an unquoted copy.

Don't suppress `make` echoing on Doxygen commands.

Ensure the tutorial is always installed even if Doxygen isn't
available.

Take better advantage of the automake installation infrastructure.

Turn off Doxygen warnings for undocumented API, for now.

But warn about absent parameter documentation for functions that are
otherwise documented.

Switch from the old AM_PATH_CHECK macro to pkg-config.

check 0.9.4 is now required to build XCB's unit tests.

The version that we were requiring was not actually new enough to let
our unit tests compile, and the AM_PATH_CHECK macro is now considered
deprecated. We know that versions of check using pkg-config are new
enough to work, and the check dependency was optional anyway, so we've
dropped support for older versions.

add Makefile.am for building/installing doxygen doc

add xcb.doxygen to EXTRA_DIST

Merge branch 'master' of git+ssh://git.freedesktop.org/git/xcb/libxcb

add AM_CHECK_DOXYGEN macro

Release libxcb 1.0 RC3 (0.9.93).

Added initial doxygen generation stuff.  This should probably be cleaned up later.

Added support for major/minor-version attributes in the xcb tag.  This is just to give some reference and help people when using *query_version.

fix doxy doc

Add doxygen config file.

Thanks to Jeremy Kolb <jkolb@brandeis.edu> for the initial version.

Quit treating xproto specially: handle it like all the extensions.

Ignore generated xinerama files.

Also test xcb_parse_display with NULL argument and display in $DISPLAY

Match only XCB-namespaced XID generators when converting to xcb_generate_id.

Thanks to Vincent Torri for pointing out that I screwed up. :-)

Define and use constants for opcode numbers.

Hard coding the opcode numbers in the function just makes it harder to figure
out what's going on, but much more to the point, not defining the opcodes in
the header makes it impossible to use the generated headers instead of the
x11proto headers in the server.

The name I settled on is very simple, for an extension by the name of xconf,
and a request by the name of list_devices, we get XCB_XCONF_LIST_DEVICES.  If
this somehow causes problems, we can probably add a _OP somewhere in there,
but.

Acked-by: Jamey Sharp <jamey@minilop.net>
Closes: #8641

Add library support for xcb-xinerama.

This is version 1.1 of the Xinerama (PanoramiX) extension.

Add note to xcbxlib.h that nothing except Xlib/XCB should use it.

Release libxcb 1.0 RC2 (0.9.92).

Bugfix: make Plan 7 'checked' requests work correctly.

The initial implementation of Plan 7 dumped all X errors into the event
queue, because the record of a pending reply was pruned too early if an
error occurred in place of the expected reply.

Document xcb_generate_id.

Move xcb_generate_id from xcbext.h to xcb.h

Since extensions no longer provide type-specific XID-generation functions,
xcb_generate_id now forms part of the xcb client API, rather than the
extension API; move it from xcbext.h to xcb.h accordingly.

Stop implicitly importing xproto; goes with proto change to explicitly import it

Declare "struct foo" or "enum foo" as well as the typedef'd name "foo".

Handle "xidunion" instead of "union" for XID unions like DRAWABLE and FONTABLE

api_conv.pl: replace xcb_*_new with calls directly to xcb_generate_id, now that we have no type-safety XID wrappers.

Refer to the "xlib lock" rather than the "IOLock" in xcbxlib.h

Remove XID wrapper structures and replace them with uint32_t typedefs

After positive feedback from several people, we have decided to remove the XID
wrapper structures that attempted to provide C type safety, and replace them
with uint32_t typedefs.  Feedback has indicated that these type-safety hacks
generated more trouble than help.

We will bump the libxcb soname at the next release.

Remove xcb_get_io_lock from the Xlib-specific API: it is no longer used.

Remove the 'int *error' out-parameter for xcb_poll_for_event.

Add xcb_xlib_lock and xcb_xlib_unlock, a special-purpose two-level recursive lock just for libX11.

Factor out pthread_mutex_lock and unlock calls for the iolock.

fix some indentation, remove args of the main function (not needed), xcb_sync -> xcb_aux_sync. Jamey, it seems that your script transforms XCBGetSetup into a type, instead of a function. Same for XCBSetupVendor. Maybe there are other fuctions that have the same problem.

update tutorial to the new api. Add some doc about cursors. Add some colors to section 6 (Opening and closing the connection to an X server). Tell me if you want to keep colors or not (colors can be modified in xcb.css)

Release libxcb 1.0 RC1 (0.9.91).

Stop installing the protocol descriptions for extensions to an extensions/
subdirectory

Actually install xcb-xlib.pc, and ship xcb-xlib.pc.in in distribution tarballs

Ensure api_conv.pl is idempotent on names of constants: XCB_[A-Z0-9_]+.

Move header files to $includedir/xcb (generally $prefix/include/xcb).

Split the Xlib compatibility functions into a separate library libxcb-xlib

We don't want to have to change the libxcb soname if we later manage to remove
the Xlib compatibility functions, and nothing except Xlib should ever use
them, so split them into a separate library.

Move debian/ directory to new branch "debian".

We no longer need xproto: do not list it as a dependency.

Lowercase library names to correspond with the new API.

Add XCBAllPlanes from xcb-image util library.

Integrate top-level .gitignore into .gitignore for each subdirectory

In preparation for the repository split, move the relevant contents of the
top-level .gitignore into the .gitignore for each immediate subdirectory.

now, I know how to use api_conv.pl :)

Remove the xcb_[extension]_init functions; use xcb_get_extension_data directly

More fixups for incorrect API conversions by api_conv.pl