[FFmpeg-devel] [PATCH] doc/general_contents.texi: add uavs3d section

2020-10-06 Thread hwrenx
From: hwren Signed-off-by: hwren --- doc/general_contents.texi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/doc/general_contents.texi b/doc/general_contents.texi index 598e0e74da..f441a75ee9 100644 --- a/doc/general_contents.texi +++ b/doc/general_contents.texi @@ -119,6 +119

[FFmpeg-devel] [PATCH v4] avformat/libsrt: print streamid at client

2020-10-06 Thread raghavendra
Print the SRT streamid at the client. Logged to info. Signed-off-by: raghavendra --- libavformat/libsrt.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavformat/libsrt.c b/libavformat/libsrt.c index 4025b24976..eed48c11cf 100644 --- a/libavformat/libsrt.c +++ b/libavformat/libs

Re: [FFmpeg-devel] ?==?utf-8?q? ?==?utf-8?q? [PATCH v4]?==?utf-8?q? avformat/libsrt: print streamid at client

2020-10-06 Thread Raghavendra Rao Sidlagatta
Hello, I believe that the stream id information should be logged to info instead of verbose as it contains important infor mation about the stream. Regards, Raghavendra Rao On Tuesday, October 06, 2020 08:18 BST, raghavendra wrote:  Print the SRT streamid at the client. Logged to info. Sign

Re: [FFmpeg-devel] [PATCH v4] avformat/libsrt: print streamid at client

2020-10-06 Thread Nicolas George
Raghavendra Rao Sidlagatta (12020-10-06): > I believe that the stream id information should be logged to info instead of > verbose as it contains important infor mation about the stream. What can an end-user do with it? Note: your MUA mangled the subject line. Regards, -- Nicolas George ___

Re: [FFmpeg-devel] ?==?utf-8?q? ?==?utf-8?q? [PATCH v4]?==?utf-8?q? avformat/libsrt: print streamid at client

2020-10-06 Thread Raghavendra Rao Sidlagatta
Hello Nicolas, I believe that the end user could retrieve the stream information and this could be handy in debugging as well. Regards, Raghavendra Rao On Tuesday, October 06, 2020 08:38 BST, Nicolas George wrote:  Raghavendra Rao Sidlagatta (12020-10-06): > I believe that the stream id infor

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/apedec: properly calculate and store absolute value

2020-10-06 Thread Anton Khirnov
Quoting Paul B Mahol (2020-10-06 02:17:12) > Signed-off-by: Paul B Mahol > --- > libavcodec/apedec.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c > index 273abe2490..aa4d8fa524 100644 > --- a/libavcodec/apedec.c > +++ b/li

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/apedec: use ff_clz() instead of while loop

2020-10-06 Thread Anton Khirnov
Quoting Paul B Mahol (2020-10-06 02:17:14) > Signed-off-by: Paul B Mahol > --- > libavcodec/apedec.c | 10 +++--- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c > index 8fe7b5ee86..ea36247eb9 100644 > --- a/libavcodec/apedec.c > +

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/apedec: use ff_clz() instead of while loop

2020-10-06 Thread Paul B Mahol
On Tue, Oct 06, 2020 at 10:08:58AM +0200, Anton Khirnov wrote: > Quoting Paul B Mahol (2020-10-06 02:17:14) > > Signed-off-by: Paul B Mahol > > --- > > libavcodec/apedec.c | 10 +++--- > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > diff --git a/libavcodec/apedec.c b/libavcodec/a

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/apedec: properly calculate and store absolute value

2020-10-06 Thread Paul B Mahol
On Tue, Oct 06, 2020 at 09:53:44AM +0200, Anton Khirnov wrote: > Quoting Paul B Mahol (2020-10-06 02:17:12) > > Signed-off-by: Paul B Mahol > > --- > > libavcodec/apedec.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c

Re: [FFmpeg-devel] ?= [PATCH v4]=?utf-8?q? avformat/libsrt: print streamid at client

2020-10-06 Thread Nicolas George
Raghavendra Rao Sidlagatta (12020-10-06): > I believe that the end user could retrieve the stream information How? > and this could be handy in debugging as well. AV_LOG_DEBUG then. > On Tuesday, October 06, 2020 08:38 BST, Nicolas George > wrote: Don't top-post. If you don't know what it me

Re: [FFmpeg-devel] [PATCH 2/4] avcodec/apedec: properly calculate and store absolute value

2020-10-06 Thread Anton Khirnov
Quoting Paul B Mahol (2020-10-06 10:23:13) > On Tue, Oct 06, 2020 at 09:53:44AM +0200, Anton Khirnov wrote: > > Quoting Paul B Mahol (2020-10-06 02:17:12) > > > Signed-off-by: Paul B Mahol > > > --- > > > libavcodec/apedec.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > >

Re: [FFmpeg-devel] [PATCH 4/4] avcodec/apedec: use ff_clz() instead of while loop

2020-10-06 Thread Anton Khirnov
Quoting Paul B Mahol (2020-10-06 10:19:13) > On Tue, Oct 06, 2020 at 10:08:58AM +0200, Anton Khirnov wrote: > > Quoting Paul B Mahol (2020-10-06 02:17:14) > > > Signed-off-by: Paul B Mahol > > > --- > > > libavcodec/apedec.c | 10 +++--- > > > 1 file changed, 3 insertions(+), 7 deletions(-) >

[FFmpeg-devel] [PATCH] Convert logfile to AVIOContext to fix file parser

2020-10-06 Thread Chris Miceli
The lack of AVIOContext means that ffmpeg was attempting to open logfiles for two pass encoding with the "protocol" part of the url on the first pass. This manifested mainly when a user needed to have a colon in the filename as they would require a url style path (according to #8898). --- fftools/

[FFmpeg-devel] [PATCH] avformat/utils: do not fallback to av1dec for probing

2020-10-06 Thread Timo Rothenpieler
--- libavformat/utils.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index a2e701ea1a..871e655e13 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -222,6 +222,10 @@ static const AVCodec *find_probe_decoder(AVFormatContext *s, const

Re: [FFmpeg-devel] [PATCH] avformat/utils: do not fallback to av1dec for probing

2020-10-06 Thread Nicolas George
Timo Rothenpieler (12020-10-06): > --- > libavformat/utils.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/utils.c b/libavformat/utils.c > index a2e701ea1a..871e655e13 100644 > --- a/libavformat/utils.c > +++ b/libavformat/utils.c > @@ -222,6 +222,10 @@ static const AVC

Re: [FFmpeg-devel] [PATCH] avformat/utils: do not fallback to av1dec for probing

2020-10-06 Thread Timo Rothenpieler
On 06.10.2020 15:38, Nicolas George wrote: Timo Rothenpieler (12020-10-06): --- libavformat/utils.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/utils.c b/libavformat/utils.c index a2e701ea1a..871e655e13 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -222

Re: [FFmpeg-devel] autofate system

2020-10-06 Thread Andriy Gelman
Hi, On Tue, 06. Oct 14:16, Chris Miceli wrote: > Hi all, > > I have written a small system called autofate ( > https://github.com/cmiceli/autofate) which is designed to basically run > fate on every commit in master, ideally capturing and then reporting broken > builds. It's written in rust so th

Re: [FFmpeg-devel] autofate system

2020-10-06 Thread Nicolas George
Andriy Gelman (12020-10-06): > I'm not familiar with the setup on fate.ffmpeg.org, but we have something > similar running patchwork.ffmpeg.org > > The site fetches patches from the mailing list. Jenkins server runs make/fate > and reports the results back to patchwork. That means anybody can r

Re: [FFmpeg-devel] [PATCH v2 3/3] avformat/mxfdec: Read Apple private Content Light Level from MXF

2020-10-06 Thread Michael Niedermayer
On Mon, Oct 05, 2020 at 10:18:32AM +0200, Tomas Härdin wrote: > tor 2020-10-01 klockan 22:13 +0200 skrev Michael Niedermayer: > > On Thu, Oct 01, 2020 at 03:29:19PM +0100, Harry Mallon wrote: > > > > On 30 Sep 2020, at 08:32, Michael Niedermayer > > > > wrote: > > > > > > > > fails on big endian

[FFmpeg-devel] [PATCH 3/3] avfilter/vf_minterpolate: Remove redundant code for freeing

2020-10-06 Thread Andreas Rheinhardt
ad73b32d2922f4237405043d19763229aee0e59e added some code for freeing in the input's config_props function, yet this is unnecessary as uninit is called anyway if config_props fails. Signed-off-by: Andreas Rheinhardt --- libavfilter/vf_minterpolate.c | 15 +++ 1 file changed, 3 inserti

[FFmpeg-devel] [PATCH 2/3] avfilter/vf_minterpolate: Fix left shift of negative value

2020-10-06 Thread Andreas Rheinhardt
This has happened when initializing the motion estimation context if width or height of the video was smaller than the block size used for motion estimation and if the motion interpolation mode indicates not to use motion estimation. The solution is of course to only initialize the motion estimati

Re: [FFmpeg-devel] [PATCH] avformat/utils: do not fallback to av1dec for probing

2020-10-06 Thread Timo Rothenpieler
Made obsolete via 214998c55ff99586b57c1ccd34a7e6f15c4fcdb8 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "un

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/ac3tab: rename ff_ac3_sample_rate_tab to avpriv_ac3_sample_rate_tab so that it can be used in libavformat

2020-10-06 Thread Anton Khirnov
Quoting Nicolas George (2020-10-05 12:53:44) > Nachiket Tarate (12020-10-05): > > What is your opinion about Anton Khirnov 's review > > comments ? > > The only way forward is to merge the libraries. The split of shared > objects brings all kinds of issues, including the problem of avpriv > symbo

Re: [FFmpeg-devel] [PATCH 2/2] libavcodec/ac3tab: rename ff_ac3_sample_rate_tab to avpriv_ac3_sample_rate_tab so that it can be used in libavformat

2020-10-06 Thread Nicolas George
Anton Khirnov (12020-10-06): > Would you please stop with spamming this propaganda in unrelated topics? > Merging the libraries is your personal opinion, not "the only way > forward". Nobody, including you, has been able to express one practical benefit of having separate libraries that could not

[FFmpeg-devel] [PATCH] libavcodec/adts_header: add frame_length field and avpriv function to parse AAC ADTS header

2020-10-06 Thread Nachiket Tarate
These will be used by HLS demuxer for SAMPLE-AES decryption. Signed-off-by: Nachiket Tarate --- libavcodec/adts_header.c | 18 ++ libavcodec/adts_header.h | 4 2 files changed, 22 insertions(+) diff --git a/libavcodec/adts_header.c b/libavcodec/adts_header.c index 0889820f

Re: [FFmpeg-devel] [PATCH] Document community process

2020-10-06 Thread Nicolas George
Jean-Baptiste Kempf (12020-10-05): > General Assembly + Main Elections > --- > doc/dev_community/community.md | 60 ++ > 1 file changed, 60 insertions(+) > create mode 100644 doc/dev_community/community.md > > diff --git a/doc/dev_community/community.md b/doc/dev_

Re: [FFmpeg-devel] [PATCH 9/9] avformat/aviobuf: increase default read buffer size to 2*max_buffer_size for streamed data

2020-10-06 Thread Marton Balint
On Wed, 30 Sep 2020, Paul B Mahol wrote: On Tue, Sep 29, 2020 at 11:10:21PM +0200, Marton Balint wrote: This should increase the effectiveness of ffio_ensure_seekback by reducing the number of buffer reallocations and memmoves/memcpys because even a small seekback window requires max_buffer_s

Re: [FFmpeg-devel] [PATCH] libavcodec/adts_header: add frame_length field and avpriv function to parse AAC ADTS header

2020-10-06 Thread James Almer
On 10/6/2020 11:30 AM, Nachiket Tarate wrote: > These will be used by HLS demuxer for SAMPLE-AES decryption. > > Signed-off-by: Nachiket Tarate > --- > libavcodec/adts_header.c | 18 ++ > libavcodec/adts_header.h | 4 > 2 files changed, 22 insertions(+) > > diff --git a/li

[FFmpeg-devel] [PATCH 1/3] avfilter/vf_minterpolate: Reject too small dimensions

2020-10-06 Thread Andreas Rheinhardt
The latter code relies upon the dimensions to be not too small; otherwise one will call av_clip() with min > max lateron which aborts in case ASSERT_LEVEL is >= 2 or one will get a nonsense result that may lead to a heap-buffer-overflow/underflow. The latter has happened in ticket #8248 which this

Re: [FFmpeg-devel] autofate system

2020-10-06 Thread Andriy Gelman
On Tue, 06. Oct 16:07, Nicolas George wrote: > Andriy Gelman (12020-10-06): > > I'm not familiar with the setup on fate.ffmpeg.org, but we have something > > similar running patchwork.ffmpeg.org > > > > The site fetches patches from the mailing list. Jenkins server runs > > make/fate > > and rep

Re: [FFmpeg-devel] [PATCH] libswcale/input: use more accurate planer rgb16 yuv conversions

2020-10-06 Thread Michael Niedermayer
On Sat, Oct 03, 2020 at 04:31:58PM -0700, mindm...@gmail.com wrote: > From: Mark Reid > > These conversion appears to be exhibiting the same rounding error as the > rgbf32 formats where. > I seperated the rounding value from the 16 and 128 offsets, I think it makes > it a little more clear. >

Re: [FFmpeg-devel] [PATCH] libavcodec/adts_header: add frame_length field and avpriv function to parse AAC ADTS header

2020-10-06 Thread Nachiket Tarate
From: ffmpeg-devel on behalf of James Almer Sent: Tuesday, October 6, 2020 8:47 PM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH] libavcodec/adts_header: add frame_length field and avpriv function to parse AAC ADTS header On 10/6/2020 11:30

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: ensure seekback for small probesize

2020-10-06 Thread Andriy Gelman
On Mon, 05. Oct 10:31, Marton Balint wrote: > > > On Sun, 4 Oct 2020, Andriy Gelman wrote: > > > From: Andriy Gelman > > > > get_packet_size() may read upto PROBE_PACKET_MAX_BUF bytes, which may be > > larger than probesize. > > > > Signed-off-by: Andriy Gelman > > --- > > > > An alternativ

[FFmpeg-devel] [PATCH 1/2] swscale/swscale_internal: interpret RGB paletted pixel formats as RGB

2020-10-06 Thread Jan Ekström
This makes isAnyRGB return true for AV_PIX_FMT_PAL8 which is currently the only pixel format with this flag. This lets us have a single query for formats where we need to force range as only full range content is supported. --- libswscale/swscale_internal.h| 2 +- tests/ref/fate/sws-pixdesc-q

[FFmpeg-devel] [PATCH 2/2] swscale: separate exported and internal range flags

2020-10-06 Thread Jan Ekström
Fixes vf_scale outputting RGB AVFrames with limited range in case either input or output specifically sets the range. Keeps the external interfaces the same, but allows us to retrieve and set nonzero value for RGB. Additionally defines the default value of the AVOption as -1 so we can differentiat

Re: [FFmpeg-devel] [PATCH 2/2] swscale: separate exported and internal range flags

2020-10-06 Thread Jan Ekström
On Wed, Oct 7, 2020 at 2:03 AM Jan Ekström wrote: > > Fixes vf_scale outputting RGB AVFrames with limited range in > case either input or output specifically sets the range. > > Keeps the external interfaces the same, but allows us to retrieve > and set nonzero value for RGB. Additionally defines

Re: [FFmpeg-devel] autofate system

2020-10-06 Thread Chris Miceli
Hi Andriy, Thanks for letting me know that there is already something doing this. It was something which I thought would exist but I couldn't see it in my searching of the ffmpeg documentation. Something which would be nice is finding a way for us to tie the results back into the fate system, or