[Xcb] [PATCH libxcb] c_client.py: Inject full_sequence into GE events

Daniel Martin consume.noise at gmail.com
Sat Jun 8 02:20:39 PDT 2013


The generic event structure xcb_ge_event_t has the full_sequence field
at the 32byte boundary. That's why we've to inject this field into GE
events while generating the structure for them. Otherwise we would read
garbage (the internal full_sequence) when accessing normal event fields
there.

Signed-off-by: Daniel Martin <consume.noise at gmail.com>
---
 src/c_client.py | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/src/c_client.py b/src/c_client.py
index 72a4007..ff5237a 100644
--- a/src/c_client.py
+++ b/src/c_client.py
@@ -2853,6 +2853,23 @@ def c_event(self, name):
     '''
     Exported function that handles event declarations.
     '''
+
+    # The generic event structure xcb_ge_event_t has the full_sequence field
+    # at the 32byte boundary. That's why we've to inject this field into GE
+    # events while generating the structure for them. Otherwise we would read
+    # garbage (the internal full_sequence) when accessing normal event fields
+    # there.
+    if hasattr(self, 'is_ge_event') and self.is_ge_event and self.name == name:
+        event_size = 0
+        for field in self.fields:
+            if field.type.size != None and field.type.nmemb != None:
+                event_size += field.type.size * field.type.nmemb
+            if event_size == 32:
+                full_sequence = Field(tcard32, tcard32.name, 'full_sequence', False, True, True)
+                idx = self.fields.index(field)
+                self.fields.insert(idx + 1, full_sequence)
+                break
+
     _c_type_setup(self, name, ('event',))
 
     # Opcode define
-- 
1.8.3



More information about the Xcb mailing list