Remove _xcb_assert_valid_sequence. One test is trivially true, and the other may...

diff --git a/src/xcb_conn.c b/src/xcb_conn.c
index 69aed02..9a7399e 100644 (file)
--- 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 (file)
--- 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