Re: [FFmpeg-devel] [PATCH] RELEASE_NOTES: fix WebVTT decoder typo (-> encoder)

2014-07-24 Thread Timothy Gu
On Thu, Jul 24, 2014 at 7:39 PM, Timothy Gu wrote: > Found-by: Clément Bœsch > Signed-off-by: Timothy Gu > --- > > I'm so sorry I didn't see Clément's comment on this during the patch > review stage… > > --- > RELEASE_NOTES | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) BTW this patch

Re: [FFmpeg-devel] [PATCH] lavf/avio: do not include bprint.h.

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 02:56:04PM +0200, Michael Niedermayer wrote: > On Thu, Jul 24, 2014 at 09:34:10AM +0200, Nicolas George wrote: > > C++ chokes on the definition of AVBPrint. > > Including avio.h from c++ code used to work. > > Fix trac ticket #3800. > > > > Signed-off-by: Nicolas George >

[FFmpeg-devel] [PATCH] RELEASE_NOTES: fix WebVTT decoder typo (-> encoder)

2014-07-24 Thread Timothy Gu
Found-by: Clément Bœsch Signed-off-by: Timothy Gu --- I'm so sorry I didn't see Clément's comment on this during the patch review stage… --- RELEASE_NOTES | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/RELEASE_NOTES b/RELEASE_NOTES index 0780595..6323cf5 100644 --- a/RELE

Re: [FFmpeg-devel] [PATCH] ffplay: remove dead code

2014-07-24 Thread Timothy Gu
On 07/24/2014 11:54 AM, Marton Balint wrote: > > On Thu, 24 Jul 2014, Michael Niedermayer wrote: > >> Hi >> >> On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote: >>> Signed-off-by: Timothy Gu >>> --- >>> ffplay.c | 2 -- >>> 1 file changed, 2 deletions(-) >>> >>> diff --git a/ffplay.

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-07-24 Thread Timothy Gu
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/24/2014 04:20 PM, Michael Niedermayer wrote: > On Thu, Jul 24, 2014 at 01:17:23PM -0700, Timothy Gu wrote: [...] > >> The cons: - Anchor names are changed. > > are there any linkes that use these anchors ? Never mind, the anchors are preserv

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 01:17:23PM -0700, Timothy Gu wrote: [...] > The cons: > - Anchor names are changed. are there any linkes that use these anchors ? > > > > > also applying this patchset will not affect how our server > > generates the html, as it does not execute files from git. > > If i

Re: [FFmpeg-devel] [PATCH] web/style: Add a comment in the generated css file for the doc in the main repo

2014-07-24 Thread Andreas Cadhalpun
On 24.07.2014 08:06, db0company wrote: --- README |2 +- src/css_credit |6 ++ 2 files changed, 7 insertions(+), 1 deletions(-) create mode 100644 src/css_credit diff --git a/README b/README index 93854ee..2276820 100644 --- a/README +++ b/README @@ -29,7 +29,7 @@ To ge

Re: [FFmpeg-devel] [PATCH] Use makeinfo to generate html doc for the new website

2014-07-24 Thread Timothy Gu
On Jul 23, 2014 7:44 PM, "Michael Niedermayer" wrote: > > On Wed, Jul 23, 2014 at 05:20:10PM -0700, Timothy Gu wrote: > > On Sun, Jul 20, 2014 at 2:26 PM, Timothy Gu wrote: > > > texi2html is deprecated by upstream in favor of makeinfo/texi2any. See: > > > > > > - https://www.gnu.org/software/tex

Re: [FFmpeg-devel] [PATCH] ffplay: remove dead code

2014-07-24 Thread Marton Balint
On Thu, 24 Jul 2014, Michael Niedermayer wrote: Hi On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote: Signed-off-by: Timothy Gu --- ffplay.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ffplay.c b/ffplay.c index af0e199..8abbeae 100644 --- a/ffplay.c +++ b/ffplay.c @@ -775,

[FFmpeg-devel] [PATCH] x86: hevc_mc: replace simple leas by adds

2014-07-24 Thread Christophe Gisquet
Hi, the impact is relatively important (3 to 25 cycles, ie up to 2%) so I also include bench.patch if anybody wants to confirm the timings. Although I thought openhevc's MC code was faster, the benchmarked functions are faster in ffmpeg. I didn't investigate if there are any MC functions much slo

[FFmpeg-devel] [PATCH] lavfi/setpts: allow to parse a metadata tag as a timestamp.

2014-07-24 Thread Nicolas George
Signed-off-by: Nicolas George --- doc/filters.texi | 22 ++ libavfilter/setpts.c | 64 ++- libavfilter/version.h | 2 +- 3 files changed, 86 insertions(+), 2 deletions(-) diff --git a/doc/filters.texi b/doc/filters.texi index

Re: [FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-24 Thread Nicolas George
Le sextidi 6 thermidor, an CCXXII, Michael Niedermayer a écrit : > the 4 patch set seems to break fate > TESTswr-resample-s16p-48000-2626 > stddev:3.88 PSNR: 84.54 MAXDIFF: 15 bytes:96000/20416 > Test swr-resample-s16p-8000-44100 failed. Look at > tests/data/fate/swr-resample-s16

Re: [FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:39:42PM +0200, Nicolas George wrote: > Messaages are passed as frames with a negative format code, > only to filter that declare supporting them. > The only message for now is EOF; this mechanism augments > the current mechanism based on request_frame() returning > AVERRO

Re: [FFmpeg-devel] [PATCH] hevc: propagate error code from set_sps()

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 17, 2014 at 06:26:44PM -0700, Timothy Gu wrote: > Signed-off-by: Timothy Gu > --- > libavcodec/hevc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Everything should be mad

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-07-24 Thread Nicolas George
Le sextidi 6 thermidor, an CCXXII, wm4 a écrit : > No, you can't do this. Yes, I can. > It breaks compatibility with Libav; No, it does not. Removing it would break compatibility. Deprecating it does nothing. > also what's even the purpose of this (API) change? Fixing a long-standing issue, se

Re: [FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-07-24 Thread wm4
On Thu, 24 Jul 2014 15:39:43 +0200 Nicolas George wrote: > Also deprecate adding a NULL frame to mark EOF. No, you can't do this. It breaks compatibility with Libav; also what's even the purpose of this (API) change? > TODO APIchanges entry, version bump. > > Signed-off-by: Nicolas George > -

[FFmpeg-devel] [PATCH 4/4] lavfi/vf_fps: accept EOF timestamp.

2014-07-24 Thread Nicolas George
This makes the FPS filter duplicate the last frame to take its duration into account, exactly like the other ones. TODO find corresponding trac ticket(s). Signed-off-by: Nicolas George --- libavfilter/vf_fps.c | 59 +++- 1 file changed, 40 inserti

[FFmpeg-devel] [PATCH 3/4] ffmpeg: use av_buffersrc_close().

2014-07-24 Thread Nicolas George
Signed-off-by: Nicolas George --- ffmpeg.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index 1c1a559..3ac6620 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -1741,12 +1741,9 @@ static int decode_audio(InputStream *ist, AVPacket *pkt, int *got_

[FFmpeg-devel] [PATCH 2/4] lavfi/buffersrc: add add av_buffersrc_close().

2014-07-24 Thread Nicolas George
Also deprecate adding a NULL frame to mark EOF. TODO APIchanges entry, version bump. Signed-off-by: Nicolas George --- libavfilter/buffersrc.c | 40 ++-- libavfilter/buffersrc.h | 15 +++ 2 files changed, 45 insertions(+), 10 deletions(-) diff --

[FFmpeg-devel] [PATCH 1/4] lavfi: add filter messages infrastructure.

2014-07-24 Thread Nicolas George
Messaages are passed as frames with a negative format code, only to filter that declare supporting them. The only message for now is EOF; this mechanism augments the current mechanism based on request_frame() returning AVERROR_EOF, with the advantage that the EOF message carries a timestamp. The me

Re: [FFmpeg-devel] [PATCH] ffplay: remove dead code

2014-07-24 Thread Michael Niedermayer
Hi On Thu, Jul 17, 2014 at 06:02:02PM -0700, Timothy Gu wrote: > Signed-off-by: Timothy Gu > --- > ffplay.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/ffplay.c b/ffplay.c > index af0e199..8abbeae 100644 > --- a/ffplay.c > +++ b/ffplay.c > @@ -775,8 +775,6 @@ static void blend_sub

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is wrong

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 03:15:56PM +0200, Carl Eugen Hoyos wrote: > On Thursday 24 July 2014 03:00:02 pm Michael Niedermayer wrote: > > On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote: > > > On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: > > > > Carl Eugen Hoyos ag.or.at

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is wrong

2014-07-24 Thread Carl Eugen Hoyos
On Thursday 24 July 2014 03:00:02 pm Michael Niedermayer wrote: > On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote: > > On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: > > > Carl Eugen Hoyos ag.or.at> writes: > > > > Hi! > > > > > > > > Attached patch fixes ticket #3777 fo

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is?wrong

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote: > On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: > > Carl Eugen Hoyos ag.or.at> writes: > > > Hi! > > > > > > Attached patch fixes ticket #3777 for me, analyzed by Oliver Fromme. > > > > What about streaming growing files?

Re: [FFmpeg-devel] [PATCH] lavf/avio: do not include bprint.h.

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 09:34:10AM +0200, Nicolas George wrote: > C++ chokes on the definition of AVBPrint. > Including avio.h from c++ code used to work. > Fix trac ticket #3800. > > Signed-off-by: Nicolas George > --- > libavformat/avio.h| 3 +-- > libavformat/aviobuf.c | 1 + > 2 files ch

Re: [FFmpeg-devel] Test of mail system pllease ignore

2014-07-24 Thread Michael Niedermayer
Hi Tim On Thu, Jul 24, 2014 at 12:48:11PM +0100, tim nicholson wrote: > Should munge the from headers for the record, works great also, thanks alot for working on / fixing yahoo/DMARC/DKIM on ffmpeg/mplayer MLs [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH] web/style: Add a comment in the generated css file for the doc in the main repo

2014-07-24 Thread Michael Niedermayer
On Thu, Jul 24, 2014 at 08:06:43AM +0200, db0company wrote: > --- > README |2 +- > src/css_credit |6 ++ > 2 files changed, 7 insertions(+), 1 deletions(-) > create mode 100644 src/css_credit applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF1336

[FFmpeg-devel] Test of mail system pllease ignore

2014-07-24 Thread tim nicholson
Should munge the from headers -- Tim. Key Fingerprint 38CF DB09 3ED0 F607 8B67 6CED 0C0B FC44 8B0B FC83 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is wrong

2014-07-24 Thread Carl Eugen Hoyos
On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote: > Carl Eugen Hoyos ag.or.at> writes: > > Hi! > > > > Attached patch fixes ticket #3777 for me, analyzed by Oliver Fromme. > > What about streaming growing files? It's pretty good to know number of > frames of the complete mp3 at start of pla

Re: [FFmpeg-devel] [PATCH]Print a warning if the invalid tiff tag type 0 is found in an exif tag

2014-07-24 Thread Thilo Borgmann
Am 22.07.14 17:43, schrieb Carl Eugen Hoyos: > Hi! > > FFmpeg currently requests a sample if an exif tag with tag type 0 is found. > Attached patch prints a warning instead, fixes ticket #3792. > > Please comment, Carl Eugen ok. -Thilo ___ ffmpeg-dev

[FFmpeg-devel] [PATCH] lavf/avio: do not include bprint.h.

2014-07-24 Thread Nicolas George
C++ chokes on the definition of AVBPrint. Including avio.h from c++ code used to work. Fix trac ticket #3800. Signed-off-by: Nicolas George --- libavformat/avio.h| 3 +-- libavformat/aviobuf.c | 1 + 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/avio.h b/libavfor