fixes + valid css

fixes + valid css

lots of fixes. Thanks to Indan Zupancic

lots of fixes. Thanks to Indan Zupancic

Add *.tar.{bz2,gz} to .gitignore for the benefit of "make distcheck".

API/ABI change: XCBSendRequest returns the sequence number instead of using an out-parameter. Now 0 is a special sequence number indicating failure.

Add 32-bit full_sequence fields to generic errors and events, so callers can always correlate events and errors with requests correctly.

Tweak to previous API change: Require that spare iovecs fall before vector[0]. Leave vector in well-defined state.

assert() that XCBSendRequest was handed enough space to set opcodes and short length fields.

Bugfix: The fixed-length part needs two iovecs reserved, just like all the other parts.

Use XPath position() function instead of xsl:number, to evaluate array indices while generating code for easier readability.

API/ABI change: XCBSendRequest callers must pad to 4-byte boundaries now. When not in RAW mode, a null pointer for iov_base is replaced by up to 3 padding bytes.

Separate notion of request-completed from current-request, and mark requests completed more aggressively. Detects some usage errors that are otherwise undetectable.

Buffer a couple CARD32s on the stack instead of using an extra iovec. Also a bugfix: do not hold a reference to longlen after it goes out of scope.

Add XCB_REQUEST_RAW flag for XCBSendRequest.

Bugfix: how about *not* leaking all pending_replies when no reply matches, as often happens with Xlib?

Move test for other writing threads *before* allocating a sequence number.

_xcb_list is no longer used, so remove it. Simplify _xcb_map.

Replace readers generic list with hand-written typesafe version.

Rename struct XCBReplyData to reader_list to follow my naming conventions and better reflect its purpose.

In struct XCBReplyData, change void *data to pthread_cond_t *data. That was dumb.

Delete unused xcb_list functions and refactor others.

Replace current_reply generic queue with hand-written version. No generic queues remain so delete their implementation.

Replace events generic queue with hand-written typesafe version.

API/ABI break: Add flags to XCBSendRequest, first for error-checked requests.
There's no more race condition between event and reply handling.
The *RequestBlind and *RequestChecked functions are not yet implemented.

Replace pending_replies generic queue with a hand-implemented typesafe version.

Replace my old generic map ADT with a growable array for the extension cache.

XCB has not had tracing features for a long time: remove the remnants.

Rearrange an if statement that's been bothering me.

Coalesce _xcb_writev into _xcb_out_write and simplify.

Take advantage of Requires and *.private fields for a more accurate pkg-config file.

Move _xcb_set_fd_flags to xcb_conn.c and make it static. xcb_util.c now has only public functions.

Move _xcb_read_block to xcb_in.c and make it static. Change calls in xcb_conn.c to _xcb_in_read_block instead.

Remove XCB_CEIL and use a simpler definition for XCB_PAD.

Quit using "-include config.h": use #ifdef HAVE_CONFIG_H etc. like everyone else.

Coalesce readn into _xcb_in_read and simplify.

Move _xcb_readn to xcb_in.c and make it static. Minor change to _xcb_read_block to not depend on _xcb_readn.

Bugfix: protect the output queue from being written while another thread is flushing it.

Simplify: Always use writev. (In _xcb_out_flush, convert the output queue to a single iovec if needed.)

Factor padding out of _xcb_out_write_block and into its callers, XCBSendRequest and write_setup.
This requires dynamically allocating memory in XCBSendRequest, but this
malloc/free pair turns out to cause a 30% speed hit for the 'x11perf -noop'
test -- so for the moment I use alloca where available and fall back to malloc
on other platforms. Later I think I'll change the contract of XCBSendRequest
so the caller is responsible for memory allocation, because the caller ought
to always be able to stack-allocate here.

Add XCBGetQueuedRequestRead for Xlib that does no syscalls, just returns whatever XCB already knows about.

Minor performance fix: Change the contract on XCBGetRequestSent so that it does not waste time re-locking. This is for Xlib, and Xlib has already locked.

Minor performance fix: Only rearrange buffers for BIG-REQUESTs.

Move request_written update back where it was for now: doing it early can cause XCBWaitForReply to wrongly believe that the request has been flushed. Eventually, we should fix bug #6021.

Remove _xcb_assert_valid_sequence. One test is trivially true, and the other may be temporarily violated without anything bad happening.

Move _xcb_write and _xcb_writev to xcb_out.c and make them static, since only _xcb_out_write calls them.

Ignore test byproducts.

More return value changes, and make _xcb_in_read_packet static since it is not called from outside xcb_in.c.

More return value changes.

Make the return value of _xcb_conn_wait boolean, instead of syscall-like.

Bugfix: move request_written update *before* _xcb_conn_wait in _xcb_out_flush. Otherwise a reply may be read before we record that we have sent the request, and then XCB gets... confused.

Remove USE_THREAD_ASSERT compile option. _xcb_conn_wait can no longer be re-entered, since we have no callback hooks now.

Remove vestiges of adjacent request combining implementation. It is very dead.

Control debugging, optimization, and warning flags all independently.

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

Update .gitignores for .o files and autofoo stuff.

Add missing entries to .gitignore files.

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

Move .cvsignore to .gitignore.

Quit defining _XOPEN_SOURCE. We never needed it...

Remove the old ChangeLog file. GIT makes that obsolete.

Workaround X server bug, fd.o #3210: if a GLXGetFBConfigs request is sent, fix the length field in the reply.

Remove xcl and CVSROOT.