| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The align pad is needed so that a in list of struct "AttributeInfo",
the 32-bit fields in subsequent "AttributeInfo" are aligned.
(all fields except "name" are 32-bit values)
The xv-protocol-spec does not contain the protocol encoding.
(it specifies the protocol on a higher level)
The X-Server uses function "WriteToClient" to write the list "name"
to the connection:
http://cgit.freedesktop.org/xorg/xserver/tree/Xext/xvdisp.c?id=xorg-server-1.17.99.901#n809
Function "WriteToClient" does 4-byte alignment padding:
http://cgit.freedesktop.org/xorg/xserver/tree/os/io.c?id=xorg-server-1.17.99.901#n804
BTW, With the current server impl, the alignpad is not strictly needed
because the field "size", which describes the length of list "name"
is set to a multiple of 4:
http://cgit.freedesktop.org/xorg/xserver/tree/Xext/xvdisp.c?id=xorg-server-1.17.99.901#n805
Signed-off-by: Christian Linhart <chris@demorecorder.com>
Reviewed-by: Peter Harris <pharris@opentext.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The align pad is needed so that a in list of struct "EncodingInfo",
the 32-bit fiels in subsequent "EncodingInfo" are aligned.
(the field "encoding" is a 32-bit value, and the field "rate"
contains two 32-bit values)
The xv-protocol-spec does not contain the protocol encoding.
(it specifies the protocol on a higher level)
The X-Server uses function "WriteToClient" to write the list "name"
to the connection:
http://cgit.freedesktop.org/xorg/xserver/tree/Xext/xvdisp.c?id=xorg-server-1.17.99.901#n456
Function "WriteToClient" does 4-byte alignment padding:
http://cgit.freedesktop.org/xorg/xserver/tree/os/io.c?id=xorg-server-1.17.99.901#n804
Signed-off-by: Christian Linhart <chris@demorecorder.com>
Reviewed-by: Peter Harris <pharris@opentext.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The (list) 'name' in the struct AdaptorInfo is not padded. Therefore
we've to add an alignment pad after it.
Reference: http://cgit.freedesktop.org/xorg/lib/libXv/tree/src/Xv.c#n250
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=79808
Signed-off-by: Daniel Martin <consume.noise@gmail.com>
Signed-off-by: Peter Harris <pharris@opentext.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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@gmail.com>
Reviewed-by: Arnaud Fontaine <arnau@debian.org>
|
|
|
|
|
|
| |
Signed-off-by: Daniel Martin <consume.noise@gmail.com>
Reviewed-by: Ran Benita <ran234@gmail.com> (and tested)
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
|
|
|
|
| |
Signed-off-by: Peter Harris <pharris@opentext.com>
|
|
|
|
|
|
|
| |
This change makes it easier to process the xml file in a single streamy
pass.
Signed-off-by: Julien Danjou <julien@danjou.info>
|
|
|
|
| |
Signed-off-by: Julien Danjou <julien@danjou.info>
|
|
|
|
| |
Signed-off-by: Julien Danjou <julien@danjou.info>
|
|
|
|
|
|
|
|
| |
I noticed that xv.xml assumes implicit padding in a couple of locations.
Please apply this patch to make the padding explicit. Or is implicit
pad support required in all consumers of xcb/proto?
Signed-off-by: Julien Danjou <julien@danjou.info>
|
| |
|
|
|
|
| |
<jcristau@debian.org>
|
|
|
|
|
|
|
| |
Only one use of <localfield> remained, for a list length expression in
xv.xml. List length parameters that don't actually appear in the
protocol should be left implicit: if no length expression is given, then
a localfield will be automatically created by c-client.xsl.
|
| |
|
|
Now that the extensions do not get installed to an extensions/ subdirectory,
move them out of the extensions/ subdirectory in the source as well.
|