From 1b83f8f8f326eca9d8852c82dd36696f81a720dc Mon Sep 17 00:00:00 2001 From: Jamey Sharp Date: Thu, 23 Feb 2006 14:30:08 -0800 Subject: [PATCH] Remove _xcb_assert_valid_sequence. One test is trivially true, and the other may be temporarily violated without anything bad happening. --- src/xcb_conn.c | 2 -- src/xcbint.h | 10 ---------- 2 files changed, 12 deletions(-) diff --git a/src/xcb_conn.c b/src/xcb_conn.c index 69aed02..9a7399e 100644 --- a/src/xcb_conn.c +++ b/src/xcb_conn.c @@ -181,8 +181,6 @@ int _xcb_conn_wait(XCBConnection *c, const int should_write, pthread_cond_t *con int ret; fd_set rfds, wfds; - _xcb_assert_valid_sequence(c); - /* If the thing I should be doing is already being done, wait for it. */ if(should_write ? c->out.writing : c->in.reading) { diff --git a/src/xcbint.h b/src/xcbint.h index 402b042..e7e6688 100644 --- a/src/xcbint.h +++ b/src/xcbint.h @@ -28,16 +28,6 @@ #ifndef __XCBINT_H #define __XCBINT_H -/* Not simply (a <= b) because eventually the 32-bit sequence number - * will wrap, causing earlier sequence numbers to be higher than later - * ones for a brief but fatal period. (a and b must be unsigned.) */ -#define _xcb_assert_sequence_less(a,b) assert((b) - (a) < 65536) - -#define _xcb_assert_valid_sequence(c) do { \ - _xcb_assert_sequence_less((c)->in.request_read, (c)->out.request_written); \ - _xcb_assert_sequence_less((c)->out.request_written, (c)->out.request); \ -} while(0) - enum workarounds { WORKAROUND_NONE, WORKAROUND_GLX_GET_FB_CONFIGS_BUG -- 2.7.4