Re: [FFmpeg-devel] Performance of P010LE/BE pixel convertion

2016-08-31 Thread Oliver Collyer
What CPU are you using? It's presumably going to vary wildly from one CPU to another? > On 1 Sep 2016, at 08:52, Ali KIZIL wrote: > > Hi all, > > I tested P010LE pixel convertion from YUV420P in NVENC Main 10 HEVC UHD 50 > fps encoding on Nvidia Pascal Titan X GPU: > > Nvidia Pascal Titan X G

[FFmpeg-devel] Performance of P010LE/BE pixel convertion

2016-08-31 Thread Ali KIZIL
Hi all, I tested P010LE pixel convertion from YUV420P in NVENC Main 10 HEVC UHD 50 fps encoding on Nvidia Pascal Titan X GPU: Nvidia Pascal Titan X GPU can not reach to 50 fps on Main 10 P010LE HEVC encoding: ffmpeg -loglevel verbose -i /media/usb1/4k_sampels/Samsung_SUHD_Picture_Quality\ Demo_N

Re: [FFmpeg-devel] [PATCH v2] Let clang-FORTIFY build; NFC.

2016-08-31 Thread George Burgess
> I'm blindly assuming the signedness isn't relevant here That's actually what matters here, sadly. The only way we've been able to get FORTIFY on clang to work nearly as well as FORTIFY on gcc is with overloading (via clang's __attribute__((overloadable)) ). This attribute is specced to use C++'s

Re: [FFmpeg-devel] FFmpeg in Outreachy Round 13 / 2016-december

2016-08-31 Thread Sarah A Sharp
On Fri, Aug 26, 2016 at 10:06 AM, Stefano Sabatini wrote: > In data Wednesday 2016-08-24 02:07:17 +0200, Michael Niedermayer ha scritto: >> Hi Sarah >> >> On Tue, Aug 23, 2016 at 10:04:55AM -0700, Sarah A Sharp wrote: >> > Hi Michael, >> > >> > I'm happy to hear FFmpeg will participate again! Do y

Re: [FFmpeg-devel] [PATCH] avformat/matroskadec: retain error codes in matroska_resync() and matroska_read_packet()

2016-08-31 Thread Michael Niedermayer
On Mon, Aug 15, 2016 at 10:27:47AM -0700, Sophia Wang wrote: > Of the 3 functions that directly add to matroska->packets, 2 of them will > always return 0 afterwards, and the third (matroska_parse_rm_audio) will > only return something other than 0 in the case of AVERROR(ENOMEM) or > AVERROR(EINVA

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 02:20:27PM +0200, Paul B Mahol wrote: > On 8/31/16, Michael Niedermayer wrote: > > On Wed, Aug 31, 2016 at 10:18:31AM +0200, Paul B Mahol wrote: > >> On 8/30/16, Nicolas George wrote: > >> > Le quartidi 14 fructidor, an CCXXIV, Paul B Mahol a ecrit : > >> >> the filter fra

Re: [FFmpeg-devel] [PATCH 09/11] avformat/mux: Restore stream ts in av_write_packet on EAGAIN

2016-08-31 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 03:24:20PM +0200, [email protected] wrote: > From: Jan Sebechlebsky > > compute_muxer_pkt_fields() stores the last seen timestamps in stream > and produces error if the same timestamp is presented again. > This is a problem if muxer works in non-blocking mode and c

Re: [FFmpeg-devel] [PATCH 3/4] V8 - SCTE-35 support in hlsenc

2016-08-31 Thread Steven Liu
2016-09-01 1:39 GMT+08:00 Carlos Fernandez Sanz : > From: Carlos Fernandez > > Signed-off-by: Carlos Fernandez > --- > libavformat/Makefile | 2 +- > libavformat/hlsenc.c | 110 --- > libavformat/scte_35.c | 527 ++ > > libavformat/sc

Re: [FFmpeg-devel] [PATCH] libavcodec/opus: Add channel mapping 2 to extradata parser

2016-08-31 Thread Michael Niedermayer
On Tue, Aug 30, 2016 at 02:48:07PM -0700, Michael Graczyk wrote: > This allows libavcodec/opus to demux ambisonics in an ogg/opus container. > Channel mapping family 2 is being added in this standards track IETF draft: > tools.ietf.org/html/draft-ietf-codec-ambisonics-00 > --- > libavcodec/opus.c

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Paul B Mahol
On 8/31/16, Nicolas George wrote: > Le quintidi 15 fructidor, an CCXXIV, Paul B Mahol a ecrit : >> Yes, it is very simple. You just need to allocate own buffers that >> would be needed >> by filter. Than you just call ctx->internal->execute()... > > I am sorry, but with the current API, to have so

[FFmpeg-devel] [PATCH] MAINTAINERS: Replace AvxSynth Team by Stephen Hutchinson for avisynth.c

2016-08-31 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 62b818d..044854b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -378,7 +378,7 @@ Muxers/Demuxers: astdec.c Paul B M

Re: [FFmpeg-devel] [PATCH] lavu/timestamp: Avoid C++-unfriendly code in installed header

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 09:01:57PM +0100, Mark Thompson wrote: > Including this header in a C++11 program (inside extern "C") will > throw an error because it looks like a user-defined literal. Add a > space between the two tokens to avoid the problem. > --- > Noted by mcjack in #ffmpeg. LGTM th

Re: [FFmpeg-devel] [PATCH 2/4] V8 - SCTE-35 extraction from mpegts

2016-08-31 Thread Michael Niedermayer
Hi On Wed, Aug 31, 2016 at 10:39:37AM -0700, Carlos Fernandez Sanz wrote: > From: Carlos Fernandez > > Signed-off-by: Carlos Fernandez > --- cus, you are mpegts maintainer, is this ok to apply ? thx [...} -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who

[FFmpeg-devel] [PATCH] lavu/timestamp: Avoid C++-unfriendly code in installed header

2016-08-31 Thread Mark Thompson
Including this header in a C++11 program (inside extern "C") will throw an error because it looks like a user-defined literal. Add a space between the two tokens to avoid the problem. --- Noted by mcjack in #ffmpeg. I think this is the only instance of the problem in all of the installed headers

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/drawutils: P010 is not supported

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 11:33:47AM +0200, Timo Rothenpieler wrote: > --- > libavfilter/drawutils.c | 2 ++ > 1 file changed, 2 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Avoid a single point of failure, be that a person or equipment.

[FFmpeg-devel] [PATCH] Add max value output option to psnr stats log.

2016-08-31 Thread Lucas Cooper
This allows retroactive calculation/aggregation of PSNR from the stats log. --- doc/filters.texi | 10 ++ libavfilter/vf_psnr.c | 20 2 files changed, 30 insertions(+) diff --git a/doc/filters.texi b/doc/filters.texi index 8cef0a0..92a49e6 100644 --- a/doc/filter

Re: [FFmpeg-devel] [PATCH 2/2] configure: fix ldl dependency for new nvenc encoder names

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 04:42:54PM +0200, Timo Rothenpieler wrote: > --- > configure | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index e30ddd2..f2c8492 100755 > --- a/configure > +++ b/configure > @@ -5388,7 +5388,8 @@ decklink_indev_extralib

Re: [FFmpeg-devel] [PATCH 2/2] configure: fix ldl dependency for new nvenc encoder names

2016-08-31 Thread James Almer
On 8/31/2016 11:42 AM, Timo Rothenpieler wrote: > --- > configure | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index e30ddd2..f2c8492 100755 > --- a/configure > +++ b/configure > @@ -5388,7 +5388,8 @@ decklink_indev_extralibs="$decklink_indev_

Re: [FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Timo Rothenpieler
On 8/31/2016 5:42 PM, Carl Eugen Hoyos wrote: > 2016-08-31 17:32 GMT+02:00 James Almer : >> On 8/31/2016 11:58 AM, Carl Eugen Hoyos wrote: >>> 2016-08-31 16:42 GMT+02:00 Timo Rothenpieler : >>> +if enabled x86; then +case $target_os in +mingw32*|mingw64*|win32|win64|linux

Re: [FFmpeg-devel] MKV Header: Writing duration early

2016-08-31 Thread compn
On Tue, 12 Jul 2016 10:15:08 + Soft Works wrote: > anyway, while I haven't looked into that yet. I also haven't > performed research > > about other ffmpeg output formats, to see if there is an existing case > > where duration is written early... > > > Before coding anything I'd like to

[FFmpeg-devel] [PATCH 0/4] V8 - SCTE-35

2016-08-31 Thread Carlos Fernandez Sanz
* Used PCR for PTS and DTS of SCTE packet Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 3 +- libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- libavformat/hlsenc.c

[FFmpeg-devel] [PATCH 3/4] V8 - SCTE-35 support in hlsenc

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 --- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 702 insertions(+), 23 deleti

[FFmpeg-devel] [PATCH 4/4] V8 - Correct Indentation

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 25092f7..a78fdae 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -370,

[FFmpeg-devel] [PATCH 1/4] V8 - Adding SCTE-35 CUI codec

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 3 ++- libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index e2dad5d..00599c4 100644 --- a/libavcodec/avcodec.h +++

[FFmpeg-devel] [PATCH 2/4] V8 - SCTE-35 extraction from mpegts

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 50 +- 1 file changed, 49 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..d000269 100644 --- a/libavformat/mpeg

[FFmpeg-devel] Patches related to bug 4988

2016-08-31 Thread Oliver Collyer
Hi In relation to this bug report from 10 months ago: https://trac.ffmpeg.org/ticket/4988 I’ve been having similar problems capturing video using v4l2 and the Magewell XI100DUSB-HDMI capture device. I’ve attached some logs to the bug report to illustrate t

Re: [FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Carl Eugen Hoyos
2016-08-31 17:32 GMT+02:00 James Almer : > On 8/31/2016 11:58 AM, Carl Eugen Hoyos wrote: >> 2016-08-31 16:42 GMT+02:00 Timo Rothenpieler : >> >>> +if enabled x86; then >>> +case $target_os in >>> +mingw32*|mingw64*|win32|win64|linux|cygwin*) >>> +;; >>> +*) >>> +

[FFmpeg-devel] [PATCH]lavfi/decimate: Do not assume a scene change for the first frame

2016-08-31 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4990 concerning the output file. It is possible that there is nothing to fix though, the input file is "dirty" and "dupthresh" is not underbid by two visually identical frames. Please comment, Carl Eugen From 195df26b7017448d0bfbbe32f4b1c7c6df8ad26c Mon Sep 17 0

Re: [FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread James Almer
On 8/31/2016 11:58 AM, Carl Eugen Hoyos wrote: > Hi! > > 2016-08-31 16:42 GMT+02:00 Timo Rothenpieler : > >> +if enabled x86; then >> +case $target_os in >> +mingw32*|mingw64*|win32|win64|linux|cygwin*) >> +;; >> +*) >> +disable nvenc >> +;;

Re: [FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Carl Eugen Hoyos
Hi! 2016-08-31 16:42 GMT+02:00 Timo Rothenpieler : > +if enabled x86; then > +case $target_os in > +mingw32*|mingw64*|win32|win64|linux|cygwin*) > +;; > +*) > +disable nvenc > +;; > +esac > +else > +disable nvenc > fi > +enabled nv

Re: [FFmpeg-devel] [PATCH 2/2] configure: fix ldl dependency for new nvenc encoder names

2016-08-31 Thread Carl Eugen Hoyos
2016-08-31 16:42 GMT+02:00 Timo Rothenpieler : > --- > configure | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index e30ddd2..f2c8492 100755 > --- a/configure > +++ b/configure > @@ -5388,7 +5388,8 @@ decklink_indev_extralibs="$decklink_indev_ex

[FFmpeg-devel] [PATCH 2/2] configure: fix ldl dependency for new nvenc encoder names

2016-08-31 Thread Timo Rothenpieler
--- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index e30ddd2..f2c8492 100755 --- a/configure +++ b/configure @@ -5388,7 +5388,8 @@ decklink_indev_extralibs="$decklink_indev_extralibs $ldl" frei0r_filter_extralibs='$ldl' frei0r_src_filter

[FFmpeg-devel] [PATCH 1/2] configure: fix nvenc detection logic

2016-08-31 Thread Timo Rothenpieler
--- configure | 34 +++--- 1 file changed, 19 insertions(+), 15 deletions(-) diff --git a/configure b/configure index 52931c3..e30ddd2 100755 --- a/configure +++ b/configure @@ -3205,7 +3205,7 @@ enable audiotoolbox enable d3d11va dxva2 vaapi vda vdpau videotoolbox_hw

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Nicolas George
Le quintidi 15 fructidor, an CCXXIV, Paul B Mahol a écrit : > Yes, it is very simple. You just need to allocate own buffers that > would be needed > by filter. Than you just call ctx->internal->execute()... I am sorry, but with the current API, to have something that really works with threads, not

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Nicolas George
Le quintidi 15 fructidor, an CCXXIV, Michael Niedermayer a écrit : > ./ffmpeg -i tickets//679/oversized_pgs_subtitles.mkv -filter_complex > '[0:s:1]scale=848x480,[0:v]overlay=shortest=1' test.avi > fails assertion: > Assertion progress failed at libavfilter/avfilter.c:1391 > > https://trac.ffmpeg

Re: [FFmpeg-devel] [PATCH 1/2] compat/avisynth: minor update for alpha offsets

2016-08-31 Thread Stephen Hutchinson
On 8/31/2016 9:37 AM, Carl Eugen Hoyos wrote: 2016-08-31 2:26 GMT+02:00 Stephen Hutchinson : -//AVS_CPUF_AVX= 0x800, // Sandy Bridge, Bulldozer + AVS_CPUF_AVX= 0x800, // Sandy Bridge, Bulldozer Is this related to alpha offsets? It's part of the changes to the header u

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread James Almer
On 8/31/2016 5:26 AM, Timo Rothenpieler wrote: >>> +echo 'NV_ENCODE_API_FUNCTION_LIST flist;' >>> +echo 'void f(void) { struct { const GUID guid; } s[] = { { >>> NV_ENC_PRESET_HQ_GUID } }; }' >> >> This will most likely prevent nvenc from being enabled for msvc 2012, but >> not ol

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
> 2016-08-31 15:26 GMT+02:00 Timo Rothenpieler : >>> 2016-08-31 15:03 GMT+02:00 Timo Rothenpieler : Forgot this, the idea with my approach is to handle the case where --enable-nvenc is requested, but the compile-check fails. >>> Just silently disabling it then seems wrong. > > It's

Re: [FFmpeg-devel] [PATCH v2 14/18] avcodec/avcodec, avformat/movenc: support embedding channel layout to stream side data

2016-08-31 Thread Carl Eugen Hoyos
Hi! 2016-08-31 13:35 GMT+02:00 Erkki Seppälä : > Added support for passing complex channel layout configuration as side > packet data (AV_PKT_DATA_AUDIO_TRACK_CHANNEL_LAYOUT) to ISO > media files as specified by ISO/IEC 14496-12. AVAudioTrackChannelLayout > has the fields to setting both predefine

Re: [FFmpeg-devel] [PATCH v2 03/18] avcodec/metaenc: added an encoder/decoder for timed metadata

2016-08-31 Thread Carl Eugen Hoyos
Hi! 2016-08-31 13:35 GMT+02:00 Erkki Seppälä : > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index 6ac6646..0d0447e6 100644 > --- a/libavcodec/avcodec.h > +++ b/libavcodec/avcodec.h > @@ -639,7 +639,7 @@ enum AVCodecID { > AV_CODEC_ID_DVD_NAV, > AV_CODEC_ID_TIMED_ID3, >

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Carl Eugen Hoyos
2016-08-31 15:26 GMT+02:00 Timo Rothenpieler : >> 2016-08-31 15:03 GMT+02:00 Timo Rothenpieler : >>> Forgot this, the idea with my approach is to handle the case where >>> --enable-nvenc is requested, but the compile-check fails. >> >>> Just silently disabling it then seems wrong. It's what FFmpeg

[FFmpeg-devel] [PATCH v2 20/18] avutil: added channel_layout_isoiec23001_8.h to HEADERS

2016-08-31 Thread Erkki Seppälä
--- libavutil/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/Makefile b/libavutil/Makefile index 1e06176..731b852 100644 --- a/libavutil/Makefile +++ b/libavutil/Makefile @@ -18,6 +18,7 @@ HEADERS = adler32.h \

Re: [FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-08-31 Thread Erkki Seppälä
On 08/31/2016 04:42 PM, Carl Eugen Hoyos wrote: Sorry! No problem :). I meant "defaulting to an additional track in the output mov file if the input file contains such a data track (instead of ignoring this track of the input file by default)". I'll look into how that part of code works and

Re: [FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-08-31 Thread Carl Eugen Hoyos
Hi! 2016-08-31 15:28 GMT+02:00 Erkki Seppälä : > > On 08/31/2016 04:04 PM, Carl Eugen Hoyos wrote: >> >> Iirc, this has the (intended or unintended) side-effect of making >> mov output of ffmpeg (the app) suddenly defaulting to an >> additional track. >> Is this correct? Is it intended? > > Hmm, i

Re: [FFmpeg-devel] [PATCH 1/2] compat/avisynth: minor update for alpha offsets

2016-08-31 Thread Carl Eugen Hoyos
2016-08-31 2:26 GMT+02:00 Stephen Hutchinson : > -//AVS_CPUF_AVX= 0x800, // Sandy Bridge, Bulldozer > + AVS_CPUF_AVX= 0x800, // Sandy Bridge, Bulldozer Is this related to alpha offsets? Carl Eugen ___ ffmpeg-devel mailing list ffm

Re: [FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-08-31 Thread Erkki Seppälä
Hello! On 08/31/2016 04:04 PM, Carl Eugen Hoyos wrote: Iirc, this has the (intended or unintended) side-effect of making mov output of ffmpeg (the app) suddenly defaulting to an additional track. Is this correct? Is it intended? Hmm, it seems mov files still have one track according to these c

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
> 2016-08-31 15:03 GMT+02:00 Timo Rothenpieler : >> Forgot this, the idea with my approach is to handle the case where >> --enable-nvenc is requested, but the compile-check fails. > >> Just silently disabling it then seems wrong. > > But this is what we do for all auto-detected features except xc

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Carl Eugen Hoyos
2016-08-31 15:03 GMT+02:00 Timo Rothenpieler : > Forgot this, the idea with my approach is to handle the case where > --enable-nvenc is requested, but the compile-check fails. > Just silently disabling it then seems wrong. But this is what we do for all auto-detected features except xcb. If chang

Re: [FFmpeg-devel] [PATCH v2 19/18] Updated Changelog

2016-08-31 Thread Carl Eugen Hoyos
[...] Imo, this should add one line to the Changelog and it should definitely not remove an empty line. Carl Eugen ___ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-08-31 Thread Carl Eugen Hoyos
Hi! 2016-08-31 13:35 GMT+02:00 Erkki Seppälä : Iirc, this has the (intended or unintended) side-effect of making mov output of ffmpeg (the app) suddenly defaulting to an additional track. Is this correct? Is it intended? Thank you, Carl Eugen ___ ffmpe

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
Forgot this, the idea with my approach is to handle the case where --enable-nvenc is requested, but the compile-check fails. Just silently disabling it then seems wrong. ___ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listi

Re: [FFmpeg-devel] [PATCH 4/4] lavf/mov: Add support for edit list parsing.

2016-08-31 Thread Michael Niedermayer
On Tue, Aug 30, 2016 at 06:37:22PM -0700, Sasi Inguva wrote: > On Sun, Aug 28, 2016 at 3:10 AM, Michael Niedermayer > wrote: > > > On Sat, Aug 27, 2016 at 03:30:24PM -0700, Sasi Inguva wrote: > > > On Fri, Aug 26, 2016 at 5:55 PM, Michael Niedermayer > > > > > wrote: > > > > > > > On Fri, Aug 26

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Paul B Mahol
On 8/31/16, Michael Niedermayer wrote: > On Wed, Aug 31, 2016 at 10:18:31AM +0200, Paul B Mahol wrote: >> On 8/30/16, Nicolas George wrote: >> > Le quartidi 14 fructidor, an CCXXIV, Paul B Mahol a ecrit : >> >> the filter frame multithreading would just internally, in filter >> >> context >> >> c

[FFmpeg-devel] [PATCH v2 19/18] Updated Changelog

2016-08-31 Thread Erkki Seppälä
Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- Changelog | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/Changelog b/Changelog index 71abe8c..bada511 100644 --- a/Changelog +++ b/Changelog @@ -17,7 +17,17 @@ version : - acrusher audio filter - bitpla

[FFmpeg-devel] [PATCH v2 16/18] avcodec/avcodec, avformat/movenc: introduced AV_PKT_DATA_TRACK_ALTERNATE_GROUP side data for expressing alternate groups

2016-08-31 Thread Erkki Seppälä
Alternate groups previously always generated for ISO media files. With this addition client code can define track groups arbitrarily. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavcodec/avcodec.h | 11 +++ libavformat/movenc.c | 9 + 2 files changed, 20 inserti

[FFmpeg-devel] [PATCH v2 13/18] avformat/mov: read urim metadata from meta

2016-08-31 Thread Erkki Seppälä
The data is read into side packet AV_PKT_DATA_TIMED_METADATA_INFO of format AVTimedMetadata. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/mov.c | 141 ++ 1 file changed, 141 insertions(+) diff --git a/libavformat/mov.

[FFmpeg-devel] [PATCH v2 10/18] avformat/mov, isom: read (multiple) track references (tag and multiple ids)

2016-08-31 Thread Erkki Seppälä
This can be useful in particular with timed meta data tracks related to multiple tracks. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/isom.h | 3 ++ libavformat/mov.c | 119 - 2 files changed, 121 insertions(+), 1 d

[FFmpeg-devel] [PATCH v2 18/18] doc/examples/muxing_with_metadata: example for dealing with timed meta data

2016-08-31 Thread Erkki Seppälä
Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- doc/examples/Makefile | 2 + doc/examples/muxing_with_metadata.c | 885 2 files changed, 887 insertions(+) create mode 100644 doc/examples/muxing_with_metadata.c diff --git a/doc/exam

[FFmpeg-devel] [PATCH v2 05/18] avformat/movenc: support for multiple and client-provided track references

2016-08-31 Thread Erkki Seppälä
Instead of one track reference, allow many, and instead of of track reference type (ie. 'cdsc'), allow many. In addition this patch allows client to explicitly add track references with side packet AV_PKT_DATA_TRACK_REFERENCES containing AVTrackReferences (which of there can many many). Internall

[FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

2016-08-31 Thread Erkki Seppälä
This includes creating an AVCodecTag table ff_codec_metadata_tags as there are for video, audio and subtitles. The tag table is used for mov-compatiblity. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/isom.c | 5 + libavformat/isom.h | 1 + libavformat/movenc.c

[FFmpeg-devel] [PATCH v2 08/18] avcodec/utils: do_decode now supports AVMEDIA_TYPE_DATA

2016-08-31 Thread Erkki Seppälä
This allows using avcodec_send_packet with data frames (ie. timed metadata) Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavcodec/utils.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 138125a..8b55464 100644 --- a/libavcodec

[FFmpeg-devel] [PATCH v2 12/18] avformat/mov: ff_mov_read_stsd_entries now deals with AVMEDIA_TYPE_DATA

2016-08-31 Thread Erkki Seppälä
Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/mov.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 1012b3c..0544d13 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2257,6 +2257,7 @@ int f

[FFmpeg-devel] [PATCH v2 17/18] doc/examples/extract_timed_metadata: added a bare-bones metadata extractor; find only the frames

2016-08-31 Thread Erkki Seppälä
Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- doc/examples/Makefile | 1 + doc/examples/extract_timed_metadata.c | 233 ++ 2 files changed, 234 insertions(+) create mode 100644 doc/examples/extract_timed_metadata.c diff --git a/doc/

[FFmpeg-devel] [PATCH v2 09/18] avcodec/utils: do_encode now supports AVMEDIA_TYPE_DATA

2016-08-31 Thread Erkki Seppälä
Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavcodec/utils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 8b55464..f0e22b9 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -2880,6 +2880,8 @@ static int do_encode(AV

[FFmpeg-devel] [PATCH v2 06/18] avformat/movenc: add the urim sample descriptor

2016-08-31 Thread Erkki Seppälä
This also adds libavformat/movmeta that contains the meta data information. Actually the sample configuration data is performed with side packet data AV_PKT_DATA_TIMED_METADATA_INFO where the value is stored inside AVTimedMetadataInfo. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- l

[FFmpeg-devel] [PATCH v2 11/18] avformat/mov: basic support for identifying (and reading) timed metadata

2016-08-31 Thread Erkki Seppälä
mov_codec_id is now able to set AVMEDIA_TYPE_DATA to the st->codec->codec_type field when the input is of type AVMEDIA_TYPE_DATA; previously it used AVMEDIA_TYPE_SUBTITLE as the value to set in that case. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/mov.c | 18 +

[FFmpeg-devel] [PATCH v2 14/18] avcodec/avcodec, avformat/movenc: support embedding channel layout to stream side data

2016-08-31 Thread Erkki Seppälä
Added support for passing complex channel layout configuration as side packet data (AV_PKT_DATA_AUDIO_TRACK_CHANNEL_LAYOUT) to ISO media files as specified by ISO/IEC 14496-12. AVAudioTrackChannelLayout has the fields to setting both predefined audio layouts, completely configuring the azimuth and

[FFmpeg-devel] [PATCH v2 07/18] avformat/movenc, isom: support metadata in mp4 files

2016-08-31 Thread Erkki Seppälä
This involves adding a new tag to the ff_mp4_obj_type table as well as modifying mp4_get_codec_tag to return 'meta' for AV_CODEC_ID_META. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/isom.c | 1 + libavformat/movenc.c | 1 + 2 files changed, 2 insertions(+) diff --gi

[FFmpeg-devel] [PATCH v2 15/18] avformat/movenc: mov_write_audio_tag can write the proper number of channels, not the hardcoded 2

2016-08-31 Thread Erkki Seppälä
It does it only when the chnl box is to be written. The specification for the chnl box (ISO 14496-12) requires proper number of channels to be written here is it is used, and without the proper number of channels available it becomes tricky to parse the data, as the number of channels in this box c

[FFmpeg-devel] [PATCH v2 02/18] avformat/movenc: mov_write_ftyp_tag: write the major brand as a compatible brand

2016-08-31 Thread Erkki Seppälä
when the option "brand" is used. This allows custom brands to end up in the compatible brands as well. Signed-off-by: Erkki Seppälä Signed-off-by: OZOPlayer --- libavformat/movenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 525d103..

[FFmpeg-devel] [PATCH v2 03/18] avcodec/metaenc: added an encoder/decoder for timed metadata

2016-08-31 Thread Erkki Seppälä
It has the codec id AV_CODEC_ID_META and type of AVMEDIA_TYPE_DATA. This codec basically passes the data forward and is used for referring timed meta data tracks by a codec. It is useful for dealing with the metadata in a similar way as other kinds of codecs. Signed-off-by: Erkki Seppälä Signed-

[FFmpeg-devel] [PATCH v2 00/18] Add timed metadata tracks, track references, channel layout

2016-08-31 Thread Erkki Seppälä
From: Erkki Seppälä Hello, developers! Here is a patch set version 2 with the goal of adding the following features to the ISO base media file format support of FFmpeg (also available at https://github.com/nokiatech/FFmpeg/tree/timed-metadata-v2): The branch is still based on the old master rep

[FFmpeg-devel] [PATCH v2 01/18] avformat/movenc: added ability to use original stream ids as track ids instead of regenerating them

2016-08-31 Thread Erkki Seppälä
Sometimes it's useful to be able to define the exact track numbers in the generated track, instead of always beginning at track id 1. Using the option use_stream_ids_as_track_ids now copies the use stream ids to track ids. Dynamically generated tracks (ie. tmcd) have their track numbers defined as

Re: [FFmpeg-devel] [PATCH 19/21] doc/examples/extract_timed_metadata: added a bare-bones metadata extractor; find only the frames

2016-08-31 Thread Michael Niedermayer
Hi On Thu, Aug 25, 2016 at 10:31:57AM +0300, Erkki Seppälä wrote: > Thanks for pointing out the use of private API. It seemed that this > would have needed to add a new function for the API for decoding > timed metadata frames, but probably a better solution is to use the > avcodec_receive_packet

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 10:18:31AM +0200, Paul B Mahol wrote: > On 8/30/16, Nicolas George wrote: > > Le quartidi 14 fructidor, an CCXXIV, Paul B Mahol a ecrit : > >> the filter frame multithreading would just internally, in filter context > >> cache frames, once enough frames are in cache - call

Re: [FFmpeg-devel] [PATCH 3/3] swscale: add support for P010LE/BE output

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 11:33:49AM +0200, Timo Rothenpieler wrote: > --- > libswscale/output.c | 98 > +++- > libswscale/utils.c | 4 +- > libswscale/x86/swscale.c | 4 +- > tests/ref/fate/filter-pixdesc-p010be | 1

Re: [FFmpeg-devel] [PATCH 05/21] libavformat/movenc: mov_write_ftyp_tag: write the major brand a compatible brand

2016-08-31 Thread Erkki Seppälä
On 08/23/2016 06:35 PM, Yusuke Nakamura wrote: +if (mov->mode == MODE_MP4 && mov->major_brand) +ffio_wfourcc(pb, mov->major_brand); /* write major brand as a compatible brand */ WTF. libavformat has not listed all compatible brands? This is a wrong approach if what David Singer (Ap

Re: [FFmpeg-devel] [PATCH v2] Let clang-FORTIFY build; NFC.

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 04:24:36AM -0400, Ronald S. Bultje wrote: > Hi George, > > On Tue, Aug 30, 2016 at 8:47 PM, George Burgess wrote: > > > Thanks for the feedback! I agree the casts aren't pretty. :) > > > > > Isn't it easier to change your fortify-clang and add a compiler option > > to dis

[FFmpeg-devel] [PATCH 2/3] avfilter/drawutils: honor shift for color component description

2016-08-31 Thread Timo Rothenpieler
--- libavfilter/drawutils.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/drawutils.c b/libavfilter/drawutils.c index a3710db..f6760be 100644 --- a/libavfilter/drawutils.c +++ b/libavfilter/drawutils.c @@ -253,7 +253,8 @@ void ff_draw_color(FFDrawContext *draw,

[FFmpeg-devel] [PATCH 1/3] avfilter/drawutils: P010 is not supported

2016-08-31 Thread Timo Rothenpieler
--- libavfilter/drawutils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavfilter/drawutils.c b/libavfilter/drawutils.c index 8153fde..a3710db 100644 --- a/libavfilter/drawutils.c +++ b/libavfilter/drawutils.c @@ -184,6 +184,8 @@ int ff_draw_init(FFDrawContext *draw, enum AVPixelFormat

[FFmpeg-devel] [PATCH 3/3] swscale: add support for P010LE/BE output

2016-08-31 Thread Timo Rothenpieler
--- libswscale/output.c | 98 +++- libswscale/utils.c | 4 +- libswscale/x86/swscale.c | 4 +- tests/ref/fate/filter-pixdesc-p010be | 1 + tests/ref/fate/filter-pixdesc-p010le | 1 + tests/ref/fate/filter-pixfmts-co

Re: [FFmpeg-devel] [PATCH] lavf/matroskaenc: improve tag-skipping checks

2016-08-31 Thread Michael Niedermayer
On Wed, Aug 31, 2016 at 01:05:31AM -0500, Rodger Combs wrote: > - Add "duration" to list of skipped attributes (we handle that elsewhere) > - Move the list to its own function and make its use consistent These 2 should be in separate patches code moving and code changes are more readable when in

Re: [FFmpeg-devel] [PATCH] configure: improve logic and checks for nvenc

2016-08-31 Thread Timo Rothenpieler
>> +echo 'NV_ENCODE_API_FUNCTION_LIST flist;' >> +echo 'void f(void) { struct { const GUID guid; } s[] = { { >> NV_ENC_PRESET_HQ_GUID } }; }' > > This will most likely prevent nvenc from being enabled for msvc 2012, but not > old > mingw32, which is failing with the error: > > s

Re: [FFmpeg-devel] [PATCH v2] Let clang-FORTIFY build; NFC.

2016-08-31 Thread Ronald S. Bultje
Hi George, On Tue, Aug 30, 2016 at 8:47 PM, George Burgess wrote: > Thanks for the feedback! I agree the casts aren't pretty. :) > > > Isn't it easier to change your fortify-clang and add a compiler option > to disable this specific error for specific targets? > > The short answer is "in some ca

Re: [FFmpeg-devel] Evolution of lavfi's design and API

2016-08-31 Thread Paul B Mahol
On 8/30/16, Nicolas George wrote: > Le quartidi 14 fructidor, an CCXXIV, Paul B Mahol a ecrit : >> the filter frame multithreading would just internally, in filter context >> cache frames, once enough frames are in cache - call workers and be done, >> repeat. At eof call workers on remaining frame