Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Paul B Mahol
On 8/16/20, Michael Niedermayer wrote: > On Sat, Aug 15, 2020 at 07:29:40PM +0200, Paul B Mahol wrote: >> On 8/15/20, Michael Niedermayer wrote: >> > On Fri, Aug 14, 2020 at 02:24:25PM +0200, Paul B Mahol wrote: >> >> On 8/14/20, Paul B Mahol wrote: >> >> > On 8/13/20, Paul B Mahol wrote: >> >>

Re: [FFmpeg-devel] [PATCH v2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-15 Thread lance . lmwang
On Sat, Aug 15, 2020 at 11:38:29PM +0800, Guangxin Xu wrote: > On Sat, Aug 15, 2020 at 10:28 PM wrote: > > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > configure | 2 +- > > libavformat/mpegts.h| 16 ++ > > libavformat/mpegtsenc.c | 84 > > +++

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Michael Niedermayer
On Sat, Aug 15, 2020 at 07:29:40PM +0200, Paul B Mahol wrote: > On 8/15/20, Michael Niedermayer wrote: > > On Fri, Aug 14, 2020 at 02:24:25PM +0200, Paul B Mahol wrote: > >> On 8/14/20, Paul B Mahol wrote: > >> > On 8/13/20, Paul B Mahol wrote: > >> >> Hi, > >> >> > >> >> patch attached. > >> >>

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/aacdec_template: add support for 22.2 / channel_config 13

2020-08-15 Thread Michael Niedermayer
On Sat, Aug 01, 2020 at 02:07:30PM +0300, Jan Ekström wrote: > --- > libavcodec/aacdec_template.c | 89 +++- > 1 file changed, 78 insertions(+), 11 deletions(-) > > diff --git a/libavcodec/aacdec_template.c b/libavcodec/aacdec_template.c > index 21db12fdab..8c5048c

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

2020-08-15 Thread Mark Thompson
On 15/08/2020 16:03, Nicolas George wrote: Mark Thompson (12020-08-14): The relation to the existing -autorotate and -autoscale options is confusing - those aren't disabled by this (because they are applied differently), but it sounds like they would be. I updated the doc to: Disable automati

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/v4l2_context: return EAGAIN to signal full buffers

2020-08-15 Thread Mark Thompson
On 17/07/2020 03:09, Andriy Gelman wrote: From: Andriy Gelman Return proper error when a frame buffers are full. This path is triggered on the DragonBoard 410c since the encoding API change in commit 827d6fe73d2f5472c1c2. Signed-off-by: Andriy Gelman --- libavcodec/v4l2_context.c | 2 +- 1

Re: [FFmpeg-devel] [PATCH v6] avdevice/xcbgrab: check return values of xcb query functions

2020-08-15 Thread Andriy Gelman
On Thu, 13. Aug 23:43, Andriy Gelman wrote: > On Sat, 08. Aug 09:55, Andriy Gelman wrote: > > On Wed, 05. Aug 14:37, Moritz Barsnick wrote: > > > On Mon, Jul 20, 2020 at 09:18:55 +0200, Alexander Strasser wrote: > > > > On 2020-07-19 19:47 -0400, Andriy Gelman wrote: > > > > > > > This check seems

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/v4l2_context: return EAGAIN to signal full buffers

2020-08-15 Thread Andriy Gelman
On Fri, 24. Jul 15:49, Andriy Gelman wrote: > On Thu, 16. Jul 22:09, Andriy Gelman wrote: > > From: Andriy Gelman > > > > Return proper error when a frame buffers are full. This path is > > triggered on the DragonBoard 410c since the encoding API change in > > commit 827d6fe73d2f5472c1c2. > > >

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Paul B Mahol
On 8/15/20, Michael Niedermayer wrote: > On Fri, Aug 14, 2020 at 02:24:25PM +0200, Paul B Mahol wrote: >> On 8/14/20, Paul B Mahol wrote: >> > On 8/13/20, Paul B Mahol wrote: >> >> Hi, >> >> >> >> patch attached. >> >> >> >> Please review and/or benchmark, especially .asm file. >> >> >> > >> > U

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Michael Niedermayer
On Fri, Aug 14, 2020 at 02:24:25PM +0200, Paul B Mahol wrote: > On 8/14/20, Paul B Mahol wrote: > > On 8/13/20, Paul B Mahol wrote: > >> Hi, > >> > >> patch attached. > >> > >> Please review and/or benchmark, especially .asm file. > >> > > > > Updated patch attached. > > > > Sorry, wrong patch.

Re: [FFmpeg-devel] [PATCH] avformat/av1dec: inline obu_read_data() and obu_prefetch() into obu_get_packet()

2020-08-15 Thread James Almer
On 8/15/2020 12:39 PM, Guangxin Xu wrote: > On Fri, Aug 14, 2020 at 10:30 PM James Almer wrote: > >> They don't really help making the demuxer more readable. >> >> Signed-off-by: James Almer >> --- >> libavformat/av1dec.c | 56 +--- >> 1 file changed, 21

Re: [FFmpeg-devel] [PATCH] avformat/av1dec: inline obu_read_data() and obu_prefetch() into obu_get_packet()

2020-08-15 Thread Guangxin Xu
On Fri, Aug 14, 2020 at 10:30 PM James Almer wrote: > They don't really help making the demuxer more readable. > > Signed-off-by: James Almer > --- > libavformat/av1dec.c | 56 +--- > 1 file changed, 21 insertions(+), 35 deletions(-) > Good improvment, LG

Re: [FFmpeg-devel] [PATCH v2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-15 Thread Guangxin Xu
On Sat, Aug 15, 2020 at 10:28 PM wrote: > From: Limin Wang > > Signed-off-by: Limin Wang > --- > configure | 2 +- > libavformat/mpegts.h| 16 ++ > libavformat/mpegtsenc.c | 84 > +++-- > 3 files changed, 99 insertions(+),

Re: [FFmpeg-devel] [PATCH 2/2] fate: disable automatic conversions on most filter tests.

2020-08-15 Thread Nicolas George
Mark Thompson (12020-08-14): > Adding this variable to the command line of large numbers of tests is > not very fun, because it's not relevant to the test definition itself. > > Is it possible that it might be better to split some of the commands > into multiple forms if there are some tests which

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

2020-08-15 Thread Nicolas George
Mark Thompson (12020-08-14): > The relation to the existing -autorotate and -autoscale options is > confusing - those aren't disabled by this (because they are applied > differently), but it sounds like they would be. I updated the doc to: Disable automatically inserting format conversion filters

[FFmpeg-devel] [PATCH v2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-15 Thread lance . lmwang
From: Limin Wang Signed-off-by: Limin Wang --- configure | 2 +- libavformat/mpegts.h| 16 ++ libavformat/mpegtsenc.c | 84 +++-- 3 files changed, 99 insertions(+), 3 deletions(-) diff --git a/configure b/configure index d4

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mpegtsenc: support DVB 6A descriptor for AC-3

2020-08-15 Thread lance . lmwang
On Fri, Aug 14, 2020 at 10:29:45PM +0200, Marton Balint wrote: > > > On Fri, 14 Aug 2020, lance.lmw...@gmail.com wrote: > > > From: Limin Wang > > > > Signed-off-by: Limin Wang > > --- > > Sorry, fix for the patch order for fate test > > > > libavformat/mpegts.h| 16 +++ > > libav

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Paul B Mahol
On 8/15/20, Alexander Strasser wrote: > Hi Derek, > hi all! > > On 2020-08-14 20:22 +0100, Derek Buitenhuis wrote: >> On 14/08/2020 20:13, Paul B Mahol wrote: >> >> Resending because I accidentally replied to James instead of the list. >> >> Woops. >> >> >> >> I guess it was not clear to me this i

Re: [FFmpeg-devel] [PATCH] avcodec/dpx: Read alternative frame rate from television header

2020-08-15 Thread Michael Niedermayer
On Fri, Aug 14, 2020 at 01:30:24AM -0700, Harry Mallon wrote: > Signed-off-by: Harry Mallon > --- > libavcodec/dpx.c | 14 +- > 1 file changed, 13 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/dpx.c b/libavcodec/dpx.c > index b1833ed9ef..694deb27c5 100644 > --- a/libavcode

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-15 Thread Michael Niedermayer
On Fri, Aug 14, 2020 at 08:20:28PM +0200, Andreas Rheinhardt wrote: > Michael Niedermayer: > > On Thu, Aug 13, 2020 at 12:41:26PM +0100, Jack Haughton wrote: > >> Commit a500b975 removed NULL input handling from this function, > >> moving the check higher up the call tree in one branch. However, >

Re: [FFmpeg-devel] [PATCH] swresample/rematrix: handle 22.2 as a 9 channel layout

2020-08-15 Thread Jan Ekström
On Fri, Aug 14, 2020 at 8:36 PM Michael Niedermayer wrote: > > On Thu, Aug 13, 2020 at 01:26:58AM +0300, Jan Ekström wrote: > > This is as far as 22.2 follows the same channel order as > > WaveFormatExtensible's channel mask (and the AV_CH_* defines). > > > > After LFE2 the side channels would fol

Re: [FFmpeg-devel] [PATCH] swresample/rematrix: handle 22.2 as a 9 channel layout

2020-08-15 Thread Jan Ekström
On Sat, Aug 15, 2020 at 12:29 PM Jan Ekström wrote: > > On Fri, Aug 14, 2020 at 8:36 PM Michael Niedermayer > wrote: > > > > On Thu, Aug 13, 2020 at 01:26:58AM +0300, Jan Ekström wrote: > > > This is as far as 22.2 follows the same channel order as > > > WaveFormatExtensible's channel mask (and t

Re: [FFmpeg-devel] [PATCH 3/5] lavc/libkvazaar: export encoded frame stats

2020-08-15 Thread myp...@gmail.com
On Sun, Aug 9, 2020 at 6:07 AM Mark Thompson wrote: > > On 26/07/2020 13:26, Jun Zhao wrote: > > From: Jun Zhao > > > > Export choosen pict_type and qp. > > > > Signed-off-by: Jun Zhao > > --- > > libavcodec/libkvazaar.c | 30 ++ > > 1 file changed, 30 insertions(+

Re: [FFmpeg-devel] [PATCH] avutil/opt: Restore NULL input handling to set_string_video_rate()

2020-08-15 Thread Michael Niedermayer
On Thu, Aug 06, 2020 at 02:43:08PM +0200, Andreas Rheinhardt wrote: > Jack Haughton: > > Commit a500b975 removed NULL input handling from this function, > > moving the check higher up the call tree in one branch. However, > > there is another call to set_string_video_rate() which may pass > > NULL,

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

2020-08-15 Thread Alexander Strasser
Hi Derek, hi all! On 2020-08-14 20:22 +0100, Derek Buitenhuis wrote: > On 14/08/2020 20:13, Paul B Mahol wrote: > >> Resending because I accidentally replied to James instead of the list. > >> Woops. > >> > >> I guess it was not clear to me this is not the initial thread, since it is > >> not > >>

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose().

2020-08-15 Thread Alexander Strasser
On 2020-08-13 00:13 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-12): > > Stupid question: Why do we transform relative URLs at all? > > > > Isn't it normally supposed to work for HTTP on the server-side too? > > > > Many clients seem to do it. Just curious why... > > At the very lea

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

2020-08-15 Thread Alexander Strasser
On 2020-08-14 11:34 +0200, Jean-Baptiste Kempf wrote: > On Wed, 12 Aug 2020, at 14:38, Alexander Strasser wrote: > > On 2020-08-12 12:32 +0200, Jean-Baptiste Kempf wrote: > > > On Wed, 12 Aug 2020, at 00:29, Alexander Strasser wrote: > > > > Definitions of non-obvious data should have a short c