Always wake up readers after writing.

Since writers must make sure they read as well, threads may have gone to
sleep waiting for the opportunity to read. The writer must wake up one
of those readers or the application can hang.

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

Fix strict-aliasing warning when getting generic event length.

xcb_ge_event_t has its length field in the same place that
xcb_generic_reply_t does, so there's no need to cast the generic reply.

Signed-off-by: Jamey Sharp <jamey@minilop.net>
Cc: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Julien Danjou <julien@danjou.info>

Delete a useless level of indirection from _xcb_out_send's parameters.

_xcb_out_send needs _xcb_conn_wait to store back its progress so it can
be reinvoked to pick up where it left off---but then _xcb_out_send
guarantees that it leaves either an empty output vector or a shut-down
connection, so *its* callers never care how much progress was made.

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

Release libxcb 1.6

Fail if fd is too big and poll() is not available

Depending on the process file limit, a file descriptor can be larger
than the capacity of fd_set. There is no portable way to create a
large enough fd_set at run-time. So we just fail if the file descriptor
number is too high and poll() is not available.

Reviewed-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Julien Danjou <julien@danjou.info>

Fix authentication on hpux and Hurd

libxcb's 010e5661 (Fix XDM-AUTHORIZATION-1 (bug #14202)) mistakenly
inverted a few lines of code, making local socket authentication fail on
hpux and Hurd: when getpeername fails, sockname needs to be initialized
by getsockname before its address family can be checked.

Signed-off-by: Samuel Thibault <samuel.thibault@ens-lyon.org>
Signed-off-by: Julien Danjou <julien@danjou.info>

Open the X11 socket with close-on-exec flag

This saves the X11 connection from leaking into children processes.
On Linux, this is fully thread-safe using SOCK_CLOEXEC. On other
systems, there is a small race condition.

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

Support xcb_discard_reply

This function is useful for dynamic language garbage collectors. Frequently
a GC cycle may run before you want to block wainting for a reply.

This function is also marginally useful for libxcb apps that issue
speculative requests (eg. xlsclients).

Reviewed-by: Jamey Sharp <jamey@minilop.net>
Tested-by: Eamon Walsh <efw@eamonwalsh.com>
Signed-off-by: Peter Harris <pharris@opentext.com>

xcb_connect_to_fd: fix descriptor leak on memory error path

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

configure.ac: Fix a typo on the last commit.

Signed-off-by: Jeremy Huddleston <jeremyhu@freedesktop.org>

darwin: xnu doesn't support poll on ttys on the master side.

<rdar://problem/7360546>

Signed-off-by: Jeremy Huddleston <jeremyhu@freedesktop.org>

Tutorial code fix

Accessed elements of names[] after freeing them in the first example.

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

DRI2 extension requires xcb-proto 1.6 or newer

Signed-off-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Julien Danjou <julien@danjou.info>

build: simplify extension building

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

Release libxcb 1.5

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

setsockopt(SO_KEEPALIVE) on TCP display connections.

This matches xtrans behaviour in SocketINETConnect, and makes it so apps
don't hang forever if their display dies.

Signed-off-by: Adam Jackson <ajax@redhat.com>

Add DRI2 support. (v2)

v2: Build fix from jcristau.

Fix typo in the tutorial.

Don't build docs if 'dot' is not found Signed-off-by: Paulo R. Zanoni <pzanoni@mandriva.com>

Fix check dependency

Bugzilla #21992

make -j check fails because the check-local rule gets executed before
the tests actually ran, so CheckLog*.xml doesn't exist.

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

Cygwin build fix: Add -no-undefined to libtool flags

-no-undefined is needed to tell libtool a shared library can be built
on platforms which require all references to be statisfied at link time.

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Julien Danjou <julien@danjou.info>

Release libxcb 1.4

Add majorCode, minorCode and resourceID fields to X generic error

Fix precedence bug: wrong length for big-requests preceded by sync.

Also replace excessively clever use of bitwise OR with equivalent
addition.

Reported-by: Geoffrey Li <geoffrey@seitopos.com>
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Josh Triplett <josh@joshtriplett.org>

Fix libxcb-randr version info

The SONAME shouldn't have been bumped in 1.3, only new symbols were
added.

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

Release libxcb 1.3

Disable Nagle on TCP socket

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

Store xcbproto version libxcb was compiled with

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

depends on recent xcb-proto and bump version of randr

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

Fix XDM-AUTHORIZATION-1 (bug #14202)

With this patch, we know use correctly the socket address or peer
address for authentication purpose.

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

util: remove useless strlen calls from decnet opening

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

util: merge common code for xcb_connect

Many code was duplicated between xcb_connect_to_display_with_auth_info
and xcb_connect(). We merge both, since the difference is just about the
xcb_auth_info_t pointer being supplied, or not.

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

util: open_abstract gets filelen as parameters

That saves us from a couple of strlen() calls.

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

auth: use snprintf() return value

That save us from a strlen().

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

auth: precompute authnameslen

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

darwin: Don't use poll() on versions of darwin before darwin10

Local socket connections do not work on hurd-i386

Local socket connections currently do not work on hurd-i386 because
xcb_auth calls getpeername() on the client socket, but hurd-i386 does
not implement anything in that case (I actually wonder what reasonable
value could be returned).  In such case the xcb code does not actually
need the peer name anyway.

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

use poll() instead of select() when available

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

kludgily hand-merged xid fixes

Signed-off-by: Bart Massey <bart@cs.pdx.edu>
Signed-off-by: Julien Danjou <julien@danjou.info>

Fix do_append() arguments

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

Copy full IPv4 mapping (Bug #20665)

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

Avoid name collisions between xidtype and enum.

These changes are necessary to build with latest xcb/proto.

Signed-off-by: Peter Harris <pharris@opentext.com>

Revert "Don't use enums in generated C code"

This commit broke xcb/util.

This reverts commit 9984b72888108a038d6b3f7dee374d17e26ef9e2.

Signed-off-by: Peter Harris <pharris@opentext.com>

Don't use enums in generated C code - use integer constants instead.

Signed-off-by: Peter Harris <pharris@opentext.com>

Release libxcb 1.2

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

Stop providing autogenerated files in tarball

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

Release libxcb 1.1.93

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

Shutup compiler warning about unused variable...

Apple: Apple launchd cleanup
Added --with-launchd option instead of just using __APPLE__
Fixed opening launchd fd when displayname=NULL

Apple: Enable support for launchd DISPLAY socket

Treat XIDs the same as other cardinal values.

This fixes a bug where c_client.py wasn't generating *_end functions,
but expected them to exist in order to find the subsequent list's start.

Signed-off-by: Peter Harris <peter.harris@hummingbird.com>

Release libxcb 1.1.92

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

Support handing off socket write permission to external code.

Libraries like Xlib, some XCB language bindings, and potentially others
have a common problem: they want to share the X connection with XCB. This
requires coordination of request sequence numbers.  Previously, XCB had an
Xlib-specific lock, and allowed Xlib to block XCB from making requests.
Now we've replaced that lock with a handoff mechanism, xcb_take_socket,
allowing external code to ask XCB for permission to take over the write
side of the socket and send raw data with xcb_writev.  The caller of
xcb_take_socket must supply a callback which XCB can call when it wants
the write side of the socket back to make a request.  This callback
synchronizes with the external socket owner, flushes any output queues if
appropriate, and then returns the sequence number of the last request sent
over the socket.

Commit by Josh Triplett and Jamey Sharp.
Handoff mechanism inspired by Keith Packard.

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.