[MPlayer-dev-eng] [Patch 1/2] dvd_set_speed fixes/addons

Reimar Döffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Sat Nov 17 19:54:54 CET 2007


Hello,
On Fri, Nov 16, 2007 at 09:30:58PM +0100, Sebastian Kemper wrote:
> attached patch fixes some glitches in dvd_set_speed() (stream_dvd.c).
> 
> - unsigned speed to int speed (because restoring the initial speed means
>   to set speed=-1)

-1 is just a way to write the "all bits one" special value, I don't
think it makes sense to use signed here.

> - use the correct ATAPI command length for SET STREAMING (12 bytes); at
>   least libata (for now anyways) won't let commands through when the
>   command length is wrong ("old" ATA is more forgiving and cuts the last
>   4 bytes)

Applied.

> - add a check to make sure the device really is an SG device (taken from
>   sg_simple2.c)
>   I'm not sure you guys like the printf() but I don't really want to
>   dive into mplayer's message system

Dropped.

> - add close(fd)'s where feasible
> - add a missing return

Fixed differently.

Greetings,
Reimar Döffinger



More information about the MPlayer-dev-eng mailing list