[FFmpeg-devel] Support for h264/SVC over RTP and SVC base layer decoding in h264
Carl Eugen Hoyos
ceffmpeg at gmail.com
Wed Nov 1 17:57:21 EET 2017
2017-11-01 16:18 GMT+01:00 Breeden, Joshua <JBREED05 at harris.com>:
>> Your patches contain a lot of trailing whitespace (afaict...)
>> that cannot be committed to our repository, please remove it.
>
> Thanks for pointing this out. I've attached new patches to fix this problem.
> patcheck complains about my non-doxy comments, and about possible
> unused variables. Variables are definitely being used.
The tool is not perfect.
> Is the etiquette to remove all comments?
No but I wonder if the variables "y" and "t" can have more useful names
(that may or may not make the comment unneeded),
> Also, I added a Changelog entry for the RTP depacketization of SVC as
> patcheck suggested.
Please do not remove the empty line.
Carl Eugen
More information about the ffmpeg-devel
mailing list