[FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/trim.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/libavfilter/trim.c b/libavfilter/trim.c index b2d9c6f..9d040dd 100644 --- a/libavfilter/trim.c +++ b/libavfilter/trim.c @@ -345,6 +345,32 @@ drop:

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Clément Bœsch
On Wed, Dec 30, 2015 at 09:17:56AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/trim.c | 27 +++ > 1 file changed, 27 insertions(+) > > diff --git a/libavfilter/trim.c b/libavfilter/trim.c > index b2d9c6f..9d040dd 100644

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Paul B Mahol
On 12/30/15, Clement Boesch wrote: > On Wed, Dec 30, 2015 at 09:17:56AM +0100, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/trim.c | 27 +++ >> 1 file changed, 27 insertions(+) >> >> diff --git

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Nicolas George
Le decadi 10 nivôse, an CCXXIV, Clement Boesch a écrit : > Yes; why ff_default_query_formats() is doing a query with > all_channel_layouts and is not actually ff_query_formats_all (which is > doing ff_all_channel_counts)? Carefulness. Filters from the fork use ff_default_query_formats() but still

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Clément Bœsch
On Wed, Dec 30, 2015 at 09:14:23AM +, Paul B Mahol wrote: > > Q: why is it not the default? > > > > The all channel counts layout? > Yes; why ff_default_query_formats() is doing a query with all_channel_layouts and is not actually ff_query_formats_all (which is doing ff_all_channel_counts)?

Re: [FFmpeg-devel] [PATCH] avfilter/trim: support all channel numbers in atrim filter

2015-12-30 Thread Ganesh Ajjanagadde
On Wed, Dec 30, 2015 at 12:17 AM, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol [...] > > +static int atrim_query_formats(AVFilterContext *ctx) > +{ > +AVFilterFormats *formats; > +AVFilterChannelLayouts *layouts; > +int ret; > + > +layouts