Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-30 Thread Gyan Doshi
On 31-05-2018 02:00 AM, Lou Logan wrote: On Tue, May 29, 2018, at 9:43 PM, Gyan Doshi wrote: Works when "--init-file=doc/t2h.pm" is omitted. Didn't investigate further. make uses the init file here. makeinfo is texi2any 6.3 makeinfo --html -I doc --no-split -D config-not-all

Re: [FFmpeg-devel] [RFC] libfdk_aac license

2018-05-30 Thread Gyan Doshi
"The Fraunhofer AAC library is licensed under a license incompatible to  the GPL. Therefore, for GPL builds, you have to pass  @code{--enable-nonfree} to configure to use it. To the best of our  knowledge, it is compatible with the LGPL" ? Will push in a day with above changes. Gyan

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-30 Thread Gyan Doshi
On 31-05-2018 04:35 AM, Michael Niedermayer wrote: If a user consults the relevant parts of the documentation before execution, they should be able to predict how the program will behave. If they do it afterwards, they should understand what the program did. Even though FFmpeg is an open

Re: [FFmpeg-devel] [PATCH 1/2] avformat/flvdec: add flv_ignore_prevtag option into flvdec

2018-05-30 Thread Steven Liu
> On May 31, 2018, at 12:40, Steven Liu wrote: > > dump the problem flv to problem_flv.flv. > liuqideMacBook-Pro:xxx liuqi$ dd if=~/Movies/Test/1.flv of=problem_flv.flv > count=81920 bs=1 > 81920+0 records in > 81920+0 records out > 81920 bytes transferred in 0.767041 secs (106800 bytes/sec)

[FFmpeg-devel] [PATCH 1/2] avformat/flvdec: add flv_ignore_prevtag option into flvdec

2018-05-30 Thread Steven Liu
dump the problem flv to problem_flv.flv. liuqideMacBook-Pro:xxx liuqi$ dd if=~/Movies/Test/1.flv of=problem_flv.flv count=81920 bs=1 81920+0 records in 81920+0 records out 81920 bytes transferred in 0.767041 secs (106800 bytes/sec) before this patch: liuqideMacBook-Pro:xxx liuqi$ ./ffplay

[FFmpeg-devel] [PATCH] avformat/flvdec: add flv_full_metadata option into flvdec

2018-05-30 Thread Steven Liu
output all the metadata context when use this option. Signed-off-by: Steven Liu --- doc/demuxers.texi| 3 +++ libavformat/flvdec.c | 6 -- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index 5007e6339f..aad94eb50e 100644 ---

[FFmpeg-devel] [PATCH] avformat/hlsenc: reimplement randomize of hls use av_get_random_seed

2018-05-30 Thread Steven Liu
for support use the mbedtls Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 22 +++--- 1 file changed, 7 insertions(+), 15 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 9c06551ea6..17b464f540 100644 --- a/libavformat/hlsenc.c +++

[FFmpeg-devel] [PATCH 2/2] avformat/flvdec: reindent code for previous commit

2018-05-30 Thread Steven Liu
Signed-off-by: Steven Liu --- libavformat/flvdec.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/libavformat/flvdec.c b/libavformat/flvdec.c index 861cf7c020..f65542711e 100644 --- a/libavformat/flvdec.c +++ b/libavformat/flvdec.c @@ -1254,19

Re: [FFmpeg-devel] [PATCH 5/6] avcodec/avcodec.h: add AV_CODEC_ID_TIMED_TEXT_MARKUP

2018-05-30 Thread Rostislav Pehlivanov
On 31 May 2018 at 01:05, Marton Balint wrote: > Signed-off-by: Marton Balint > --- > libavcodec/avcodec.h| 1 + > libavcodec/codec_desc.c | 8 > libavcodec/version.h| 2 +- > 3 files changed, 10 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/avcodec.h

Re: [FFmpeg-devel] Patch: Replace quotes for inline asm detection.

2018-05-30 Thread James Almer
On 5/30/2018 10:32 PM, Michael Niedermayer wrote: > On Wed, May 30, 2018 at 09:48:51AM -0700, Frank Liberato wrote: >> Please find attached a one line patch: >> >> >>> Commit 8c893aa3cd5 removed quotes that were required to detect >>> inline asm in clank: >>> >>> check_insn armv5te qadd

Re: [FFmpeg-devel] Patch: Replace quotes for inline asm detection.

2018-05-30 Thread Michael Niedermayer
On Wed, May 30, 2018 at 09:48:51AM -0700, Frank Liberato wrote: > Please find attached a one line patch: > > > > Commit 8c893aa3cd5 removed quotes that were required to detect > > inline asm in clank: > > > > check_insn armv5te qadd r0, r0, r0 > > .../test.c:1:34: error: expected

[FFmpeg-devel] [PATCH 3/6] avformat/mxfdec: only disallow seek on metadata streams

2018-05-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 3d8c8e204c..e80ef62d57 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -3447,7 +3447,7 @@ static int

[FFmpeg-devel] [PATCH 4/6] avformat/mxfdec: remove check for NULL MXFTrack in mxf_set_pts

2018-05-30 Thread Marton Balint
It cannot happen for video streams. Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index e80ef62d57..cf1cd71987 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@

[FFmpeg-devel] [PATCH 6/6] avformat/mxfdec: add support for recognizing timed text streams

2018-05-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 8 1 file changed, 8 insertions(+) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index cf1cd71987..d9ce09cc75 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -1280,6 +1280,7 @@ static const MXFCodecUL

[FFmpeg-devel] [PATCH 5/6] avcodec/avcodec.h: add AV_CODEC_ID_TIMED_TEXT_MARKUP

2018-05-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 8 libavcodec/version.h| 2 +- 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index fb0c6fae70..91ccef538e 100644 ---

[FFmpeg-devel] [PATCH 2/6] avformat/mxfdec: recognize SMPTE 436 VBI data

2018-05-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 1099616174..3d8c8e204c 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -1278,6 +1278,7 @@ static const MXFCodecUL

[FFmpeg-devel] [PATCH 1/6] avformat/mxfdec: use MXFCodecUL struct to store essence description for data_essence_container_uls

2018-05-30 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxf.h| 1 + libavformat/mxfdec.c | 16 2 files changed, 5 insertions(+), 12 deletions(-) diff --git a/libavformat/mxf.h b/libavformat/mxf.h index ffcc429a8b..19f8d8a9f5 100644 --- a/libavformat/mxf.h +++ b/libavformat/mxf.h @@

[FFmpeg-devel] [PATCH]lavc/profiles: Mention AAC-LC only once

2018-05-30 Thread Carl Eugen Hoyos
Hi! I noticed that AAC-LC is listed twice in ff_aac_profiles[] - is there a reason? And why do some codecs call a profile "codec-main" and some just "main"? Please comment, Carl Eugen From e34da28a84796513ed7690c1e51fbab7f740f832 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Thu, 31 May

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-30 Thread Michael Niedermayer
On Wed, May 30, 2018 at 10:59:22AM +0530, Gyan Doshi wrote: > > > On 30-05-2018 04:57 AM, Carl Eugen Hoyos wrote: > >2018-05-27 6:16 GMT+02:00, Gyan Doshi : > > > >>v2 attached. > > > >>+In the absence of any map options for a particular output file, ffmpeg > >>inspects the output > >>+format

Re: [FFmpeg-devel] [PATCH 2/2] qt-faststart - optimize the offset change loop

2018-05-30 Thread Michael Niedermayer
On Wed, May 30, 2018 at 05:39:50AM +, Eran Kornblau wrote: > > > > > > -Original Message- > > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > > Michael Niedermayer > > Sent: Wednesday, May 30, 2018 12:37 AM > > To: FFmpeg development discussions and

Re: [FFmpeg-devel] [PATCH v3 2/3] lavc, doc, configure: add libdavs2 video decoder

2018-05-30 Thread Mark Thompson
On 30/05/18 02:19, hwren wrote: > Add avs2 video decoder via libdavs2 library. > > Signed-off-by: hwren > --- > Changelog | 1 + > configure | 4 + > doc/decoders.texi | 10 +++ > doc/general.texi | 8 ++ > libavcodec/Makefile| 1 + >

Re: [FFmpeg-devel] [PATCH v3 1/3] lavc,doc: add avs2 codec

2018-05-30 Thread Jan Ekström
On Wed, May 30, 2018 at 4:19 AM, hwren wrote: > Signed-off-by: hwren > --- > doc/APIchanges | 3 +++ > libavcodec/avcodec.h| 1 + > libavcodec/codec_desc.c | 7 +++ > libavcodec/version.h| 4 ++-- > 4 files changed, 13 insertions(+), 2 deletions(-) > > diff --git

Re: [FFmpeg-devel] [PATCH] mxfdec: fix NULL pointer dereference in mxf_read_packet_old

2018-05-30 Thread Marton Balint
On Thu, 17 Nov 2016, Andreas Cadhalpun wrote: Metadata streams have priv_data set to NULL. Signed-off-by: Andreas Cadhalpun --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index a1a79ce..2ad0c28 100644

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mxfdec: recognize SMPTE 436 VBI data

2018-05-30 Thread Marton Balint
On Wed, 30 May 2018, Tomas Härdin wrote: mån 2018-05-28 klockan 00:27 +0200 skrev Marton Balint: On Sun, 27 May 2018, Tomas Härdin wrote: > sön 2018-05-27 klockan 21:21 +0200 skrev Marton Balint: > > > Signed-off-by: Marton Balint > > > > --- > >  libavformat/mxfdec.c | 5 - > >  1

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-30 Thread Lou Logan
On Tue, May 29, 2018, at 9:43 PM, Gyan Doshi wrote: > > They do here. Which version of makeinfo do you have? 6.5 Works when "--init-file=doc/t2h.pm" is omitted. Didn't investigate further. > More importantly, which version is on the project server? 5.2

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-30 Thread Jonathan Morley
Thank you. I will add that and get you a new patch for review. > On May 30, 2018, at 11:04 AM, Marton Balint wrote: > > > > On Tue, 29 May 2018, Jonathan Morley wrote: > >> Thank you Marton, That makes sense to me, but can you please clarify which >> context is the most appropriate to use?

Re: [FFmpeg-devel] [PATCH] Limited timecode support for lavd/decklink

2018-05-30 Thread Marton Balint
On Tue, 29 May 2018, Jonathan Morley wrote: Thank you Marton, That makes sense to me, but can you please clarify which context is the most appropriate to use? Just put it in decklink_ctx in the Status section. Don't forget to free it in ff_decklink_read_close. Regards, Marton

Re: [FFmpeg-devel] [PATCH] tools/crypto_bench: add support for mbedcrypto

2018-05-30 Thread James Almer
On 5/27/2018 6:25 PM, James Almer wrote: > Requires mbed TLS 2.7.0 or newer > > Signed-off-by: James Almer > --- > tools/crypto_bench.c | 111 +-- > 1 file changed, 108 insertions(+), 3 deletions(-) Pushed alongside a couple extra additions.

[FFmpeg-devel] Patch: Replace quotes for inline asm detection.

2018-05-30 Thread Frank Liberato
Please find attached a one line patch: > Commit 8c893aa3cd5 removed quotes that were required to detect > inline asm in clank: > > check_insn armv5te qadd r0, r0, r0 > .../test.c:1:34: error: expected string literal in 'asm' > void foo(void){ __asm__ volatile(qadd r0, r0, r0); }

Re: [FFmpeg-devel] [PATCH v2] avformat/hls: Properly expose intercepted ID3 tags to the API.

2018-05-30 Thread Richard Shaffer
On Fri, May 18, 2018 at 4:09 PM, wrote: > From: Richard Shaffer > > The HLS demuxer will process any ID3 tags at the beginning of a segment in > order to obtain timestamp data. However, when this data was parsed on a > second > or subsequent segment, the updated metadata would be discarded.

[FFmpeg-devel] [PATCH] configure fix arm inline defines

2018-05-30 Thread John Cox
Hi I believe there is a bug in the arm feature detection for inline asm in configure and I have a patch for it. Currently using a command line like: ./configure --enable-cross-compile --arch=arm --cpu=cortex-a7 --target-os=linux --cross-prefix=arm-linux-gnueabihf- gives in config.h: #define

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/idctdsp: Transmit studio_profile to init instead of using AVCodecContext profile

2018-05-30 Thread Michael Niedermayer
On Mon, May 28, 2018 at 10:29:56PM +0200, Michael Niedermayer wrote: > These 2 fields are not always the same, it is simpler to always use the same > field > for detecting studio profile > > Fixes: null pointer dereference > Fixes: ffmpeg_crash_3.avi > > Found-by: Thuan Pham , Marcel Böhme,

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Set st->start_time for video streams explicitly.

2018-05-30 Thread Derek Buitenhuis
On Tue, May 29, 2018 at 11:39 PM, Sasi Inguva wrote: > If start_time is not set, ffmpeg takes the duration from the global > movie instead of the per stream duration. > Signed-off-by: Sasi Inguva Probably OK, though it seems your git client didn't have your email configured. - Derek

Re: [FFmpeg-devel] [GSOC] [PATCH] DNN module introduction and SRCNN filter update

2018-05-30 Thread Pedro Arthur
Please, send patch 0002 in a new email and discuss it there. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-05-30 Thread Vasile Toncu
Hello, I've sent a wrong version in the previous email for patch 3. Please ignore. This is the corect one. Have a good day, Vasile Toncu >From 5898c531066f8e6c034eb1c882000f60ba4f0cb9 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Tue, 29 May 2018 18:29:06 +0300 Subject: [PATCH]

Re: [FFmpeg-devel] [PATCH v2] avcodec/vc1: fix overlap smoothing filter for P frames

2018-05-30 Thread Carl Eugen Hoyos
2018-05-30 13:53 GMT+02:00, Jerome Borsboom : > The v_overlap_filter needs to run on the colocated block of the previous > macroblock. For the luma plane, the colocated block is located two blocks > on the left instead of one. In addition, the overlap filter needs to run > on the non-edge blocks

Re: [FFmpeg-devel] [PATCH 2/5] Added reinterlace filter

2018-05-30 Thread Vasile Toncu
Hello, Sorry for the late reply, I've been working on other projects. Here is patch 2, which adds vf_reinterlace filter. To my understanding, patch 2 and patch 3 will be merged at the same time, so there is no need to add documentation for reinterlace as it will replace tinterlace. Thank

Re: [FFmpeg-devel] [PATCH 3/5] Renamed reinterlace to tinterlace

2018-05-30 Thread Vasile Toncu
Hello, Here is patch 3 which renames reinterlace to tinterlace. Fate tests and warnings should be ok now. In patch 4 I will add the new options to the filter. Thank you, Vasile Toncu >From 25d4f0154fe2a8bde3a47491f80ce131ebb20330 Mon Sep 17 00:00:00 2001 From: Vasile Toncu Date: Tue, 29 May

Re: [FFmpeg-devel] [PATCH v2] avcodec/vc1: fix overlap smoothing filter for P frames

2018-05-30 Thread Jerome Borsboom
The v_overlap_filter needs to run on the colocated block of the previous macroblock. For the luma plane, the colocated block is located two blocks on the left instead of one. In addition, the overlap filter needs to run on the non-edge blocks of the first macroblock row and column. Signed-off-by:

Re: [FFmpeg-devel] [PATCH 5/5] avformat/mxfdec: recognize SMPTE 436 VBI data

2018-05-30 Thread Tomas Härdin
mån 2018-05-28 klockan 00:27 +0200 skrev Marton Balint: > > On Sun, 27 May 2018, Tomas Härdin wrote: > > > sön 2018-05-27 klockan 21:21 +0200 skrev Marton Balint: > > > > Signed-off-by: Marton Balint > > > > > > --- > > >  libavformat/mxfdec.c | 5 - > > >  1 file changed, 4 insertions(+),

Re: [FFmpeg-devel] [PATCH 2/2] qt-faststart - optimize the offset change loop

2018-05-30 Thread Tomas Härdin
tis 2018-05-29 klockan 14:36 + skrev Eran Kornblau: > Hi, > > The attached is a slightly more optimized (and IMHO elegant) code for > updating the stco/co64 offsets Looks OK to me. Those & 0xFF are actually superfluous, but the compiler should remove them anyway. /Tomas

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg - rewrite Stream Selection chapter

2018-05-30 Thread Gyan Doshi
On 30-05-2018 04:39 AM, Lou Logan wrote: The subject nit still exists. Fixed locally. Using an itemized list here will look better in my opinion. @itemize @item for video, it is the stream with the highest resolution, @item ... @end itemize Changed. +@subsubheading Example: automatic