2019-04-08 22:37 GMT+02:00, Shivam Goyal <[email protected]>:
> 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
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".
