[FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: fix Tags master on seekable output if there are tags after the last stream duration

2016-10-06 Thread James Almer
The dynamic AVIOContext would get closed pointing to the wrong position in the buffer. This is a regression since 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd. Signed-off-by: James Almer --- Example: ./ffmpeg -i fate-samples/vorbis/vorbis_chapter_extension_demo.ogg -c:a copy -metadata:c key=value ou

[FFmpeg-devel] [PATCH 2/2 v2] avformat/matroskaenc: fix targets for attachment tags

2016-10-06 Thread James Almer
Attachment tags were being written targetting non-existent streams in the output file. Also filter filename and mimetype entries, as they are standard elements in the Attachment master. Signed-off-by: James Almer --- Now rebased after the CRC32 patchset. The fileuid is changed to be four bytes l

Re: [FFmpeg-devel] [OPW] OPW Project Proposal

2016-10-06 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 11:23:11PM +0530, Pallavi Kumari wrote: > I am looking for mentor(s). Kindly let me know if anyone interested. iam interrested, i intend to mentor only one applicant though iam also the mentor for "Improve Selftest coverage" (hint if someone wants to take mentoring tha

[FFmpeg-devel] [PATCH] lavf/segment: decide whether to rename based on list URI

2016-10-06 Thread Rodger Combs
This fixes the case of writing segments to local files, but the list over a network protocol. --- libavformat/segment.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 33a5cf0..55dcaf0 100644 --- a/libavformat/segment.c +++ b/

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Michael Niedermayer
On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: > This matrix needs to be applied after all others have (currently only > display matrix from trak), but cannot be handled in movie box, since > streams are not allocated yet. > > So store it in main context and if not identity, app

[FFmpeg-devel] [PATCH] libavformat/tee: Add fifo support for tee

2016-10-06 Thread sebechlebskyjan
From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- This commit makes use of fifo muxer together with tee muxer easier, fifo muxer does not have to be explicitly specified for each slave. For the most simple use case it is sufficient to turn fifo muxer on for all slaves by switching on us

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread compn
On Thu, 6 Oct 2016 16:40:12 -0400 Vittorio Giovara wrote: > > what is the intended / correct SAR / DAR for this sample ? > > without the patch Stream #0:0(und): Video: h264 (High) (avc1 / > 0x31637661), yuv420p, 540x576 [SAR 1:1 DAR 15:16], 102 kb/s, 25 fps, > 25 tbr, 12800 tbn, 50 tbc (defa

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread James Almer
On 10/6/2016 6:10 PM, Dave Rice wrote: > >> On Oct 6, 2016, at 4:33 PM, James Almer wrote: >> >> On 10/6/2016 3:28 PM, Michael Niedermayer wrote: >>> On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote: On 10/6/2016 7:29 AM, Michael Niedermayer wrote: > On Mon, Oct 03, 2016 at 08

[FFmpeg-devel] [PATCH] lavfi/metadata: fix metadata deletion if comparison returns false

2016-10-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavfilter/f_metadata.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavfilter/f_metadata.c b/libavfilter/f_metadata.c index f4a929c..f2c71d9 100644 --- a/libavfilter/f_metadata.c +++ b/libavfilter/f_metadata.c @@ -330,9 +330,7 @@ stat

[FFmpeg-devel] [PATCH] lavfi/sidedata: add filter for manipulating frame side data

2016-10-06 Thread Marton Balint
This is a similar filter to f_metadata, only it works on side data. Since adding side data from a user provided arbitrary binary string is unsafe, because current code assumes that a side data of a certain kind has the proper size, this filter only implements selection and deletion. Also, no value

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Dave Rice
> On Oct 6, 2016, at 4:33 PM, James Almer wrote: > > On 10/6/2016 3:28 PM, Michael Niedermayer wrote: >> On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote: >>> On 10/6/2016 7:29 AM, Michael Niedermayer wrote: On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer wrote: > This

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Vittorio Giovara
On Thu, Oct 6, 2016 at 4:07 PM, Michael Niedermayer wrote: > On Thu, Oct 06, 2016 at 12:12:07AM -0400, Vittorio Giovara wrote: >> On Wed, Oct 5, 2016 at 10:07 PM, Michael Niedermayer >> wrote: >> > On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: >> >> This matrix needs to be app

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread James Almer
On 10/6/2016 3:28 PM, Michael Niedermayer wrote: > On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote: >> On 10/6/2016 7:29 AM, Michael Niedermayer wrote: >>> On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer wrote: This patchsets implements the feature requested on ticket #4347.

Re: [FFmpeg-devel] [PATCH] mov: Evaluate the movie display matrix

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 12:12:07AM -0400, Vittorio Giovara wrote: > On Wed, Oct 5, 2016 at 10:07 PM, Michael Niedermayer > wrote: > > On Wed, Oct 05, 2016 at 05:25:33PM -0400, Vittorio Giovara wrote: > >> This matrix needs to be applied after all others have (currently only > >> display matrix fro

Re: [FFmpeg-devel] [PATCH] Cherry-pick commit 6803a298f4338c19c3032d2417c6e857eb6d95be Support for MIPS cpu P6600

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 10:46:57AM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Signed-off-by: Shivraj Patil > Signed-off-by: Michael Niedermayer > --- > configure |6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) 6803a298f4338c19c3032d2417c6e857eb6d95be back

Re: [FFmpeg-devel] [PATCH] Cherry-pick commit id c1cc13cd2a9b8d6d2810ec42454f328a1a0d5efa avutil/mips/generic_macros_msa: rename macro variable which causes segfault for mips r6

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 10:21:25AM +0530, shivraj.pa...@imgtec.com wrote: > From: Shivraj Patil > > Signed-off-by: Shivraj Patil > Signed-off-by: Michael Niedermayer > --- > libavutil/mips/generic_macros_msa.h | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) commit local

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 11:27:08AM -0300, James Almer wrote: > On 10/6/2016 7:29 AM, Michael Niedermayer wrote: > > On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer wrote: > >> This patchsets implements the feature requested on ticket #4347. > >> The first three patches are preparation work. T

Re: [FFmpeg-devel] [PATCH] vf_fps: Don't flush a cached frame if it should have been dropped

2016-10-06 Thread Nicolas George
Le tridi 13 vendémiaire, an CCXXV, Derek Buitenhuis a écrit : > This "break" may actually be more correct output, but I am not familiar enough > with lavfi or vf_fps to say. Removing the last frame in case where it should > have > been removed was the entire point of this patch. > > Perhaps someo

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: don't write an empty Colour master element

2016-10-06 Thread James Almer
On 10/2/2016 8:14 PM, James Almer wrote: > On 9/27/2016 8:49 PM, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavformat/matroskaenc.c | 53 >> ++- >> 1 file changed, 34 insertions(+), 19 deletions(-) > > Ping for patchset. Pushed. __

[FFmpeg-devel] dh option in nnedi3

2016-10-06 Thread Jon bae
Hello All, sorry for that question - I know is open source and we can add option by ours self, but my knowledge is to limited for that. Is it possible to integrate the dh (double high) option from vapoursynth nnedi3 filter to the ffmpeg nnedi filter? I have start copy parts from the sources, but

Re: [FFmpeg-devel] ffserver: current status and future plans

2016-10-06 Thread Stephan Holljes
On Fri, Sep 23, 2016 at 6:08 PM, Michael Niedermayer wrote: > On Fri, Sep 23, 2016 at 05:46:06PM +0200, Michael Fritscher wrote: >> Hello, >> >> as promised I started to work on ffserver. The focus of the first >> week was to remake a fresh building environment - incl. FATE - and >> get an overvie

Re: [FFmpeg-devel] [PATCH 3/3] lavc/mediacodecdec: fix size variable shadowing in ff_mediacodec_dec_decode

2016-10-06 Thread Matthieu Bouron
On Thu, Oct 06, 2016 at 03:59:31PM +0200, Michael Niedermayer wrote: > On Thu, Oct 06, 2016 at 02:31:57PM +0200, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > Fixes incompatible pointer type warning on 64-bit systems. > > --- > > libavcodec/mediacodecdec.c | 1 - > > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread James Almer
On 10/6/2016 7:29 AM, Michael Niedermayer wrote: > On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer wrote: >> This patchsets implements the feature requested on ticket #4347. >> The first three patches are preparation work. The first one isn't >> strictly related to the implementation, but com

Re: [FFmpeg-devel] [PATCH] ffmpeg: remove unused and errorneous AVFrame timestamp check

2016-10-06 Thread Hendrik Leppkes
On Tue, Oct 4, 2016 at 4:44 PM, James Almer wrote: > On 10/4/2016 11:35 AM, Hendrik Leppkes wrote: >> On Tue, Oct 4, 2016 at 4:32 PM, wm4 wrote: >>> On Tue, 4 Oct 2016 14:15:03 +0200 >>> Michael Niedermayer wrote: >>> On Tue, Oct 04, 2016 at 01:52:02PM +0200, Hendrik Leppkes wrote: > On

Re: [FFmpeg-devel] [PATCH 3/3] lavc/mediacodecdec: fix size variable shadowing in ff_mediacodec_dec_decode

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 02:31:57PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > Fixes incompatible pointer type warning on 64-bit systems. > --- > libavcodec/mediacodecdec.c | 1 - > 1 file changed, 1 deletion(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730

Re: [FFmpeg-devel] [PATCH 3/4] lavc/utils: avcodec_string: dump field order when known

2016-10-06 Thread Tobias Rapp
On 04.10.2016 06:49, Rodger Combs wrote: --- libavcodec/utils.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 6f4df93..a0931c6 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -3228,6 +3228,21 @@ void avcodec_string(

[FFmpeg-devel] [PATCH 2/3] lavc/mediacodecdec: rename dequeued_buffer_nb to output_buffer_count

2016-10-06 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mediacodecdec.c | 6 +++--- libavcodec/mediacodecdec.h | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 2ab173b..3d519ca 100644 --- a/libavcodec/mediacodecdec.c +++ b/libavcod

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Jerome Martinez
Le 06/10/2016 à 12:29, Michael Niedermayer a écrit : Does this reduce writing speed ? in the same manner as e.g. reducing FFV1 writing speed with default configuration, i.e. with CRC per slice (same kind of job). On my machine (i7 from 2012), CRC computing takes less than 1% of the CPU when I

[FFmpeg-devel] [PATCH 1/3] lavc/mediacodecdec: remove first output buffer timing debug log

2016-10-06 Thread Matthieu Bouron
From: Matthieu Bouron --- libavcodec/mediacodecdec.c | 8 libavcodec/mediacodecdec.h | 3 --- 2 files changed, 11 deletions(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 0faa4cf..2ab173b 100644 --- a/libavcodec/mediacodecdec.c +++ b/libavcodec/mediacodec

[FFmpeg-devel] [PATCH 3/3] lavc/mediacodecdec: fix size variable shadowing in ff_mediacodec_dec_decode

2016-10-06 Thread Matthieu Bouron
From: Matthieu Bouron Fixes incompatible pointer type warning on 64-bit systems. --- libavcodec/mediacodecdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c index 3d519ca..126de17 100644 --- a/libavcodec/mediacodecdec.c +++ b/libavcod

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-06 Thread Michael Niedermayer
On Thu, Oct 06, 2016 at 03:08:18PM +0700, nablet developer wrote: > > > > > you have set the author for your git to "Nablet Developer" > > is this intended? Do you not want your name to be in the Author field ? > > If you dont mind your name to be in the author field then please > > resubmit the

Re: [FFmpeg-devel] [PATCH]lavu/hwaccel_vaapi: Add a fixme for the missing byte_order check

2016-10-06 Thread wm4
On Thu, 6 Oct 2016 10:34:35 +0100 Mark Thompson wrote: > On 06/10/16 09:05, Carl Eugen Hoyos wrote: > > 2016-10-05 21:55 GMT+02:00 Mark Thompson : > > > > What I meant is: > > How can "LE" make sense for an 8bit planar format? > > It does not. So, another reason to ignore that field. > > >

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Michael Niedermayer
On Mon, Oct 03, 2016 at 08:36:56PM -0300, James Almer wrote: > This patchsets implements the feature requested on ticket #4347. > The first three patches are preparation work. The first one isn't > strictly related to the implementation, but comes in handy > nonetheless. > > Patches 4 to 11 can be

Re: [FFmpeg-devel] [PATCH]lavu/hwaccel_vaapi: Add a fixme for the missing byte_order check

2016-10-06 Thread Carl Eugen Hoyos
2016-10-06 11:34 GMT+02:00 Mark Thompson : >> Do you think vaapi's P010 should be mapped to FFmpeg's >> P010LE instead of P010? > I would prefer to only consider this problem once we have > some working system to test on. No objections. Thank you, Carl Eugen

Re: [FFmpeg-devel] [PATCH]lavu/hwaccel_vaapi: Add a fixme for the missing byte_order check

2016-10-06 Thread Mark Thompson
On 06/10/16 09:05, Carl Eugen Hoyos wrote: > 2016-10-05 21:55 GMT+02:00 Mark Thompson : > > What I meant is: > How can "LE" make sense for an 8bit planar format? It does not. So, another reason to ignore that field. > (And what does it tell us about the author?) > This of course assumes that "Y

Re: [FFmpeg-devel] [PATCH 00/11] CRC32 support for Matroska muxer

2016-10-06 Thread Jerome Martinez
Le 04/10/2016 à 01:36, James Almer a écrit : This patchsets implements the feature requested on ticket #4347. I tested the patches and it works as expected: the CRC-32 elements are in the first position and values are the right ones. ___ ffmpeg-deve

Re: [FFmpeg-devel] [PATCH] avcodec/qsvenc_h264: fix segfault when a53 SEI is not available

2016-10-06 Thread nablet developer
> > you have set the author for your git to "Nablet Developer" > is this intended? Do you not want your name to be in the Author field ? > If you dont mind your name to be in the author field then please > resubmit the patch with correct Author > yes, it's intentional, please apply as is with g

Re: [FFmpeg-devel] [PATCH]lavu/hwaccel_vaapi: Add a fixme for the missing byte_order check

2016-10-06 Thread Carl Eugen Hoyos
2016-10-05 21:55 GMT+02:00 Mark Thompson : > On 05/10/16 19:02, Carl Eugen Hoyos wrote: >> 2016-10-05 16:39 GMT+02:00 Mark Thompson : >>> On 05/10/16 13:06, Carl Eugen Hoyos wrote: I cannot test here but afaict, the current implementation of vaapi_pix_fmt_from_fourcc() can't be corre