[FFmpeg-devel] [Patch v4] flv: Add XV (Xunlie Video) Support, Updated patch for ticket #3720

Carl Eugen Hoyos ceffmpeg at gmail.com
Tue Apr 9 14:40:12 EEST 2019


2019-04-09 11:36 GMT+02:00, Shivam Goyal <shivgo at iitk.ac.in>:
> flv: Add XV (Xunlie Video) Support.
>
> Updated patch for ticket #3720.

Above is an acceptable commit message, slightly
better would be:
"
lavf/flv: Add XV (Xunlie Video) Support.

Fixes ticket #3720.
"
I applied your patch with git am and what I get as commit
message is:
Updated patch for ticket #3720. flv: Add XV (Xunlei Video) support

This is not ok.

The following probably doesn't work for some very large files:

> +    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;

I suggested:
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;

There are still "for(" that should be "for (", "if(" that should be "if ("
and "+ rot )" that should be "+ rot)" (or no parentheses).
The script tools/patcheck should find them, it will also tell you
to add a line to Changelog and bump libavformat minor version.

Carl Eugen


More information about the ffmpeg-devel mailing list