[FFmpeg-devel] [PATCH v3] Added XV Support

Carl Eugen Hoyos ceffmpeg at gmail.com
Tue Apr 9 00:11:11 EEST 2019


2019-04-08 22:37 GMT+02:00, Shivam Goyal <shivgo at iitk.ac.in>:
> Updated patch for ticket #3720 .
>
> I have tried to improve the patch as suggested.

Please fix the commit message as suggested by Lauri
and please make above (first) line part of the commit
message.

The first avio_seek() should be a skip, the header
function always starts reading the file from the top.

> +    offset = (((avio_r8(ic) + rot)&0xff)<<24) |
> +                (((avio_r8(ic) + rot)&0xff)<<16) |
> +                (((avio_r8(ic) + rot)&0xff)<<8) |
> +                (((avio_r8(ic) + rot)&0xff)) + 0x200000;

offset = ((avio_r8(ic) + rot & 0xff) << 24 |
          (avio_r8(ic) + rot & 0xff) << 16...

assuming this produces no warnings.
There is no 12-char indentation, only vertical alignment.

Carl Eugen


More information about the ffmpeg-devel mailing list