[FFmpeg-devel] [PATCH v4 2/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Jason Stevens
Add missing dnxhr mxf container essence ULs to the mxf encoder. set proper mxf frame size for dnxhr using libavformat/dnxhd's get dnxhr size function. This fixes dnxhr mxf files being quarantined by Avid Media Composer. Signed-off-by: Jason Stevens --- libavformat/mxfenc.c | 53

[FFmpeg-devel] [PATCH v4 1/2] libavcodec/dnxhd: change ff_dnxhd_get_hr_frame_size to avpriv_

2018-09-07 Thread Jason Stevens
refactor ff_dnxhd_get_hr_frame_size to avpriv_dnxhd_get_hr_frame_size, to allow cross library usage in libavformat/mxfenc this change makes this function no longer be always inlined. Signed-off-by: Jason Stevens --- libavcodec/dnxhd_parser.c | 2 +- libavcodec/dnxhddata.c| 13

[FFmpeg-devel] [PATCH v4 0/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Jason Stevens
version 4 of this patch set changes ff_dnxhd_get_hr_frame_size to not be inlined, the same way as avpriv_dnxhd_get_frame_size. Jason Stevens (2): libavcodec/dnxhd: change ff_dnxhd_get_hr_frame_size to avpriv_ libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

Re: [FFmpeg-devel] [PATCH v3 1/2] libavcodec/dnxhd: change ff_dnxhd_get_hr_frame_size to avpriv_

2018-09-07 Thread Michael Niedermayer
On Fri, Sep 07, 2018 at 01:03:12AM -0700, Jason Stevens wrote: > change ff_dnxhd_get_hr_frame_size to avpriv_dnxhd_get_hr_frame_size, to allow > cross library usage in libavformat/mxfenc > > Signed-off-by: Jason Stevens > --- > libavcodec/dnxhd_parser.c | 2 +- > libavcodec/dnxhddata.h| 2

Re: [FFmpeg-devel] Reduce static table size for VLC tables in h264_cavlc.c

2018-09-07 Thread Carl Eugen Hoyos
2018-09-08 1:50 GMT+02:00, Dale Curtis : > These tables represent ~70k so moving the allocation to when > they're actually used reduces memory usage in cases where the > h264 decoder isn't used. Is there a performance penalty? Carl Eugen ___

[FFmpeg-devel] Reduce static table size for VLC tables in h264_cavlc.c

2018-09-07 Thread Dale Curtis
These tables represent ~70k so moving the allocation to when they're actually used reduces memory usage in cases where the h264 decoder isn't used. From e1cbe52a1f41a39698136efb4695d8d019117853 Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Fri, 31 Aug 2018 16:50:23 -0700 Subject: [PATCH]

[FFmpeg-devel] Don't calculate duration using AV_NOPTS_VALUE for start_time.

2018-09-07 Thread Dale Curtis
Found by ClusterFuzz, https://crbug.com/879852 From 68614e9a099ee4ae754da5fa36fbb6a570f4aa73 Mon Sep 17 00:00:00 2001 From: Dale Curtis Date: Fri, 7 Sep 2018 15:37:09 -0700 Subject: [PATCH] Don't calculate duration using AV_NOPTS_VALUE for start_time. Found by ClusterFuzz,

[FFmpeg-devel] [PATCH] libavcodec/libaomenc.c: Added code for computing PSNR/SSIM for libaom encoder.

2018-09-07 Thread Sam John
--- libavcodec/libaomenc.c | 117 + 1 file changed, 96 insertions(+), 21 deletions(-) diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c index 9431179886..e62057177d 100644 --- a/libavcodec/libaomenc.c +++ b/libavcodec/libaomenc.c @@ -50,6 +50,9

Re: [FFmpeg-devel] segment: Create missing directories

2018-09-07 Thread Marton Balint
On Thu, 6 Sep 2018, Julien Gaulmin wrote: I find this patch very useful. Here is a version against master as recommended by Ricardo. Works well for me. Thanks Georgi. Le 20/02/2018 à 12:17, Georgi Chorbadzhiyski a écrit : The attached patch allows segment muxer to be used for file

Re: [FFmpeg-devel] segment: Create missing directories

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 05:02:28PM +0200, Julien Gaulmin wrote: > I find this patch very useful. Here is a version against master as > recommended by Ricardo. > > Works well for me. Thanks Georgi. > > Le 20/02/2018 à 12:17, Georgi Chorbadzhiyski a écrit : > > The attached patch allows segment

[FFmpeg-devel] [PATCH 5/5] decklink: Add support for output of Active Format Description (AFD)

2018-09-07 Thread Devin Heitmueller
Implement support for including AFD in decklink output when putting out 10-bit VANC data. Updated to reflect feedback from Marton Balint , Carl Eugen Hoyos and Aaron Levinson . Signed-off-by: Devin Heitmueller --- libavdevice/decklink_enc.cpp | 54 ++--

[FFmpeg-devel] [PATCH 4/5] v210enc: Pass through Active Format Description (AFD) data

2018-09-07 Thread Devin Heitmueller
When encoding to V210, make sure the AFD side data makes it through in the resulting AVPacket. This is needed so the decklink output module can put out AFD when in 10-bit mode. Signed-off-by: Devin Heitmueller --- libavcodec/v210enc.c | 8 1 file changed, 8 insertions(+) diff --git

[FFmpeg-devel] [PATCH 2/5] libavdevice/decklink: Add support for EIA-708 output over SDI

2018-09-07 Thread Devin Heitmueller
Hook in libklvanc and use it for output of EIA-708 captions over SDI. The bulk of this patch is just general support for ancillary data for the Decklink SDI module - the real work for construction of the EIA-708 CDP and VANC line construction is done by libklvanc. Libklvanc can be found at:

[FFmpeg-devel] [PATCH 3/5] Allow AFD data to be embedded in AVPacket

2018-09-07 Thread Devin Heitmueller
Create a new AVPacket side data type for Active Format Description, which mirrors the side data type found in AVFrame. The primary use case for this is ensuring AFD gets preserved in the V210 encoder, so that the decklink libavdevice can output AFD. Signed-off-by: Devin Heitmueller ---

[FFmpeg-devel] [PATCH 0/5] Support for Decklink output of EIA-708 and AFD

2018-09-07 Thread Devin Heitmueller
The following patches add support for output of 708 and AFD over the Decklink SDI interface. This series is a subset of a series submitted in early January, with the hope of getting the less controversial parts merged upstream. Note compared to the previous series this includes a bit of

[FFmpeg-devel] [PATCH 1/5] v210enc: Pass through A53 CC data

2018-09-07 Thread Devin Heitmueller
When encoding to V210, make sure the CC side data makes it through in the resulting AVPacket. This is needed so the decklink output module can put out captions when in 10-bit mode. Signed-off-by: Devin Heitmueller --- libavcodec/v210enc.c | 9 + 1 file changed, 9 insertions(+) diff

Re: [FFmpeg-devel] [PATCH] avcodec/aacenc: report channel layout by name

2018-09-07 Thread Moritz Barsnick
On Fri, Jul 06, 2018 at 23:49:50 +0200, Moritz Barsnick wrote: > Possibly useful in the error case. Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avformat/rtpenc_mpegts: copy metadata to mpegts sub-muxer

2018-09-07 Thread Moritz Barsnick
On Mon, Jul 02, 2018 at 13:38:04 +0200, Moritz Barsnick wrote: > Fixes #7293. Ping. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/h264dec: remove unnecessary checks in h264_decode_frame

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 05:01:49PM +0800, Zhao Zhili wrote: > These conditions are checked again in is_extra(). This patch makes no > functional changes. > --- > libavcodec/h264dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) will apply with patch this depends on thx [...] --

Re: [FFmpeg-devel] [PATCH] avcodec/h264dec: check number of SPS in is_extra

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 05:15:11PM +0800, Zhao Zhili wrote: > --- > libavcodec/h264dec.c | 2 ++ > 1 file changed, 2 insertions(+) will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/mips: [loongson] fix improper use of register constraints.

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 04:10:52PM +0800, Shiyou Yin wrote: > Constraint "g" means compiler can store variable in memory or register. > When we use constraint "g" for a variable and this variable was operated by > instruction which only support register operands may lead "invalid operands" >

Re: [FFmpeg-devel] [PATCH] avfilter: add bm3d filter

2018-09-07 Thread Paul B Mahol
On 5/13/18, Paul B Mahol wrote: > On 5/13/18, Michael Niedermayer wrote: >> On Sat, May 12, 2018 at 09:04:44AM +0200, Paul B Mahol wrote: >>> On 5/12/18, Michael Niedermayer wrote: >>> > On Fri, May 11, 2018 at 04:03:07PM +0200, Paul B Mahol wrote: >>> >> Signed-off-by: Paul B Mahol >>> >> ---

Re: [FFmpeg-devel] segment: Create missing directories

2018-09-07 Thread Julien Gaulmin
I find this patch very useful. Here is a version against master as recommended by Ricardo. Works well for me. Thanks Georgi. Le 20/02/2018 à 12:17, Georgi Chorbadzhiyski a écrit : > The attached patch allows segment muxer to be used for file archiving by > allowing it to automatically create the

Re: [FFmpeg-devel] [PATCH] avfilter/vf_interlace: fix numerical options

2018-09-07 Thread Thomas Mundt
Am Do., 30. Aug. 2018 um 10:37 Uhr schrieb Thomas Mundt : > Am Mo., 27. Aug. 2018 um 20:40 Uhr schrieb Thomas Mundt < > tmund...@gmail.com>: > >> Am Mo., 27. Aug. 2018 um 19:40 Uhr schrieb Paul B Mahol > >: >> >>> On 8/27/18, Thomas Mundt wrote: >>> > Am Fr., 24. Aug. 2018 um 15:05 Uhr schrieb

Re: [FFmpeg-devel] [PATCH 2/2] Add FATE test for QT format audio descriptors in MP4

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 04:35:44PM +0100, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis > --- > tests/fate/mov.mak | 3 +++ > tests/ref/fate/mov-mp4-with-mov-in24-ver | 3 +++ > 2 files changed, 6 insertions(+) > create mode 100644

Re: [FFmpeg-devel] [PATCH 3/4] lavf/movenc: Fail when codec tag is invalid for format

2018-09-07 Thread John Stebbins
On 09/07/2018 08:16 AM, James Almer wrote: > On 9/7/2018 12:09 PM, John Stebbins wrote: >> On 09/06/2018 03:40 PM, James Almer wrote: >>> On 8/27/2018 6:07 PM, John Stebbins wrote: On 08/27/2018 02:03 PM, James Almer wrote: > On 8/27/2018 5:48 PM, John Stebbins wrote: >> On 08/27/2018

Re: [FFmpeg-devel] [PATCH 2/2] Add FATE test for QT format audio descriptors in MP4

2018-09-07 Thread Derek Buitenhuis
On 06/09/2018 16:35, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis > --- > tests/fate/mov.mak | 3 +++ > tests/ref/fate/mov-mp4-with-mov-in24-ver | 3 +++ > 2 files changed, 6 insertions(+) > create mode 100644 tests/ref/fate/mov-mp4-with-mov-in24-ver

Re: [FFmpeg-devel] [PATCH 3/4] lavf/movenc: Fail when codec tag is invalid for format

2018-09-07 Thread James Almer
On 9/7/2018 12:09 PM, John Stebbins wrote: > On 09/06/2018 03:40 PM, James Almer wrote: >> On 8/27/2018 6:07 PM, John Stebbins wrote: >>> On 08/27/2018 02:03 PM, James Almer wrote: On 8/27/2018 5:48 PM, John Stebbins wrote: > On 08/27/2018 01:29 PM, James Almer wrote: >> On 8/27/2018

Re: [FFmpeg-devel] [PATCH 3/4] lavf/movenc: Fail when codec tag is invalid for format

2018-09-07 Thread John Stebbins
On 09/06/2018 03:40 PM, James Almer wrote: > On 8/27/2018 6:07 PM, John Stebbins wrote: >> On 08/27/2018 02:03 PM, James Almer wrote: >>> On 8/27/2018 5:48 PM, John Stebbins wrote: On 08/27/2018 01:29 PM, James Almer wrote: > On 8/27/2018 4:57 PM, John Stebbins wrote: >> Fixes ticket

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: Parse encryption init info from streams.

2018-09-07 Thread Carl Eugen Hoyos
2018-09-06 22:43 GMT+02:00, Jacob Trimble : [...] Given the number of (non-trivial!) commits you have in FFmpeg, I suggest you ask Michael for commit rights (and send him your public key), you can then threaten to apply the patch yourself. Carl Eugen

Re: [FFmpeg-devel] [PATCH v3 2/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Carl Eugen Hoyos
2018-09-07 10:54 GMT+02:00, Tomas Härdin : > fre 2018-09-07 klockan 01:03 -0700 skrev Jason Stevens: >> Add missing dnxhr mxf container essence ULs to the mxf encoder. >> set proper mxf frame size for dnxhr using libavformat/dnxhd's get dnxhr >> size function. >> >> This fixes dnxhr mxf files

Re: [FFmpeg-devel] [PATCH v3 2/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread jay
I've not seen a smpte spec that contains the DNXHR mxf ULs. The mxf UL definitions can be looked up at registry.smpte-ra.org search for "VC-3", CIDs 1270-1274 are DNXHR. DNXHR in already supported in mov and mkv. -Jason -Original Message- From: "Tomas Härdin" Sent: Friday, September 7,

Re: [FFmpeg-devel] [PATCH] avfilter: add lut1d filter

2018-09-07 Thread Paul B Mahol
On 8/23/18, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 31 +++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_lut3d.c | 438 +++ > 4 files changed, 471 insertions(+) >

Re: [FFmpeg-devel] [PATCH 2/2] avutil/file: allow mapping 0 byte files with av_file_map

2018-09-07 Thread Michael Niedermayer
On Thu, Sep 06, 2018 at 08:58:37PM +0200, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavutil/file.c | 8 > libavutil/file.h | 2 ++ > 2 files changed, 10 insertions(+) probably ok thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH v3 2/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Tomas Härdin
fre 2018-09-07 klockan 01:03 -0700 skrev Jason Stevens: > Add missing dnxhr mxf container essence ULs to the mxf encoder. > set proper mxf frame size for dnxhr using libavformat/dnxhd's get dnxhr size > function. > > This fixes dnxhr mxf files being quarantined by Avid Media Composer. > > >

[FFmpeg-devel] [PATCH v3 1/2] libavcodec/dnxhd: change ff_dnxhd_get_hr_frame_size to avpriv_

2018-09-07 Thread Jason Stevens
change ff_dnxhd_get_hr_frame_size to avpriv_dnxhd_get_hr_frame_size, to allow cross library usage in libavformat/mxfenc Signed-off-by: Jason Stevens --- libavcodec/dnxhd_parser.c | 2 +- libavcodec/dnxhddata.h| 2 +- libavcodec/dnxhdenc.c | 2 +- 3 files changed, 3 insertions(+), 3

[FFmpeg-devel] [PATCH v3 2/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Jason Stevens
Add missing dnxhr mxf container essence ULs to the mxf encoder. set proper mxf frame size for dnxhr using libavformat/dnxhd's get dnxhr size function. This fixes dnxhr mxf files being quarantined by Avid Media Composer. Signed-off-by: Jason Stevens --- libavformat/mxfenc.c | 53

[FFmpeg-devel] [PATCH v3 0/2] libavformat/mxfenc: add missing dnxhr mxfcontainer essence ULs

2018-09-07 Thread Jason Stevens
v3 of this patch now uses the libavcodec/dnxhd's get dnxhr frame size function. change ff_dnxhd_get_hr_frame_size to avpriv_dnxhd_get_hr_frame_size, to allow cross library usage in libavformat/mxfenc Add missing dnxhr mxf container essence ULs to the mxf encoder. set proper mxf frame size for