videoproto: Changes to 'master'

Alan Coopersmith alanc at kemper.freedesktop.org
Thu Jun 27 22:08:43 PDT 2013


 Xvproto.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1a96222d64cde75e20fd9bb7a717cf269c7e9a59
Author: Alan Coopersmith <alan.coopersmith at oracle.com>
Date:   Sun Jun 23 00:34:52 2013 -0700

    Xvproto.h: fix xvQueryEncodingsReply to match actual wire protocol
    
    padl3 was declared as CARD32, which for proper alignment, got two
    additional bytes of padding silently inserted before it.  Declaring
    it as CARD16 aligns it properly after the CARD16 num_encodings.
    
    Fixes clang warning:
    Xvproto.h:462:10: warning: padding struct 'struct _QueryEncodingsReply' with 2 bytes to align 'padl3' [-Wpadded]
      CARD32 padl3 B32;
             ^
    This makes sizeof(xvQueryEncodingsReply) match the sz_xvQueryEncodingsReply
    which is used in libXv & the X server for the size sent over the wire.
    
    Reported-by: Daniel Martin <consume.noise at gmail.com>
    Reviewed-by: Julien Cristau <jcristau at debian.org>
    Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>



More information about the xorg-commit mailing list