screensaver.xml contains the following specification: <event name="Notify" number="0"> <field type="CARD8" name="code"/> <field type="BYTE" name="state"/> <!-- enum State --> <pad bytes="1" /> <field type="CARD16" name="sequence_number"/> <field type="TIMESTAMP" name="time"/> <field type="WINDOW" name="root"/> <field type="WINDOW" name="window"/> <field type="BYTE" name="kind"/> <!-- enum Kind --> <field type="BOOL" name="forced"/> <pad bytes="14"/> </event> The sequence_number is added automatically when the struct type is generated (but then it is called sequence). Similarly, the code field is what xcb calls response_type. These fields should not be specified in the XML file. I also do not know why the 1-byte padding is there (added for v1.1 extension update); maybe to hide symptoms resulting from the duplicate fields? This was pointed out when screensaver.xml was first submitted [1] but apparently it has never been fixed; seems like I am the first to actually use the screensaver extension with xcb. [1] http://lists.freedesktop.org/archives/xcb/2005-July/000910.html
Thanks for your detailed report. You're totally right with your comment - those fields shouldn't be there. Now, it's fixed: a50670c screensaver: Remove wrong fields from Notify event
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.