[FFmpeg-devel] [Patch] New filter -- dejudder

Nicholas Robbins nickrobbins at yahoo.com
Wed Jan 29 23:18:29 CET 2014


> On Wednesday, January 29, 2014 5:03 PM, Andrey Utkin <andrey.krieger.utkin at gmail.com> wrote:

> > Sorry if this has been already said, but you have to follow coding
> style - spacing, braces placing etc. I don't know/remember if there's
> exact description of FFmpeg coding style guidelines, but to be short
> it's K&R style + allowance to use some of C99 features.
> In details - somewhere you lack spaces around '=', '+', and 
> space after comma.
> I haven't read all your patch yet, though.
> 
> -- 
> Andrey Utkin

I tried to make it consistent, Lukasz Marek's suggestions helped with that I think.
New patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Adding-dejudder-filter-to-remove-judder-produced-by-.patch
Type: text/x-patch
Size: 8235 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140129/1a7d28d2/attachment.bin>


More information about the ffmpeg-devel mailing list