Re: [FFmpeg-devel] [PATCH] added slicing yuv, rgb

2014-12-17 Thread Clément Bœsch
> Subject: Re: [FFmpeg-devel] [PATCH] added slicing yuv, rgb You probably want to say what part it affects, as well as mentioning threading. So probably something like "avfilter/lut: support rgb and yuv slice threading" You might also want to mention in the commit description wha

Re: [FFmpeg-devel] [PATCH] added slicing yuv, rgb

2014-12-17 Thread Michael Niedermayer
On Tue, Dec 16, 2014 at 10:44:27PM -0800, Yayoi wrote: > --- > libavfilter/allfilters.c | 2 +- > libavfilter/vf_lut.c | 119 > ++- > 2 files changed, 88 insertions(+), 33 deletions(-) ./ffplay -i matrixbench_mpeg2.mpg -vf "lutrgb=c0=1/val" show

Re: [FFmpeg-devel] [PATCH] added slicing yuv, rgb

2014-12-17 Thread Carl Eugen Hoyos
Yayoi 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 consid

[FFmpeg-devel] [PATCH] added slicing yuv, rgb

2014-12-17 Thread Yayoi
--- libavfilter/allfilters.c | 2 +- libavfilter/vf_lut.c | 119 ++- 2 files changed, 88 insertions(+), 33 deletions(-) diff --git a/libavfilter/allfilters.c b/libavfilter/allfilters.c index adb86be..76341bd 100644 --- a/libavfilter/allfilters.c +