[VDPAU] [PATCH] vdpauinfo: add option processing

[Aaron Plattner]

On 08/28/2014 01:02 PM, Rémi Denis-Courmont wrote:
> Le jeudi 28 août 2014, 10:27:31 Aaron Plattner a écrit :
>> Add a rudimentary option parser.  Parse the options --display and --screen,
>> to specify the argument to XOpenDisplay and the screen within the X display
>> to use, respectively.
>
> Shouldn't the screen number be taken from the Display, I mean
> XScreenNumberOfScreen()?

If it's not specified explicitly by the --screen parameter, then it uses 
DefaultScreen(display) so yeah, you can also use DISPLAY=:0.1 or 
vdpauinfo --display :0.1 instead of using the --screen parameter.

>> Signed-off-by: Aaron Plattner <aplattner at nvidia.com>
>> ---
>> This is the last thing I wanted to do before declaring vdpauinfo version
>> 1.0. Does anyone else have anything to get into it before that?
>
> I wish VDPAU as a whole would use XCB instead of (or as an alternative to) the
> horrible and antique Xlib, but that is wishful thinking on my part.

If that would be useful to you, you can propose adding a 
VdpDeviceCreateX11Xcb or something.  I don't want libvdpau to have a 
hard dependency on XCB though, since it currently supports distributions 
that predate it and it would be a shame to lose that.

-- 
Aaron
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------