Track 64-bit sequence numbers internally.

External APIs that used 32-bit sequence numbers continue to do so.

Commit by Josh Triplett and Jamey Sharp.

Use sequence number ranges in pending replies

This allows optimizing adjacent pending replies with the same flags, and
will help support default flags for a range of future requests.

Commit by Josh Triplett and Jamey Sharp.

Inline _xcb_lock_io, _xcb_unlock_io, and _xcb_wait_io.

These functions are once again a single pthread call, so just make that
call directly.

Remove libxcb-xlib and xcbxlib.h.

build: fix configure.ac AC_DEFINE

Rather use AC_DEFINE_UNQUOTED and only once.

Signed-off-by: Julien Danjou <julien@danjou.info>

allow compile-time setting for XCB queue buffer size

Signed-off-by: Julien Danjou <julien@danjou.info>

Remove duplicate XCB_EXTENSION calls for Composite extension

fix tiny memory leak in read_packet (leak only happens when malloc returns NULL so it's very rare)

Signed-off-by: Julien Danjou <julien@danjou.info>

Added generation of extern "C" for compatibility with C++

The auto-generated header files now include an extern "C"
declaration for compatibility with C++.

Signed-off-by: Julien Danjou <julien@danjou.info>

Add support for the abstract socket namespace under Linux

Based on same in Xtrans.

Signed-off-by: Julien Danjou <julien@danjou.info>

Fix some fd leaks in _xcb_open_*()

Signed-off-by: Julien Danjou <julien@danjou.info>

fixed overly aggressive warning about fixed field following variable

added small fix to support trailing fixed fields; also warning for non-pad fixed fields

factorize m4 macros and add one to set X extensions

Signed-off-by: Julien Danjou <julien@danjou.info>

Initialize all fields of addrinfo

Signed-off-by: Julien Danjou <julien@danjou.info>

Use ifdef instead of if for defined value

Signed-off-by: Julien Danjou <julien@danjou.info>

Set namelen unsigned

Signed-off-by: Julien Danjou <julien@danjou.info>

Rename index to idx to avoid shadowing

Signed-off-by: Julien Danjou <julien@danjou.info>

Use a signed size in read_block()

Signed-off-by: Julien Danjou <julien@danjou.info>

Use unsigned to compare and rename sync

- i must be unsigned to be compare in the loop
- sync shadow global sync() function

Signed-off-by: Julien Danjou <julien@danjou.info>

Fix htonl() arg & convert sizeof() to signed

Signed-off-by: Julien Danjou <julien@danjou.info>

initialize global_id to 0

Signed-off-by: Julien Danjou <julien@danjou.info>

Bump to 1.1.91.

Make EXTHEADERS, EXTSOURCES, EXTENSION_XML unconditional of configure flags.

yay, make distcheck works now even when some extensions are disabled.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>

fix FreeBSD support

The GNU/kFreeBSD (and BSDs in general) have a different
layout of struct sockaddr, sockaddr_in, sockaddr_un ...

The first member do not have to be "sa_family",
they also have "sa_len" field.

Signed-off-by: Julien Danjou <julien@danjou.info>

Bump to 1.1.90.

Fix variable declaration formatting

Add xcb_ge_event_t and handling for long events.

GenericEvent can be more than 32 bytes long. Ensure that the required data is
pulled off the wire and tack it onto the event.

Due to the structure of the xcb_generic_event_t, the data is appended AFTER
the full_sequence field.

Fix libxcb/src compile with srcdir != builddir.

Replace a stray c-client.xsl in the libxcb SOURCES.  Fixes make distcheck.

Use the python install path from xcb-xproto.pc to locate the xcbgen package.

Add mention of PYTHONPATH if xcbgen cannot be found.

Add Python parser C language-dependent part.

Add SELinux extension support, disabled by default.

Add XInput extension support, disabled by default.

Inputs to AC_CONFIG_FILES are automatically distributed.
Hence, it is not necessary to explicitly add them to EXTRA_DIST.

Generated the configure.ac and Makefile.am's in libxcb with the idea of
making each extension library individually selectable for build.

Signed-off-by: Eamon Walsh <ewalsh@tycho.nsa.gov>

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

make IPv6 optional

Remove file that wasn't meant to be committed.

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

Add comment noting the requirement to free replies when finished.

Add generated comment noting requirement to free replies after use.

Release libxcb 1.1

Revert "Generate error constants as XCB_BAD_*, similar to Xlib."

Since several extensions named their errors like "BadFoo", this patch
results in names like XCB_EXT_BAD_BAD_FOO, which is really awful. Those
extensions are already kind of awful, as they produce structure names
like xcb_ext_bad_foo_error_t, which is redundant.

A patch that removes "Bad" from the XML extension descriptions, while
maintaining API and ABI compatibility in XCB, is needed before this
patch can be released.

This reverts commit 158c9b6ba18b39f424bd524fceb66f3fec0d1616.

Don't hold the xlib-xcb lock while sleeping: that allows deadlock.

With this patch, `ico -threads 2` runs without deadlock.

Many thanks to Christoph Pfister <christophpfister@gmail.com> for
pointing out the problem, providing detailed analyses, explaining it to
me repeatedly until I understood what was going on, and proposing and
reviewing possible solutions.

Signed-off-by: Jamey Sharp <jamey@minilop.net>
Acked-by: Christoph Pfister <christophpfister@gmail.com>

Factor pthread_cond_wait(iolock) to _xcb_wait_io.

This parallels the _xcb_lock_io and _xcb_unlock_io factoring.

Don't abort() on locking assertions if LIBXCB_ALLOW_SLOPPY_LOCK is set.

But do still print a full backtrace, on platforms where that's
supported.

This commit follows the spirit of Novell's libxcb-sloppy-lock.diff.

I strongly opposed proposals like this one for a long time. Originally I
had a very good reason: libX11, when compiled to use XCB, would crash
soon after a locking correctness violation, so it was better to have an
informative assert failure than a mystifying crash soon after.

It took some time for me to realize that I'd changed the libX11
implementation (for unrelated reasons) so that it could survive most
invalid locking situations, as long as it wasn't actually being used
from multiple threads concurrently.

The other thing that has changed is that most of the code with incorrect
locking has now been fixed. The value of the assert is accordingly
lower.

However, remaining broken callers do need to be fixed. That's why libXCB
will still noisily print a stacktrace (if possible) on each assertion
failure, even when assert isn't actually invoked to abort() the program;
and that's why aborting is still default. This environment variable is
provided only for use as a temporary workaround for broken applications.

Signed-off-by: Jamey Sharp <jamey@minilop.net>
Acked-by: Josh Triplett <josh@freedesktop.org>

Allow unix:<screen> style display names again.

https://bugzilla.novell.com/show_bug.cgi?id=289007
This notion is used in a lot of scripts.

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.