[PATCH] icccm: atoms previously defined in xcb_atom are now defined in xproto

Arnaud Fontaine arnau at debian.org
Sun Dec 13 09:31:55 PST 2009


---
 configure.ac          |    2 +-
 icccm/Makefile.am     |    4 +-
 icccm/icccm.c         |   88 +++++++++++++++++++++++++------------------------
 icccm/xcb-icccm.pc.in |    2 +-
 4 files changed, 49 insertions(+), 47 deletions(-)

diff --git a/configure.ac b/configure.ac
index d785502..f2c2dbe 100644
--- a/configure.ac
+++ b/configure.ac
@@ -56,7 +56,7 @@ PKG_CHECK_MODULES(XCB, xcb >= 1.4)
 ######
 # Check version of xcb-proto that xcb was compiled against
 ######
-xcbproto_required=1.5
+xcbproto_required=1.6
 
 # Moved from AX_COMPARE_VERSION to maintain proper output
 AC_PROG_AWK
diff --git a/icccm/Makefile.am b/icccm/Makefile.am
index 9ad4238..da755d4 100644
--- a/icccm/Makefile.am
+++ b/icccm/Makefile.am
@@ -8,9 +8,9 @@ xcbinclude_HEADERS = xcb_icccm.h
 AM_CFLAGS = $(CWARNFLAGS)
 
 libxcb_icccm_la_SOURCES = icccm.c
-libxcb_icccm_la_CPPFLAGS = $(XCB_CFLAGS) $(XCB_ATOM_CFLAGS) $(XCB_EVENT_CFLAGS) \
+libxcb_icccm_la_CPPFLAGS = $(XCB_CFLAGS) $(XPROTO_CFLAGS) $(XCB_EVENT_CFLAGS) \
 	$(XCB_PROPERTY_CFLAGS)
-libxcb_icccm_la_LIBADD = $(XCB_LIBS) $(XCB_ATOM_LIBS) $(XCB_PROPERTY_LIBS)
+libxcb_icccm_la_LIBADD = $(XCB_LIBS) $(XPROTO_LIBS) $(XCB_PROPERTY_LIBS)
 libxcb_icccm_la_LDFLAGS = -version-info 1:0:0
 
 pkgconfig_DATA = xcb-icccm.pc
diff --git a/icccm/icccm.c b/icccm/icccm.c
index 18c2cf2..0206e4b 100644
--- a/icccm/icccm.c
+++ b/icccm/icccm.c
@@ -31,8 +31,10 @@
 #include <limits.h>
 #include <string.h>
 
+#include <xcb/xcb.h>
+#include <xcb/xproto.h>
+
 #include "xcb_icccm.h"
-#include "xcb_atom.h"
 
 xcb_get_property_cookie_t
 xcb_get_text_property(xcb_connection_t *c,
@@ -83,7 +85,7 @@ xcb_set_wm_name_checked(xcb_connection_t *c, xcb_window_t window,
                         xcb_atom_t encoding, uint32_t name_len,
                         const char *name)
 {
-  return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window, WM_NAME,
+  return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window, XCB_ATOM_WM_NAME,
 				     encoding, 8, name_len, name);
 }
 
@@ -91,7 +93,7 @@ xcb_void_cookie_t
 xcb_set_wm_name(xcb_connection_t *c, xcb_window_t window, xcb_atom_t encoding,
                 uint32_t name_len, const char *name)
 {
-  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, WM_NAME,
+  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, XCB_ATOM_WM_NAME,
 			     encoding, 8, name_len, name);
 }
 
@@ -99,14 +101,14 @@ xcb_get_property_cookie_t
 xcb_get_wm_name(xcb_connection_t *c,
                 xcb_window_t window)
 {
-  return xcb_get_text_property(c, window, WM_NAME);
+  return xcb_get_text_property(c, window, XCB_ATOM_WM_NAME);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_name_unchecked(xcb_connection_t *c,
                           xcb_window_t window)
 {
-  return xcb_get_text_property_unchecked(c, window, WM_NAME);
+  return xcb_get_text_property_unchecked(c, window, XCB_ATOM_WM_NAME);
 }
 
 uint8_t
@@ -122,7 +124,7 @@ uint8_t
 xcb_watch_wm_name(xcb_property_handlers_t *prophs, uint32_t long_len,
                   xcb_generic_property_handler_t handler, void *data)
 {
-  return xcb_property_set_handler(prophs, WM_NAME, long_len, handler, data);
+  return xcb_property_set_handler(prophs, XCB_ATOM_WM_NAME, long_len, handler, data);
 }
 
 /* WM_ICON_NAME */
@@ -133,7 +135,7 @@ xcb_set_wm_icon_name_checked(xcb_connection_t *c, xcb_window_t window,
                              const char *name)
 {
   return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window,
-				     WM_ICON_NAME, encoding, 8, name_len,
+				     XCB_ATOM_WM_ICON_NAME, encoding, 8, name_len,
 				     name);
 }
 
@@ -141,7 +143,7 @@ xcb_void_cookie_t
 xcb_set_wm_icon_name(xcb_connection_t *c, xcb_window_t window,
                      xcb_atom_t encoding, uint32_t name_len, const char *name)
 {
-  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, WM_ICON_NAME,
+  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, XCB_ATOM_WM_ICON_NAME,
 			     encoding, 8, name_len, name);
 }
 
@@ -149,14 +151,14 @@ xcb_get_property_cookie_t
 xcb_get_wm_icon_name(xcb_connection_t *c,
                      xcb_window_t window)
 {
-  return xcb_get_text_property(c, window, WM_ICON_NAME);
+  return xcb_get_text_property(c, window, XCB_ATOM_WM_ICON_NAME);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_icon_name_unchecked(xcb_connection_t *c,
                                xcb_window_t window)
 {
-  return xcb_get_text_property_unchecked(c, window, WM_ICON_NAME);
+  return xcb_get_text_property_unchecked(c, window, XCB_ATOM_WM_ICON_NAME);
 }
 
 uint8_t
@@ -172,7 +174,7 @@ uint8_t
 xcb_watch_wm_icon_name(xcb_property_handlers_t *prophs, uint32_t long_len,
                        xcb_generic_property_handler_t handler, void *data)
 {
-  return xcb_property_set_handler(prophs, WM_ICON_NAME, long_len, handler,
+  return xcb_property_set_handler(prophs, XCB_ATOM_WM_ICON_NAME, long_len, handler,
 				  data);
 }
 
@@ -184,7 +186,7 @@ xcb_set_wm_client_machine_checked(xcb_connection_t *c, xcb_window_t window,
                                   const char *name)
 {
   return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window,
-				     WM_CLIENT_MACHINE, encoding, 8, name_len,
+				     XCB_ATOM_WM_CLIENT_MACHINE, encoding, 8, name_len,
 				     name);
 }
 
@@ -194,7 +196,7 @@ xcb_set_wm_client_machine(xcb_connection_t *c, xcb_window_t window,
                           const char *name)
 {
   return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window,
-			     WM_CLIENT_MACHINE, encoding, 8, name_len,
+			     XCB_ATOM_WM_CLIENT_MACHINE, encoding, 8, name_len,
 			     name);
 }
 
@@ -202,14 +204,14 @@ xcb_get_property_cookie_t
 xcb_get_wm_client_machine(xcb_connection_t *c,
                           xcb_window_t window)
 {
-  return xcb_get_text_property(c, window, WM_CLIENT_MACHINE);
+  return xcb_get_text_property(c, window, XCB_ATOM_WM_CLIENT_MACHINE);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_client_machine_unchecked(xcb_connection_t *c,
                                     xcb_window_t window)
 {
-  return xcb_get_text_property_unchecked(c, window, WM_CLIENT_MACHINE);
+  return xcb_get_text_property_unchecked(c, window, XCB_ATOM_WM_CLIENT_MACHINE);
 }
 
 uint8_t
@@ -225,7 +227,7 @@ uint8_t
 xcb_watch_wm_client_machine(xcb_property_handlers_t *prophs, uint32_t long_len,
                             xcb_generic_property_handler_t handler, void *data)
 {
-  return xcb_property_set_handler(prophs, WM_CLIENT_MACHINE, long_len, handler,
+  return xcb_property_set_handler(prophs, XCB_ATOM_WM_CLIENT_MACHINE, long_len, handler,
 				  data);
 }
 
@@ -234,20 +236,20 @@ xcb_watch_wm_client_machine(xcb_property_handlers_t *prophs, uint32_t long_len,
 xcb_get_property_cookie_t
 xcb_get_wm_class(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_property(c, 0, window, WM_CLASS, STRING, 0L, 2048L);
+  return xcb_get_property(c, 0, window, XCB_ATOM_WM_CLASS, XCB_ATOM_STRING, 0L, 2048L);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_class_unchecked(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_property_unchecked(c, 0, window, WM_CLASS, STRING, 0L, 2048L);
+  return xcb_get_property_unchecked(c, 0, window, XCB_ATOM_WM_CLASS, XCB_ATOM_STRING, 0L, 2048L);
 }
 
 uint8_t
 xcb_get_wm_class_from_reply(xcb_get_wm_class_reply_t *prop,
                             xcb_get_property_reply_t *reply)
 {
-  if(!reply || reply->type != STRING || reply->format != 8)
+  if(!reply || reply->type != XCB_ATOM_STRING || reply->format != 8)
     return 0;
 
   prop->_reply = reply;
@@ -285,20 +287,20 @@ xcb_get_wm_class_reply_wipe(xcb_get_wm_class_reply_t *prop)
 xcb_get_property_cookie_t
 xcb_get_wm_transient_for(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_property(c, 0, window, WM_TRANSIENT_FOR, WINDOW, 0, 1);
+  return xcb_get_property(c, 0, window, XCB_ATOM_WM_TRANSIENT_FOR, XCB_ATOM_WINDOW, 0, 1);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_transient_for_unchecked(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_property_unchecked(c, 0, window, WM_TRANSIENT_FOR, WINDOW, 0, 1);
+  return xcb_get_property_unchecked(c, 0, window, XCB_ATOM_WM_TRANSIENT_FOR, XCB_ATOM_WINDOW, 0, 1);
 }
 
 uint8_t
 xcb_get_wm_transient_for_from_reply(xcb_window_t *prop,
                                     xcb_get_property_reply_t *reply)
 {
-  if(!reply || reply->type != WINDOW || reply->format != 32 || !reply->length)
+  if(!reply || reply->type != XCB_ATOM_WINDOW || reply->format != 32 || !reply->length)
     return 0;
 
   *prop = *((xcb_window_t *) xcb_get_property_value(reply));
@@ -406,7 +408,7 @@ xcb_set_wm_size_hints_checked(xcb_connection_t *c, xcb_window_t window,
                               xcb_atom_t property, xcb_size_hints_t *hints)
 {
   return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window, property,
-				     WM_SIZE_HINTS, 32, sizeof(*hints) >> 2,
+				     XCB_ATOM_WM_SIZE_HINTS, 32, sizeof(*hints) >> 2,
 				     hints);
 }
 
@@ -415,7 +417,7 @@ xcb_set_wm_size_hints(xcb_connection_t *c, xcb_window_t window,
                       xcb_atom_t property, xcb_size_hints_t *hints)
 {
   return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, property,
-			     WM_SIZE_HINTS, 32, sizeof(*hints) >> 2, hints);
+			     XCB_ATOM_WM_SIZE_HINTS, 32, sizeof(*hints) >> 2, hints);
 }
 
 xcb_get_property_cookie_t
@@ -423,14 +425,14 @@ xcb_get_wm_size_hints(xcb_connection_t *c, xcb_window_t window,
                       xcb_atom_t property)
 {
   /* NumPropSizeElements = 18 (ICCCM version 1). */
-  return xcb_get_property(c, 0, window, property, WM_SIZE_HINTS, 0L, 18);
+  return xcb_get_property(c, 0, window, property, XCB_ATOM_WM_SIZE_HINTS, 0L, 18);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_size_hints_unchecked(xcb_connection_t *c, xcb_window_t window,
                                 xcb_atom_t property)
 {
-  return xcb_get_property_unchecked(c, 0, window, property, WM_SIZE_HINTS,
+  return xcb_get_property_unchecked(c, 0, window, property, XCB_ATOM_WM_SIZE_HINTS,
                                     0L, 18);
 }
 
@@ -444,7 +446,7 @@ xcb_get_wm_size_hints_from_reply(xcb_size_hints_t *hints, xcb_get_property_reply
 
   int length = xcb_get_property_value_length(reply) / (reply->format / 8);
 
-  if (!(reply->type == WM_SIZE_HINTS &&
+  if (!(reply->type == XCB_ATOM_WM_SIZE_HINTS &&
         (reply->format == 8  || reply->format == 16 ||
          reply->format == 32) &&
         /* OldNumPropSizeElements = 15 (pre-ICCCM) */
@@ -490,26 +492,26 @@ xcb_void_cookie_t
 xcb_set_wm_normal_hints_checked(xcb_connection_t *c, xcb_window_t window,
                                 xcb_size_hints_t *hints)
 {
-  return xcb_set_wm_size_hints_checked(c, window, WM_NORMAL_HINTS, hints);
+  return xcb_set_wm_size_hints_checked(c, window, XCB_ATOM_WM_NORMAL_HINTS, hints);
 }
 
 xcb_void_cookie_t
 xcb_set_wm_normal_hints(xcb_connection_t *c, xcb_window_t window,
                         xcb_size_hints_t *hints)
 {
-  return xcb_set_wm_size_hints(c, window, WM_NORMAL_HINTS, hints);
+  return xcb_set_wm_size_hints(c, window, XCB_ATOM_WM_NORMAL_HINTS, hints);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_normal_hints(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_wm_size_hints(c, window, WM_NORMAL_HINTS);
+  return xcb_get_wm_size_hints(c, window, XCB_ATOM_WM_NORMAL_HINTS);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_normal_hints_unchecked(xcb_connection_t *c, xcb_window_t window)
 {
-  return xcb_get_wm_size_hints_unchecked(c, window, WM_NORMAL_HINTS);
+  return xcb_get_wm_size_hints_unchecked(c, window, XCB_ATOM_WM_NORMAL_HINTS);
 }
 
 uint8_t
@@ -601,23 +603,23 @@ xcb_void_cookie_t
 xcb_set_wm_hints_checked(xcb_connection_t *c, xcb_window_t window,
                          xcb_wm_hints_t *hints)
 {
-  return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window, WM_HINTS,
-				     WM_HINTS, 32, sizeof(*hints) >> 2, hints);
+  return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window, XCB_ATOM_WM_HINTS,
+				     XCB_ATOM_WM_HINTS, 32, sizeof(*hints) >> 2, hints);
 }
 
 xcb_void_cookie_t
 xcb_set_wm_hints(xcb_connection_t *c, xcb_window_t window,
                  xcb_wm_hints_t *hints)
 {
-  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, WM_HINTS,
-			     WM_HINTS, 32, sizeof(*hints) >> 2, hints);
+  return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, XCB_ATOM_WM_HINTS,
+			     XCB_ATOM_WM_HINTS, 32, sizeof(*hints) >> 2, hints);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_hints(xcb_connection_t *c,
                  xcb_window_t window)
 {
-  return xcb_get_property(c, 0, window, WM_HINTS, WM_HINTS, 0L,
+  return xcb_get_property(c, 0, window, XCB_ATOM_WM_HINTS, XCB_ATOM_WM_HINTS, 0L,
                           XCB_NUM_WM_HINTS_ELEMENTS);
 }
 
@@ -625,7 +627,7 @@ xcb_get_property_cookie_t
 xcb_get_wm_hints_unchecked(xcb_connection_t *c,
                            xcb_window_t window)
 {
-  return xcb_get_property_unchecked(c, 0, window, WM_HINTS, WM_HINTS, 0L,
+  return xcb_get_property_unchecked(c, 0, window, XCB_ATOM_WM_HINTS, XCB_ATOM_WM_HINTS, 0L,
                                     XCB_NUM_WM_HINTS_ELEMENTS);
 }
 
@@ -633,7 +635,7 @@ uint8_t
 xcb_get_wm_hints_from_reply(xcb_wm_hints_t *hints,
                             xcb_get_property_reply_t *reply)
 {
-  if(!reply || reply->type != WM_HINTS || reply->format != 32)
+  if(!reply || reply->type != XCB_ATOM_WM_HINTS || reply->format != 32)
     return 0;
 
   int length = xcb_get_property_value_length(reply);
@@ -670,7 +672,7 @@ xcb_set_wm_protocols_checked(xcb_connection_t *c, xcb_atom_t wm_protocols,
                              xcb_atom_t *list)
 {
   return xcb_change_property_checked(c, XCB_PROP_MODE_REPLACE, window,
-				     wm_protocols, ATOM, 32, list_len, list);
+				     wm_protocols, XCB_ATOM_ATOM, 32, list_len, list);
 }
 
 xcb_void_cookie_t
@@ -678,14 +680,14 @@ xcb_set_wm_protocols(xcb_connection_t *c, xcb_atom_t wm_protocols,
                      xcb_window_t window, uint32_t list_len, xcb_atom_t *list)
 {
   return xcb_change_property(c, XCB_PROP_MODE_REPLACE, window, wm_protocols,
-			     ATOM, 32, list_len, list);
+			     XCB_ATOM_ATOM, 32, list_len, list);
 }
 
 xcb_get_property_cookie_t
 xcb_get_wm_protocols(xcb_connection_t *c, xcb_window_t window,
                      xcb_atom_t wm_protocol_atom)
 {
-  return xcb_get_property(c, 0, window, wm_protocol_atom, ATOM, 0, UINT_MAX);
+  return xcb_get_property(c, 0, window, wm_protocol_atom, XCB_ATOM_ATOM, 0, UINT_MAX);
 }
 
 xcb_get_property_cookie_t
@@ -693,14 +695,14 @@ xcb_get_wm_protocols_unchecked(xcb_connection_t *c,
                                xcb_window_t window,
                                xcb_atom_t wm_protocol_atom)
 {
-  return xcb_get_property_unchecked(c, 0, window, wm_protocol_atom, ATOM, 0,
+  return xcb_get_property_unchecked(c, 0, window, wm_protocol_atom, XCB_ATOM_ATOM, 0,
                                     UINT_MAX);
 }
 
 uint8_t
 xcb_get_wm_protocols_from_reply(xcb_get_property_reply_t *reply, xcb_get_wm_protocols_reply_t *protocols)
 {
-  if(!reply || reply->type != ATOM || reply->format != 32)
+  if(!reply || reply->type != XCB_ATOM_ATOM || reply->format != 32)
     return 0;
 
   protocols->_reply = reply;
diff --git a/icccm/xcb-icccm.pc.in b/icccm/xcb-icccm.pc.in
index 0586560..f85d0cb 100644
--- a/icccm/xcb-icccm.pc.in
+++ b/icccm/xcb-icccm.pc.in
@@ -6,6 +6,6 @@ includedir=@includedir@
 Name: XCB ICCCM library
 Description: XCB ICCCM binding
 Version: @PACKAGE_VERSION@
-Requires: xcb xcb-atom xcb-property
+Requires: xcb xcb-proto xcb-property
 Libs: -L${libdir} -lxcb-icccm @LIBS@
 Cflags: -I${includedir}
-- 
1.6.6.rc1


--=-=-=--


More information about the Xcb mailing list