[Xcb] [PATCH 2/7] xv: Remove obsolete comments in EncodingInfo and QueryEncodings

Daniel Martin consume.noise at gmail.com
Sun Oct 13 04:44:36 PDT 2013


The padding in EncodingInfo (struct) and QueryEncodings (reply) has been
fixed upstream:
    Xvproto: fix xvEncodingInfo to match actual wire protocol
    http://cgit.freedesktop.org/xorg/proto/videoproto/commit/?id=d970512
and
    Xvproto.h: fix xvQueryEncodingsReply to match actual wire protocol
    http://cgit.freedesktop.org/xorg/proto/videoproto/commit/?id=1a96222

So, we can remove the comments.

Signed-off-by: Daniel Martin <consume.noise at gmail.com>
---
 src/xv.xml | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/src/xv.xml b/src/xv.xml
index a05767c..0b55d36 100644
--- a/src/xv.xml
+++ b/src/xv.xml
@@ -112,10 +112,6 @@ <struct name="EncodingInfo">
         <field type="CARD16" name="name_size" />
         <field type="CARD16" name="width" />
         <field type="CARD16" name="height" />
-        <!--
-        Some versions of Xvproto.h incorrectly have
-        this padding after "rate".
-        -->
         <pad bytes="2" />
         <field type="Rational" name="rate" />
         <list type="char" name="name">
@@ -234,10 +230,6 @@ <request name="QueryEncodings" opcode="2">
         <reply>
             <pad bytes="1" />
             <field type="CARD16" name="num_encodings" />
-            <!--
-            In Xvproto.h this padding is 24 bytes
-            in actuality it is 22 bytes
-            -->
             <pad bytes="22" />
             <list type="EncodingInfo" name="info">
                 <fieldref>num_encodings</fieldref>
-- 
1.8.4



More information about the Xcb mailing list