Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Reimar Döffinger
On 21.01.2015, at 07:17, Agatha Hu wrote: > On 2015/1/18 4:01, Philip Langdale wrote: >> There is a long sad story behind all this, but it's somewhat ambiguous as to >> whether DVD content should be treated as 720 pixels wide or 704 pixels, with >> 16 pixels cut off. If you decide is should be 704

Re: [FFmpeg-devel] Post-processing filter Documentation

2015-01-21 Thread Stefano Sabatini
On date Tuesday 2015-01-20 22:34:15 +0530, Arwa Arif encoded: > > > > Still missing benchmarks and pp filter covering. > > > > > What is meant by pp filter covering? I mean that there are no images obtained by applying pp (and uspp as well). > Do you want benchmark for each image? And by benchm

Re: [FFmpeg-devel] [PATCH] Port mp=eq/eq2 to FFmpeg

2015-01-21 Thread Stefano Sabatini
On date Tuesday 2015-01-20 22:30:54 +0530, Arwa Arif encoded: [...] > > Add also an entry to add support to .process_command(). You can > > implement it in a later patch. > > > > > What is meant by adding support to process_command? grep for .process_command in the libavfilter dir, and you will kn

Re: [FFmpeg-devel] OpenCL device selection documentation

2015-01-21 Thread Stefano Sabatini
On date Monday 2015-01-19 12:00:06 +, Gupta, Maneesh encoded: > Hi, > > I was searching the FFMPEG documentation on enabling OpenCL & > selecting/specifying an OpenCL device. Here is what I could find: > > 1) Enabling OpenCL while building OpenCL: > https://trac.ffmpeg.org/wiki/HWAccelIntro

Re: [FFmpeg-devel] [PATCH] ffprobe: fix memleaks on errors

2015-01-21 Thread Stefano Sabatini
On date Wednesday 2015-01-21 05:47:13 +0100, Michael Niedermayer encoded: > Found-by: Andreas Cadhalpun > Signed-off-by: Michael Niedermayer > --- > ffprobe.c | 19 --- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git a/ffprobe.c b/ffprobe.c > index faddc16..3

[FFmpeg-devel] [PATCH] doc/APIchanges: Add av_find_best_pix_fmt_of_2() and av_get_pix_fmt_loss()

2015-01-21 Thread Michael Niedermayer
Found-by: wm4 Signed-off-by: Michael Niedermayer --- doc/APIchanges |3 +++ 1 file changed, 3 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index 7d25679..ca923de 100644 --- a/doc/APIchanges +++ b/doc/APIchanges @@ -312,6 +312,9 @@ API changes, most recent first: 2014-04-29 - x

Re: [FFmpeg-devel] [PATCH] doc/APIchanges: Add av_find_best_pix_fmt_of_2() and av_get_pix_fmt_loss()

2015-01-21 Thread wm4
On Wed, 21 Jan 2015 12:29:04 +0100 Michael Niedermayer wrote: > Found-by: wm4 > Signed-off-by: Michael Niedermayer > --- > doc/APIchanges |3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/APIchanges b/doc/APIchanges > index 7d25679..ca923de 100644 > --- a/doc/APIchanges > +++

Re: [FFmpeg-devel] Post-processing filter Documentation

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 09:42:03AM +0100, Stefano Sabatini wrote: > On date Tuesday 2015-01-20 22:34:15 +0530, Arwa Arif encoded: > > > > > > Still missing benchmarks and pp filter covering. > > > > > > > > > What is meant by pp filter covering? > > I mean that there are no images obtained by app

Re: [FFmpeg-devel] [PATCH] ffprobe: fix memleaks on errors

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 10:41:11AM +0100, Stefano Sabatini wrote: > On date Wednesday 2015-01-21 05:47:13 +0100, Michael Niedermayer encoded: > > Found-by: Andreas Cadhalpun > > Signed-off-by: Michael Niedermayer > > --- > > ffprobe.c | 19 --- > > 1 file changed, 12 insertions

Re: [FFmpeg-devel] [PATCH] avcodec/nvenc: add repeatSPSPPS flag for streaming

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 08:32:07AM +0100, Timo Rothenpieler wrote: > >Still using attachment to prevent word-wrapping. > >For video streaming, repeatSPSPPS flag should be set as 1 besides > >disableSPSPPS=0, elsewise if running ffmpeg before vlc/ffmpeg, there > >will be no video header, decoder can

Re: [FFmpeg-devel] [PATCH] doc/APIchanges: Add av_find_best_pix_fmt_of_2() and av_get_pix_fmt_loss()

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 12:59:03PM +0100, wm4 wrote: > On Wed, 21 Jan 2015 12:29:04 +0100 > Michael Niedermayer wrote: > > > Found-by: wm4 > > Signed-off-by: Michael Niedermayer > > --- > > doc/APIchanges |3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/doc/APIchanges b/doc

Re: [FFmpeg-devel] [PATCH] avcodec/h264: Partially decode and display single fields try #2

2015-01-21 Thread Kieran Kunhya
> yes, its overall 3 lines shorter as well > ive locally changed it to this: Thanks, much easier to understand. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] Port mp=eq/eq2 to FFmpeg

2015-01-21 Thread arwa arif
> > I still expect that eq and eq2 should have the same performances, > since the adjust callback is set depending on the parameter values. So > we should have a single eq filter. > > Please investigate about why you get different benchmark values. > I used this command: ffmpeg -benchmark -i matr

[FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Derek Buitenhuis
Signed-off-by: Derek Buitenhuis --- libavcodec/libx265.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c index 923c750..5e4931a 100644 --- a/libavcodec/libx265.c +++ b/libavcodec/libx265.c @@ -226,6 +226,11 @@ static int libx265_encode_frame(AV

[FFmpeg-devel] [PATCH] avformat/dashenc: fix format string generation

2015-01-21 Thread Rodger Combs
--- libavformat/dashenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index fa497fd..6e5a41b 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -276,7 +276,7 @@ static DASHTmplId dash_read_tmpl_id(const char *iden

Re: [FFmpeg-devel] [PATCH] avformat/dashenc: fix format string generation

2015-01-21 Thread Derek Buitenhuis
On 1/21/2015 3:36 PM, Rodger Combs wrote: > -const char *number_format = DASH_TMPL_ID_TIME ? "lld" : "d"; > +const char *number_format = (id_type == DASH_TMPL_ID_TIME) ? > "lld" : "d"; Why not inttypes.h format? - Derek ___ ffmp

[FFmpeg-devel] [PATCH] avformat/mov: Stricter sanity checks on the display_matrix

2015-01-21 Thread Michael Niedermayer
Fixes "broken" tkhd Found-by: koda Signed-off-by: Michael Niedermayer --- libavformat/mov.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 556411f..6851947 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2874,7 +2874

Re: [FFmpeg-devel] [PATCH] avfilter/vf_lenscorrection: support 8bit planar RGB formats

2015-01-21 Thread Michael Niedermayer
On Tue, Jan 20, 2015 at 05:17:16PM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavfilter/vf_lenscorrection.c | 1 + > 1 file changed, 1 insertion(+) LGTM [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No great genius has ever existed wi

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 03:18:48PM +, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis > --- > libavcodec/libx265.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c > index 923c750..5e4931a 100644 > --- a/libavcodec/libx265.c

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Derek Buitenhuis
On 1/21/2015 4:28 PM, Michael Niedermayer wrote: > this is missing pic==NULL checks or should be under the if() above > or something like that No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c: https://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavcodec/libx264.c;h=4da0e

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Timo Rothenpieler
> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c: Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it will just crash. signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing lis

Re: [FFmpeg-devel] Parallelization

2015-01-21 Thread Derek Buitenhuis
On 12/31/2014 3:53 AM, Derek Buitenhuis wrote: > On 12/31/2014 10:00 PM, David Ison wrote: >> 3. I am planning initial implementations using OpenMP. There is good >> support on Gnu, LLVM and other compilers >> . > > I think it is probably unlikely that O

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Derek Buitenhuis
On 1/21/2015 4:46 PM, Timo Rothenpieler wrote: >> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like libx264.c: > > Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it > will just crash. I meant to send a reply saying I misread, but responded to the wrong mail

Re: [FFmpeg-devel] Parallelization

2015-01-21 Thread Derek Buitenhuis
On 1/21/2015 4:46 PM, Derek Buitenhuis wrote: > On 12/31/2014 3:53 AM, Derek Buitenhuis wrote: >> On 12/31/2014 10:00 PM, David Ison wrote: >>> 3. I am planning initial implementations using OpenMP. There is good >>> support on Gnu, LLVM and other compilers >>>

Re: [FFmpeg-devel] [PATCH] avfilter/vf_lenscorrection: support 8bit planar RGB formats

2015-01-21 Thread Paul B Mahol
On 1/21/15, Michael Niedermayer wrote: > On Tue, Jan 20, 2015 at 05:17:16PM +, Paul B Mahol wrote: >> Signed-off-by: Paul B Mahol >> --- >> libavfilter/vf_lenscorrection.c | 1 + >> 1 file changed, 1 insertion(+) > > LGTM applied > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6

Re: [FFmpeg-devel] [PATCH] Port mp=eq/eq2 to FFmpeg

2015-01-21 Thread Paul B Mahol
On 1/21/15, arwa arif wrote: >> >> I still expect that eq and eq2 should have the same performances, >> since the adjust callback is set depending on the parameter values. So >> we should have a single eq filter. >> >> Please investigate about why you get different benchmark values. >> > > I used

[FFmpeg-devel] [PATCH] libavformat/img2dec: fix warning when !HAVE_GLOB

2015-01-21 Thread Christophe Gisquet
Hi, the attached patch fixes a warning under MinGW (no idea about msys2). -- Christophe ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] ffmpeg_opt: expand format for strftime

2015-01-21 Thread Christophe Gisquet
Hi, another warning under MinGW fixed, because the format specifiers are not supported, cf. for instance: https://msdn.microsoft.com/en-us/library/fe06s4ak.aspx -- Christophe From 6d3e4eb5df0eacf3df011dbe3e05a82f814f63b1 Mon Sep 17 00:00:00 2001 From: Christophe Gisquet Date: Tue, 20 Jan 2015 2

Re: [FFmpeg-devel] [PATCH] libavformat/img2dec: fix warning when !HAVE_GLOB

2015-01-21 Thread Christophe Gisquet
2015-01-21 20:06 GMT+01:00 Christophe Gisquet : > Hi, > > the attached patch fixes a warning under MinGW (no idea about msys2). With aforementioned patch. -- Christophe From 54b6d1b588d4a15f51345f8ca1fc5b8ab0660b7e Mon Sep 17 00:00:00 2001 From: Christophe Gisquet Date: Tue, 20 Jan 2015 22:03:1

Re: [FFmpeg-devel] [PATCH] libx265: Pass through user-set frame type

2015-01-21 Thread Reimar Döffinger
On Wed, Jan 21, 2015 at 04:52:33PM +, Derek Buitenhuis wrote: > On 1/21/2015 4:46 PM, Timo Rothenpieler wrote: > >> No? It has X265_TYPE_AUTO as the fallback if it is NULL. Just like > >> libx264.c: > > > > Is falls back to that if pic->pict_type is NULL, but if pic is NULL, it > > will just

Re: [FFmpeg-devel] [PATCH] Port mp=eq/eq2 to FFmpeg

2015-01-21 Thread arwa arif
On Thu, Jan 22, 2015 at 12:09 AM, Paul B Mahol wrote: > On 1/21/15, arwa arif wrote: > >> > >> I still expect that eq and eq2 should have the same performances, > >> since the adjust callback is set depending on the parameter values. So > >> we should have a single eq filter. > >> > >> Please in

Re: [FFmpeg-devel] [PATCH] avfilter/vf_showinfo: show timebase & framerate too

2015-01-21 Thread Michael Niedermayer
On Sat, Jan 17, 2015 at 03:17:13PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavfilter/vf_showinfo.c | 28 +++- > 1 file changed, 27 insertions(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BA

Re: [FFmpeg-devel] [PATCH] libavformat/img2dec: fix warning when !HAVE_GLOB

2015-01-21 Thread Michael Niedermayer
On Wed, Jan 21, 2015 at 08:11:19PM +0100, Christophe Gisquet wrote: > 2015-01-21 20:06 GMT+01:00 Christophe Gisquet : > > Hi, > > > > the attached patch fixes a warning under MinGW (no idea about msys2). > > With aforementioned patch. > > -- > Christophe > img2dec.c |4 +++- > 1 file chang

[FFmpeg-devel] [PATCH] configure: add --build-date and --build-time options

2015-01-21 Thread Andreas Cadhalpun
Hi, Debian tries to make binary packages build reproducible [1]. FFmpeg embeds date and time in the program and documentation, which makes it hard to have binary reproducible packages. The attached patch allows to specify the build date and time to use with a configure option, solving this p

[FFmpeg-devel] [PATCH] configure: use ar and ranlib in deterministic mode

2015-01-21 Thread Andreas Cadhalpun
Hi, ar and ranlib add timestamp and uid/gid for the object files in static libraries. If used in deterministic mode, these are set to zero, making the binaries reproducible. Attached patch achieves that. Best regards, Andreas >From 1dfc47cb27ae6f3d88b8b7ca690b980842003ee7 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH] librtmp: Allow changing the socket send buffer size

2015-01-21 Thread Michael Niedermayer
On Mon, Jan 19, 2015 at 09:13:15PM -0600, Brian Brice wrote: > While streaming with librtmp within Windows, the 8K default size of > SO_SNDBUF affects network performance negatively. Being able to adjust > this value can improve things greatly. > > This patch adds the -rtmp_buffer_size option to l

Re: [FFmpeg-devel] [PATCH] Port mp=eq/eq2 to FFmpeg

2015-01-21 Thread James Almer
On 21/01/15 5:08 PM, arwa arif wrote: > diff --git a/libavfilter/x86/Makefile b/libavfilter/x86/Makefile > index b93154e..8222e3f 100644 > --- a/libavfilter/x86/Makefile > +++ b/libavfilter/x86/Makefile > @@ -1,3 +1,4 @@ > +OBJS-$(CONFIG_EQ_FILTER) += x86/vf_eq.o > OBJS-$(CONFI

Re: [FFmpeg-devel] [PATCH] configure: add --build-date and --build-time options

2015-01-21 Thread Timothy Gu
On Wed Jan 21 2015 at 2:11:09 PM Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > Hi, > > Debian tries to make binary packages build reproducible [1]. > > FFmpeg embeds date and time in the program and documentation, which > makes it hard to have binary reproducible packages. > > The

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Agatha Hu
On 2015/1/21 16:00, Reimar Döffinger wrote: On 21.01.2015, at 07:17, Agatha Hu wrote: On 2015/1/18 4:01, Philip Langdale wrote: Here's the reply from NVENC engineers It is not same thing as SAR. It is the display aspect ratio i.e width/height = DAR/SAR The calculation below should be li

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Philip Langdale
On Thu, 22 Jan 2015 12:15:44 +0800 Agatha Hu wrote: > We will fix the issue in driver, overscan compensation will be > applied to input DAR *only* if the DAR is 4:3 or 16:9, otherwise > won't unnecessarily modify the aspect ratio for resolutions like > 720x480 and 720x576. > The fix will be on fu

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Reimar Döffinger
On 22.01.2015, at 05:15, Agatha Hu wrote: > On 2015/1/21 16:00, Reimar Döffinger wrote: >> On 21.01.2015, at 07:17, Agatha Hu wrote: >>> On 2015/1/18 4:01, Philip Langdale wrote: > >>> >>> Here's the reply from NVENC engineers >>> It is not same thing as SAR. It is the display aspect rati

[FFmpeg-devel] Duration/bitrate estimation

2015-01-21 Thread Marcus Johnson
What functions are used to estimate the duration and bit rate of a file? I'd like to add support for these features for the DTS format, and would rather not dig around the source looking for it. thanks in advance. ___ ffmpeg-devel mailing list ffmpeg-dev

Re: [FFmpeg-devel] [PATCH] nvenc: Compensate for hardware trying to mess with aspect ratio of DVD content.

2015-01-21 Thread Agatha Hu
在 2015/1/22 12:29, Philip Langdale 写道: On Thu, 22 Jan 2015 12:15:44 +0800 Agatha Hu wrote: We will fix the issue in driver, overscan compensation will be applied to input DAR *only* if the DAR is 4:3 or 16:9, otherwise won't unnecessarily modify the aspect ratio for resolutions like 720x480 an

Re: [FFmpeg-devel] [shan...@gmail.com: [libav-devel] Decoder for CEA-608 and World System Teletext]

2015-01-21 Thread Anshul
On 01/22/2015 12:12 PM, Shan wrote: On 2015-01-20T11:52:39+0100 u...@pkh.me relayed: The patches are welcome though, it might be easier to work with. Here's a base64 patch for the CEA-608, EIA-708, Teletext, DVB VBI and MXF VANC decoder, as well as some other related changes as well as suppo