Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: export the full UMID as metadata

2015-01-06 Thread tim nicholson
On 05/01/15 20:41, Mark Reid wrote: > Hi, > This patch exports the full umid of packages as metadata. ffmpeg currently > only exports the material number portion of the umid. > > The new format is >--- > example: >060a2b340101010101010f00-13-00-53dc416b9a770251060e2b347f7f2a80 > I th

[FFmpeg-devel] [PATCH] lavfi: check av_strdup() return value

2015-01-06 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/af_amix.c | 2 ++ libavfilter/af_join.c | 2 ++ libavfilter/split.c | 2 ++ 3 files changed, 6 insertions(+) diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c index e40969f..fd9d135 100644 --- a/libavfilter/af_amix.c +++ b/libavfilter/af_ami

Re: [FFmpeg-devel] [PATCH] lavfi: check av_strdup() return value

2015-01-06 Thread Clément Bœsch
On Tue, Jan 06, 2015 at 09:43:52AM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/af_amix.c | 2 ++ > libavfilter/af_join.c | 2 ++ > libavfilter/split.c | 2 ++ > 3 files changed, 6 insertions(+) > > diff --git a/libavfilter/af_amix.c b/libavfilter/af_amix.c > in

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-06 Thread Paul B Mahol
On 1/5/15, Michael Niedermayer wrote: > On Mon, Jan 05, 2015 at 04:33:44PM +, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/Makefile | 2 + >> libavfilter/allfilters.c | 2 + >> libavfilter/f_deinterleave.c | 143 >> +

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-06 Thread Stefano Sabatini
On date Monday 2015-01-05 16:33:44 +, Paul B Mahol encoded: > Signed-off-by: Paul B Mahol > --- > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > libavfilter/f_deinterleave.c | 143 > +++ > 3 files changed, 147 insertion

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-06 Thread Paul B Mahol
On 1/6/15, Stefano Sabatini wrote: > On date Monday 2015-01-05 16:33:44 +, Paul B Mahol encoded: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/Makefile | 2 + >> libavfilter/allfilters.c | 2 + >> libavfilter/f_deinterleave.c | 143 >> ++

Re: [FFmpeg-devel] [PATCH] avformat/aiffdec: only read codec tag when there is space in header

2015-01-06 Thread Reimar Döffinger
On 04.01.2015, at 22:44, Peter Ross wrote: > Signed-off-by: Peter Ross > --- > Revised patch. Now passes FATE. > > libavformat/aiffdec.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c > index 91ef2a4..8dbed32 100644 > ---

Re: [FFmpeg-devel] mpegvideo : interlaced_frame with picture_structure == PICT_FRAME

2015-01-06 Thread Vanista Herion
Good explanation, I neglected telecine; wasn't aware of how it worked. Agreed, yadif should handle that, the decoder cannot reverse-telecine by itself without some kind of hinting of the original frames correspondence. Thanks a lot. 2015-01-05 18:35 GMT-05:00 Michael Niedermayer : > On Mon, Jan 0

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-06 Thread compn
On Tue, 6 Jan 2015 10:10:53 + Paul B Mahol wrote: > On 1/6/15, Stefano Sabatini wrote: > > On date Monday 2015-01-05 16:33:44 +, Paul B Mahol encoded: > >> Signed-off-by: Paul B Mahol > >> 3 files changed, 147 insertions(+) > >> create mode 100644 libavfilter/f_deinterleave.c > > > >

Re: [FFmpeg-devel] [PATCH 2/2] avformat/id3v2: support USLT tags

2015-01-06 Thread Michael Niedermayer
On Tue, Jan 06, 2015 at 07:19:28AM +0100, wm4 wrote: > On Mon, 5 Jan 2015 21:10:31 +0100 > Michael Niedermayer wrote: > > > On Mon, Jan 05, 2015 at 06:56:20PM +0100, wm4 wrote: > > > I think this turned out pretty terrible. There's no good way to add new > > > custom tags that write to AVFormatCo

Re: [FFmpeg-devel] [PATCH] lavfi: add deinterleave filters

2015-01-06 Thread tim nicholson
On 06/01/15 16:31, compn wrote: > On Tue, 6 Jan 2015 10:10:53 + > Paul B Mahol wrote: > >> On 1/6/15, Stefano Sabatini wrote: >>> On date Monday 2015-01-05 16:33:44 +, Paul B Mahol encoded: Signed-off-by: Paul B Mahol 3 files changed, 147 insertions(+) create mode 10064

[FFmpeg-devel] MOV edit list support using side data and libavfilter

2015-01-06 Thread Clément Bœsch
Latest attempt, I hope people like it better. Basic idea: export on demand the elst atom using side data and let the application (here ffmpeg/ffplay through cmdutils) parse it and honor it as they fit (here with libavfilter). I'm a bit uncertain about the FATE changes, and I also don't like the i

[FFmpeg-devel] [PATCH 1/5] avformat/mov: export edit list as stream side data

2015-01-06 Thread Clément Bœsch
From: Clément Bœsch Add a AV_PKT_DATA_MOV_TIMELINE stream side data which will be used by the MOV/MP4 demuxer when "ignore_editlist" option is set to "export" (or -1, not the default). Note: exporting the timeline as side data inhibits current demuxer heuristics. TODO: bump micro --- libavcode

[FFmpeg-devel] [PATCH 2/5] cmdutils: add insert_timeline_graph()

2015-01-06 Thread Clément Bœsch
From: Clément Bœsch This function will be used in the following commits in ffmpeg and ffplay. --- cmdutils.c | 170 + cmdutils.h | 12 + 2 files changed, 182 insertions(+) diff --git a/cmdutils.c b/cmdutils.c index b35180e..0e22e5

[FFmpeg-devel] [PATCH 3/5] ffmpeg: add support for MOV edit lists

2015-01-06 Thread Clément Bœsch
From: Clément Bœsch --- doc/ffmpeg.texi |3 + ffmpeg.h|2 + ffmpeg_filter.c | 32 +- ffmpeg_opt.c| 13 +- tests/ref/fate/gsm-toast| 1000 +++ tests/ref/fate/pcm_s16

[FFmpeg-devel] [PATCH 5/5] Changelog: mention edit list support

2015-01-06 Thread Clément Bœsch
From: Clément Bœsch --- Changelog | 1 + 1 file changed, 1 insertion(+) diff --git a/Changelog b/Changelog index 4575e8b..43998cc 100644 --- a/Changelog +++ b/Changelog @@ -11,6 +11,7 @@ version : - tblend filter - cropdetect support for non 8bpp, absolute (if limit >= 1) and relative (if li

[FFmpeg-devel] [PATCH] ffserver: Use av_malloc_array()

2015-01-06 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- ffserver.c |6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/ffserver.c b/ffserver.c index c4cf390..19d0b25 100644 --- a/ffserver.c +++ b/ffserver.c @@ -2810,11 +2810,9 @@ static int prepare_sdp_description(FFServerStream *stream,

[FFmpeg-devel] [PATCH 4/5] ffplay: add support for MOV edit lists

2015-01-06 Thread Clément Bœsch
From: Clément Bœsch --- doc/ffplay.texi | 3 +++ ffplay.c| 25 ++--- 2 files changed, 25 insertions(+), 3 deletions(-) diff --git a/doc/ffplay.texi b/doc/ffplay.texi index 45731a2..126c19b 100644 --- a/doc/ffplay.texi +++ b/doc/ffplay.texi @@ -174,6 +174,9 @@ input

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: read project_name metadata

2015-01-06 Thread Mark Reid
On Mon, Jan 5, 2015 at 11:26 PM, Clément Bœsch wrote: > On Mon, Jan 05, 2015 at 04:53:05PM -0800, Mark Reid wrote: > > Hi, > > MXF files generated by Media Composer or LibMXF can contain a > > project name property in the Preface. Lots of existing samples have them. > > > > http://samples.ffmpeg.

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: read project_name metadata

2015-01-06 Thread Clément Bœsch
On Tue, Jan 06, 2015 at 10:06:13AM -0800, Mark Reid wrote: > On Mon, Jan 5, 2015 at 11:26 PM, Clément Bœsch wrote: > > > On Mon, Jan 05, 2015 at 04:53:05PM -0800, Mark Reid wrote: > > > Hi, > > > MXF files generated by Media Composer or LibMXF can contain a > > > project name property in the Pref

Re: [FFmpeg-devel] [PATCH] libavformat/mxfdec.c: export the full UMID as metadata

2015-01-06 Thread Mark Reid
On Tue, Jan 6, 2015 at 12:03 AM, tim nicholson < nichot20-at-yahoo@ffmpeg.org> wrote: > On 05/01/15 20:41, Mark Reid wrote: > > Hi, > > This patch exports the full umid of packages as metadata. ffmpeg > currently > > only exports the material number portion of the umid. > > > > The new format

Re: [FFmpeg-devel] [PATCH 2/5] cmdutils: add insert_timeline_graph()

2015-01-06 Thread wm4
On Tue, 6 Jan 2015 18:09:58 +0100 Clément Bœsch wrote: > From: Clément Bœsch > > This function will be used in the following commits in ffmpeg and > ffplay. > --- > cmdutils.c | 170 > + > cmdutils.h | 12 + > 2 files changed,

Re: [FFmpeg-devel] [PATCH 2/5] cmdutils: add insert_timeline_graph()

2015-01-06 Thread Clément Bœsch
On Tue, Jan 06, 2015 at 08:28:48PM +0100, wm4 wrote: > On Tue, 6 Jan 2015 18:09:58 +0100 > Clément Bœsch wrote: > > > From: Clément Bœsch > > > > This function will be used in the following commits in ffmpeg and > > ffplay. > > --- > > cmdutils.c | 170 > > +++

Re: [FFmpeg-devel] [PATCH 2/5] cmdutils: add insert_timeline_graph()

2015-01-06 Thread wm4
On Tue, 6 Jan 2015 20:40:51 +0100 Clément Bœsch wrote: > On Tue, Jan 06, 2015 at 08:28:48PM +0100, wm4 wrote: > > On Tue, 6 Jan 2015 18:09:58 +0100 > > Clément Bœsch wrote: > > > > > From: Clément Bœsch > > > > > > This function will be used in the following commits in ffmpeg and > > > ffpla

Re: [FFmpeg-devel] [PATCH 2/5] cmdutils: add insert_timeline_graph()

2015-01-06 Thread Clément Bœsch
On Tue, Jan 06, 2015 at 10:06:15PM +0100, wm4 wrote: > On Tue, 6 Jan 2015 20:40:51 +0100 > Clément Bœsch wrote: > > > On Tue, Jan 06, 2015 at 08:28:48PM +0100, wm4 wrote: > > > On Tue, 6 Jan 2015 18:09:58 +0100 > > > Clément Bœsch wrote: > > > > > > > From: Clément Bœsch > > > > > > > > This

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-06 Thread Michael Niedermayer
On Tue, Jan 06, 2015 at 01:56:01AM -0300, James Almer wrote: > It's not an installed header. Use libavutil/avconfig.h instead. assuming noone objects to this "hack" ... ugly but it should fix the build so ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those

Re: [FFmpeg-devel] [PATCH 4/4] nvenc: Try and do interlaced encoding.

2015-01-06 Thread Agatha Hu
On 2015/1/6 15:36, Philip Langdale wrote: Doesn't work. Signed-off-by: Philip Langdale --- libavcodec/libnvenc.c | 7 +++ libavcodec/libnvenc.h | 3 +++ libavcodec/nvencoder.c | 6 -- 3 files changed, 14 insertions(+), 2 deletions(-) diff --git a/libavcodec/libnvenc.c b/libavcod

Re: [FFmpeg-devel] [PATCH] avutil/opencl: don't include config.h

2015-01-06 Thread Wei Gao
2015-01-06 12:56 GMT+08:00 James Almer : > It's not an installed header. Use libavutil/avconfig.h instead. > > Signed-off-by: James Almer > --- > Untested as i don't have an OpenCL SDK installed. > > configure | 2 +- > libavutil/opencl.h | 2 +- > 2 files changed, 2 insertions(+), 2 de