> Am I reading this cross-eyed or is this just totally stupid?
>
>       off = lseek( fd, 32 * (off_t) DVD_VIDEO_LB_LEN, SEEK_SET );
> -      if( off == ( 32 * (off_t) DVD_VIDEO_LB_LEN ) ) {
> +      if (off == 16) {
>
> unless lseek errors, off should always == 32 * DVD_VIDEO_LB_LEN and that new 
> test should never succeed.

I was just reading and saying the same things to myself. If Jean
applies this I don't think it ever fires. Or lseek returns something
magical on OSX
and Windows.

I take it HandBrake does not apply this patch?

E

-- 
Erik Hovland
[email protected]
http://hovland.org/
_______________________________________________
DVDnav-discuss mailing list
[email protected]
https://lists.mplayerhq.hu/mailman/listinfo/dvdnav-discuss

Reply via email to