[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
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.
More information about the ffmpeg-devel