Yayoi <yayoi.ukai <at> gmail.com> writes:

> +++ b/libavfilter/allfilters.c

> -
> +    

This change cannot be committed to our 
repository (trailing whitespace), 
please remove it.

Your commit message should contain a 
hint to vf_lut, most people here seem 
to believe in "lavfi/vf_lut: Added..."

Did you consider to split your patch in 
two? One patch for the factorisation 
and one for the actual adding of the 
slice mode? (May be overkill.)

Thank you, Carl Eugen

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to