Re: [FFmpeg-devel] [PATCH v2] Add GBRAP12 pixel format

2016-02-06 Thread Michael Niedermayer
On Sat, Feb 06, 2016 at 12:36:10AM +, Kieran Kunhya wrote: > Output still looks a bit odd though how can this be reproduced ? > > --- > libavutil/pixdesc.c | 28 > libavutil/pixfmt.h| 4 > libswscale/input.c| 4 > l

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread Michael Niedermayer
On Fri, Feb 05, 2016 at 10:41:26AM -0500, Ronald S. Bultje wrote: > Hi, > > On Fri, Feb 5, 2016 at 9:34 AM, Hendrik Leppkes wrote: > > > On Fri, Feb 5, 2016 at 3:33 PM, Derek Buitenhuis > > wrote: > > > On 2/5/2016 2:19 PM, Michael Niedermayer wrote: > > >> because the code builds the header a

Re: [FFmpeg-devel] [PATCH 1/2] dirac_dwt: Don't pass information in context as arguments

2016-02-06 Thread Rostislav Pehlivanov
On 6 February 2016 at 03:37, Timothy Gu wrote: > --- > libavcodec/dirac_dwt.c | 13 ++--- > libavcodec/dirac_dwt_template.c | 25 + > 2 files changed, 19 insertions(+), 19 deletions(-) > > LGTM d->temp = (uint8_t *)(((TYPE *)d->temp) + 8); might appear c

Re: [FFmpeg-devel] [PATCH 2/2] dirac_dwt: Rename init2 to init

2016-02-06 Thread Rostislav Pehlivanov
On 6 February 2016 at 03:37, Timothy Gu wrote: > The functions are all private. > --- > libavcodec/dirac_dwt.c | 12 ++-- > libavcodec/dirac_dwt.h | 6 +++--- > libavcodec/dirac_dwt_template.c | 10 +- > libavcodec/diracdec.c | 4 ++-- > 4 files chan

[FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Paul B Mahol
Hi, patch attached. From c719b95ce84bb1c3db3e6b751d80a840f35726b0 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 Feb 2016 11:19:45 +0100 Subject: [PATCH] avfilter: add metadata filters Signed-off-by: Paul B Mahol --- doc/filters.texi | 45 ++ libavfilter/Makefile

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread Ronald S. Bultje
Hi, On Sat, Feb 6, 2016 at 5:32 AM, Michael Niedermayer wrote: > On Fri, Feb 05, 2016 at 10:41:26AM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Fri, Feb 5, 2016 at 9:34 AM, Hendrik Leppkes > wrote: > > > > > On Fri, Feb 5, 2016 at 3:33 PM, Derek Buitenhuis > > > wrote: > > > > On 2/5/201

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread James Almer
On 2/6/2016 10:42 AM, Ronald S. Bultje wrote: > Hi, > > On Sat, Feb 6, 2016 at 5:32 AM, Michael Niedermayer > wrote: > >> On Fri, Feb 05, 2016 at 10:41:26AM -0500, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Fri, Feb 5, 2016 at 9:34 AM, Hendrik Leppkes >> wrote: >>> On Fri, Feb 5, 2016 at 3

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Moritz Barsnick
On Sat, Feb 06, 2016 at 12:47:25 +0100, Paul B Mahol wrote: > patch attached. Nice. (Untested.) > +Manipulate frames metadata. Either: frame metadata [using this as a generic term] or: frames' metadata I prefer the former. > +If both @code{value} and @code{key} is set, select fram

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Andreas Cadhalpun
On 03.02.2016 00:41, Andreas Cadhalpun wrote: > I see, it passes '-b $(SRC_PATH)' to lcov, which is now unnecessary/harmful > for out-of-tree builds using the src link. > The first attached patch changes it to simply use the current directory in > that case. > The second patch was necessary to fix

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Andreas Cadhalpun
On 02.02.2016 01:04, Andreas Cadhalpun wrote: > Subject: [PATCH] configure: fall back to using full path if src is a directory > > Or when building in-tree. > Also don't try to remove src on distclean in these cases. > > Signed-off-by: Andreas Cadhalpun > --- > Makefile | 5 - > configure

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread Nicolas George
L'octidi 18 pluviôse, an CCXXIV, James Almer a écrit : > The couple times i dealt with a trac ticket that wasn't a bug i used > "addressed" instead of "fixed". I would say it "addresses" a ticket if the ticket is valid but the commit does not really fix the issue, for example print a helpful speci

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 27.01.2016 01:22, Andreas Cadhalpun wrote: > On 27.01.2016 01:16, Ronald S. Bultje wrote: >> Please wait until my patch is finished. > > Do you think that will happen in the coming days? This seems not to be the case, so I intend to push these two patches soon. They can easily be reverted once

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 17:25:42 +0100 Andreas Cadhalpun wrote: > On 27.01.2016 01:22, Andreas Cadhalpun wrote: > > On 27.01.2016 01:16, Ronald S. Bultje wrote: > >> Please wait until my patch is finished. > > > > Do you think that will happen in the coming days? > > This seems not to be the c

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 17:31, wm4 wrote: > On Sat, 6 Feb 2016 17:25:42 +0100 > Andreas Cadhalpun wrote: > >> On 27.01.2016 01:22, Andreas Cadhalpun wrote: >>> On 27.01.2016 01:16, Ronald S. Bultje wrote: Please wait until my patch is finished. >>> >>> Do you think that will happen in the coming d

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 17:32:12 +0100 Andreas Cadhalpun wrote: > On 06.02.2016 17:31, wm4 wrote: > > On Sat, 6 Feb 2016 17:25:42 +0100 > > Andreas Cadhalpun wrote: > > > >> On 27.01.2016 01:22, Andreas Cadhalpun wrote: > >>> On 27.01.2016 01:16, Ronald S. Bultje wrote: > Please wait un

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 17:35, wm4 wrote: > On Sat, 6 Feb 2016 17:32:12 +0100 > Andreas Cadhalpun wrote: > >> On 06.02.2016 17:31, wm4 wrote: >>> On Sat, 6 Feb 2016 17:25:42 +0100 >>> Andreas Cadhalpun wrote: >>> On 27.01.2016 01:22, Andreas Cadhalpun wrote: > On 27.01.2016 01:16, Ronald S.

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 17:38:45 +0100 Andreas Cadhalpun wrote: > On 06.02.2016 17:35, wm4 wrote: > > On Sat, 6 Feb 2016 17:32:12 +0100 > > Andreas Cadhalpun wrote: > > > >> On 06.02.2016 17:31, wm4 wrote: > >>> On Sat, 6 Feb 2016 17:25:42 +0100 > >>> Andreas Cadhalpun wrote: > >>> >

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 17:42, wm4 wrote: > On Sat, 6 Feb 2016 17:38:45 +0100 > Andreas Cadhalpun wrote: > >> On 06.02.2016 17:35, wm4 wrote: >>> On Sat, 6 Feb 2016 17:32:12 +0100 >>> Andreas Cadhalpun wrote: So what do you suggest instead to get VLC's hwaccels working again soon? >>> >>> Send a pa

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 17:48:17 +0100 Andreas Cadhalpun wrote: > On 06.02.2016 17:42, wm4 wrote: > > On Sat, 6 Feb 2016 17:38:45 +0100 > > Andreas Cadhalpun wrote: > > > >> On 06.02.2016 17:35, wm4 wrote: > >>> On Sat, 6 Feb 2016 17:32:12 +0100 > >>> Andreas Cadhalpun wrote: > So what d

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 18:02, wm4 wrote: > On Sat, 6 Feb 2016 17:48:17 +0100 > Andreas Cadhalpun wrote: > >> On 06.02.2016 17:42, wm4 wrote: >>> On Sat, 6 Feb 2016 17:38:45 +0100 >>> Andreas Cadhalpun wrote: >>> On 06.02.2016 17:35, wm4 wrote: > On Sat, 6 Feb 2016 17:32:12 +0100 > Andre

[FFmpeg-devel] [PATCH] avfilter: add threshold filter

2016-02-06 Thread Paul B Mahol
Hi, patch attached. From 5352dbf21552746b2bf4359abd36b67a1b73a1bf Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 Feb 2016 17:11:07 +0100 Subject: [PATCH] avfilter: add threshold filter Signed-off-by: Paul B Mahol --- doc/filters.texi | 49 +++ libavfilter/Makefile

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 18:15:37 +0100 Andreas Cadhalpun wrote: > On 06.02.2016 18:02, wm4 wrote: > > On Sat, 6 Feb 2016 17:48:17 +0100 > > Andreas Cadhalpun wrote: > > > >> On 06.02.2016 17:42, wm4 wrote: > >>> On Sat, 6 Feb 2016 17:38:45 +0100 > >>> Andreas Cadhalpun wrote: > >>> >

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Mats Peterson
On 02/06/2016 06:25 PM, wm4 wrote: A regression in VLC. Make them fix it. VLC = Very Lame Crap. Just kidding. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 18:25, wm4 wrote: > On Sat, 6 Feb 2016 18:15:37 +0100 > Andreas Cadhalpun wrote: > >> On 06.02.2016 18:02, wm4 wrote: >>> On Sat, 6 Feb 2016 17:48:17 +0100 >>> Andreas Cadhalpun wrote: This has nothing to do with my convenience. It is about fixing an important regression

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Paul B Mahol
On 2/6/16, Andreas Cadhalpun wrote: > On 06.02.2016 18:25, wm4 wrote: >> On Sat, 6 Feb 2016 18:15:37 +0100 >> Andreas Cadhalpun wrote: >> >>> On 06.02.2016 18:02, wm4 wrote: On Sat, 6 Feb 2016 17:48:17 +0100 Andreas Cadhalpun wrote: > This has nothing to do with my convenience. It

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread wm4
On Sat, 6 Feb 2016 18:46:59 +0100 Andreas Cadhalpun wrote: > On 06.02.2016 18:25, wm4 wrote: > > On Sat, 6 Feb 2016 18:15:37 +0100 > > Andreas Cadhalpun wrote: > > > >> On 06.02.2016 18:02, wm4 wrote: > >>> On Sat, 6 Feb 2016 17:48:17 +0100 > >>> Andreas Cadhalpun wrote: > This has

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread James Almer
On 2/6/2016 2:46 PM, Andreas Cadhalpun wrote: > Thus nothing changed and I still intend to apply these patches soon. No you wont, that's not how it works. This patch has one person in favor and more than one against. It can't be applied. How to solve this? Convince those against, either with argum

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Carl Eugen Hoyos
wm4 googlemail.com> writes: > You want to revert a change that was often discussed > and found to be the best way to go about the issue, Please remind me again how I can reproduce the "issue", I seem to have forgotten. Thank you, Carl Eugen ___ ffm

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 18:52, Paul B Mahol wrote: > On 2/6/16, Andreas Cadhalpun wrote: >> On 06.02.2016 18:25, wm4 wrote: >>> On Sat, 6 Feb 2016 18:15:37 +0100 >>> Andreas Cadhalpun wrote: All of that has been said before. >>> >>> Indeed. >> >> Thus nothing changed and I still intend to apply these

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Andreas Cadhalpun
On 06.02.2016 18:55, wm4 wrote: > On Sat, 6 Feb 2016 18:46:59 +0100 > Andreas Cadhalpun wrote: >> You're not constructive. > > Even if that is true, neither are you. I try to be constructive. The problem is that all proposed alternative ways (change VLC to use one thread for hwaccel and fall bac

[FFmpeg-devel] [PATCH 02/12] avutil/parseutils: accept everything in av_parse_time that ff_iso8601_to_unix_time accepts

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavutil/parseutils.c| 8 ++-- tests/ref/fate/parseutils | 1 + 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c index fd8cf2b..0efc608 100644 --- a/libavutil/parseutils.c +++ b/libavutil/parse

[FFmpeg-devel] [PATCH 04/12] avformat/asfenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/asfenc.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/libavformat/asfenc.c b/libavformat/asfenc.c index a6a8242..14f92e2 100644 --- a/libavformat/asfenc.c +++ b/libavformat/asfenc.c @@ -388,7 +388,7 @@ static int asf_

[FFmpeg-devel] [PATCH 01/12] avutil/parseutils: use microsecond precision when parsing "now" in av_parse_time()

2016-02-06 Thread Marton Balint
Use av_gettime() instead of time(0) for querying current time. Signed-off-by: Marton Balint --- libavutil/parseutils.c| 12 +++- tests/ref/fate/parseutils | 4 ++-- 2 files changed, 9 insertions(+), 7 deletions(-) diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c index 9

[FFmpeg-devel] [PATCH 06/12] avformat/ffmenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
FYI this muxer bails out on parse error and not just warn the user. Signed-off-by: Marton Balint --- libavformat/ffmenc.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/libavformat/ffmenc.c b/libavformat/ffmenc.c index f0b5743..bbab1db 100644 --- a/libavformat/ffmenc

[FFmpeg-devel] [PATCH 10/12] avformat/matroskaenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Also increase precision to microsecs, and avoid writing creation_time as a simple tag metadata item. Signed-off-by: Marton Balint --- libavformat/matroskaenc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index f7

[FFmpeg-devel] [PATCH 08/12] avformat/movenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/movenc.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index c6fba21..eeb893b 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5370,13 +5370,7 @@ static int mov_wr

[FFmpeg-devel] [PATCH 11/12] avformat/utils: remove ff_iso8601_to_unix_time

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/internal.h | 5 - libavformat/utils.c| 12 2 files changed, 17 deletions(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index f88cc9a..67fffa0 100644 --- a/libavformat/internal.h +++ b/libavformat/internal.h @@ -

[FFmpeg-devel] [PATCH 03/12] avformat/utils: add ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/internal.h | 11 +++ libavformat/utils.c| 17 + 2 files changed, 28 insertions(+) diff --git a/libavformat/internal.h b/libavformat/internal.h index 2cb3481..f88cc9a 100644 --- a/libavformat/internal.h +++ b/libavformat/int

[FFmpeg-devel] [PATCH 05/12] avformat/dvenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/dvenc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/dvenc.c b/libavformat/dvenc.c index 502cc26..6cefe8b 100644 --- a/libavformat/dvenc.c +++ b/libavformat/dvenc.c @@ -302,7 +302,6 @@ static DVMuxContext* dv_init_

[FFmpeg-devel] [PATCH 09/12] avformat/mxfenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/mxfenc.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 6da8b10..cd13f89 100644 --- a/libavformat/mxfenc.c +++ b/libavformat/mxfenc.c @@ -2041,7 +2041,6 @@ static int mxf_write_

[FFmpeg-devel] [PATCH 07/12] avformat/gxfenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Marton Balint
Signed-off-by: Marton Balint --- libavformat/gxfenc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/libavformat/gxfenc.c b/libavformat/gxfenc.c index 12031f7..8229fe9 100644 --- a/libavformat/gxfenc.c +++ b/libavformat/gxfenc.c @@ -436,13 +436,11 @@ static int gxf_write_

[FFmpeg-devel] [PATCH 12/12] ffmpeg: remove hardcoded 'now' creation_time support

2016-02-06 Thread Marton Balint
Every date parsing routine now uses av_parse_time which handles 'now' and provides greater precision as well. This change also enables the segmenter muxer to set the proper 'now' creation time at the beginning of each segment. Signed-off-by: Marton Balint --- ffmpeg_opt.c | 11 --- 1 fil

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread Ronald S. Bultje
Hi, On Sat, Feb 6, 2016 at 10:50 AM, Nicolas George wrote: > L'octidi 18 pluviôse, an CCXXIV, James Almer a écrit : > > The couple times i dealt with a trac ticket that wasn't a bug i used > > "addressed" instead of "fixed". > > I would say it "addresses" a ticket if the ticket is valid but the

Re: [FFmpeg-devel] [PATCH] avcodec: only warn about hwaccel with frame threads

2016-02-06 Thread Clément Bœsch
On Sat, Feb 06, 2016 at 06:37:56PM +0100, Mats Peterson wrote: > On 02/06/2016 06:25 PM, wm4 wrote: > >A regression in VLC. Make them fix it. > > > > > VLC = Very Lame Crap. Just kidding. > Please refrain from such comment, even for "kidding". Regards, -- Clément B. signature.asc Descripti

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Clément Bœsch
On Sat, Feb 06, 2016 at 04:26:43PM +0100, Andreas Cadhalpun wrote: > On 03.02.2016 00:41, Andreas Cadhalpun wrote: > > I see, it passes '-b $(SRC_PATH)' to lcov, which is now unnecessary/harmful > > for out-of-tree builds using the src link. > > The first attached patch changes it to simply use the

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

2016-02-06 Thread Paul B Mahol
On 2/5/16, Paul B Mahol wrote: > On 2/5/16, Paul B Mahol wrote: >> Hi, >> >> patch attached. >> > > Improved version attached. > Much better version attached. From 9d9cbbb1c92bbd415bcfb45b4b93346ed66ee88e Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 5 Feb 2016 12:54:06 +0100 Subject:

Re: [FFmpeg-devel] [FFmpeg-cvslog] build: use a link instead of changing current directory when compiling

2016-02-06 Thread Clément Bœsch
On Wed, Feb 03, 2016 at 12:41:01AM +0100, Andreas Cadhalpun wrote: [...] > The second patch was necessary to fix the following error: > lcov: ERROR: cannot write to coverage.info! > > Does it work for you without that patch? > I confirm it fails the same way as you without the second patch. --

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Paul B Mahol
On 2/6/16, Moritz Barsnick wrote: > On Sat, Feb 06, 2016 at 12:47:25 +0100, Paul B Mahol wrote: >> patch attached. > > Nice. (Untested.) > > > >> +Manipulate frames metadata. > Either: > frame metadata [using this as a generic term] > or: > frames' metadata > > I prefer the former. > >

Re: [FFmpeg-devel] [PATCH] avfilter: add metadata filters

2016-02-06 Thread Paul B Mahol
On 2/6/16, Paul B Mahol wrote: > Hi, > > patch attached. > Improved version attached. From 46283dd6b38ce8f1346249707986212546026e39 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sat, 6 Feb 2016 11:19:45 +0100 Subject: [PATCH] avfilter: add metadata filters Signed-off-by: Paul B Mahol ---

Re: [FFmpeg-devel] [PATCH] avformat/mp3enc: Assert that the header we assembled is valid

2016-02-06 Thread Michael Niedermayer
On Sat, Feb 06, 2016 at 02:46:19PM -0500, Ronald S. Bultje wrote: > Hi, > > On Sat, Feb 6, 2016 at 10:50 AM, Nicolas George wrote: > > > L'octidi 18 pluviôse, an CCXXIV, James Almer a écrit : > > > The couple times i dealt with a trac ticket that wasn't a bug i used > > > "addressed" instead of

Re: [FFmpeg-devel] [PATCH 06/12] avformat/ffmenc: use ff_parse_creation_time_metadata

2016-02-06 Thread Michael Niedermayer
On Sat, Feb 06, 2016 at 08:13:17PM +0100, Marton Balint wrote: > FYI this muxer bails out on parse error and not just warn the user. > > Signed-off-by: Marton Balint > --- > libavformat/ffmenc.c | 8 ++-- > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/libavformat/ffmenc.

Re: [FFmpeg-devel] [PATCH 01/12] avutil/parseutils: use microsecond precision when parsing "now" in av_parse_time()

2016-02-06 Thread Michael Niedermayer
On Sat, Feb 06, 2016 at 08:13:12PM +0100, Marton Balint wrote: > Use av_gettime() instead of time(0) for querying current time. > > Signed-off-by: Marton Balint > --- > libavutil/parseutils.c| 12 +++- > tests/ref/fate/parseutils | 4 ++-- > 2 files changed, 9 insertions(+), 7 delet

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-06 Thread Mats Peterson
On 02/06/2016 05:17 AM, Mats Peterson wrote: On 02/06/2016 05:05 AM, Mats Peterson wrote: This patch removes the monowhite switching code for 1 bpp raw AVI without a palette. I don't see any reason to keep it, since it's semantically incorrect to use monowhite for palettized data in my book, it

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-06 Thread Mats Peterson
On 02/07/2016 02:39 AM, Mats Peterson wrote: On 02/06/2016 05:17 AM, Mats Peterson wrote: On 02/06/2016 05:05 AM, Mats Peterson wrote: This patch removes the monowhite switching code for 1 bpp raw AVI without a palette. I don't see any reason to keep it, since it's semantically incorrect to use

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-06 Thread Mats Peterson
On 02/07/2016 02:45 AM, Mats Peterson wrote: The BMP code in FFmpeg handles this correctly by using pal8 exclusively for 1 bpp. Speaking of BMP, "ffmpeg -i 1bpp.bmp -vcodec rawvideo out.nut" will fail as well (since BMP correctly uses pal8), so if there ever was a "regression", it was there

[FFmpeg-devel] [PATCH] x86/cpu: set avxslow cpuflag on bdver4 and btver2 CPUs

2016-02-06 Thread James Almer
They are also slow when using 256 bit wide registers Signed-off-by: James Almer --- See https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=3a33f4ce8ea1efdebec7f7138d00e9be9a12d630 libavutil/x86/cpu.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/libavutil/x86/cpu

Re: [FFmpeg-devel] [PATCH 01/12] avutil/parseutils: use microsecond precision when parsing "now" in av_parse_time()

2016-02-06 Thread Marton Balint
On Sun, 7 Feb 2016, Michael Niedermayer wrote: On Sat, Feb 06, 2016 at 08:13:12PM +0100, Marton Balint wrote: Use av_gettime() instead of time(0) for querying current time. Signed-off-by: Marton Balint --- libavutil/parseutils.c| 12 +++- tests/ref/fate/parseutils | 4 ++-- 2 f

Re: [FFmpeg-devel] [PATCH] lavc/rawdec: Remove monowhite switching code for 1 bpp AVI without a palette

2016-02-06 Thread Mats Peterson
On 02/07/2016 03:26 AM, Mats Peterson wrote: On 02/07/2016 02:45 AM, Mats Peterson wrote: The BMP code in FFmpeg handles this correctly by using pal8 exclusively for 1 bpp. Speaking of BMP, "ffmpeg -i 1bpp.bmp -vcodec rawvideo out.nut" will fail as well (since BMP correctly uses pal8), so if