Re: [FFmpeg-devel] [PATCH] avutil/pixelutils: add small buffers tests

2014-08-31 Thread Clément Bœsch
On Sun, Aug 31, 2014 at 07:45:24PM +0200, Michael Niedermayer wrote: On Sun, Aug 31, 2014 at 06:04:21PM +0200, Clément Bœsch wrote: --- libavutil/pixelutils.c| 98 +-- tests/ref/fate/pixelutils | 12 ++ 2 files changed, 89 insertions

Re: [FFmpeg-devel] [PATCH] build: simplify libwebp check

2014-09-01 Thread Clément Bœsch
On Mon, Sep 01, 2014 at 09:36:45AM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: From: Clément Bœsch clement at stupeflix.com Check is simpler, faster, and explicits the required version. Tested (compilation and runtime) with libwebp v0.2.0, v0.2.1 and v0.4.1

Re: [FFmpeg-devel] [PATCH] build: simplify libwebp check

2014-09-01 Thread Clément Bœsch
On Mon, Sep 01, 2014 at 11:33:34PM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: From: Clément Bœsch clement at stupeflix.com Check is simpler, faster, and explicits the required version. faster? Please remove this if I don't miss anything again

Re: [FFmpeg-devel] [PATCH] rl.h: Use on-stack temporary VLC tables instead of having them static.

2014-09-01 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 01:59:50AM +0200, Michael Niedermayer wrote: On Mon, Sep 01, 2014 at 11:25:57PM +0200, Reimar Döffinger wrote: On Mon, Sep 01, 2014 at 10:59:51PM +0200, Reimar Döffinger wrote: On Mon, Sep 01, 2014 at 10:19:36PM +0200, Michael Niedermayer wrote: On Mon, Sep 01,

Re: [FFmpeg-devel] [FFmpeg-cvslog] 8svx: Return proper error codes

2014-09-02 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 09:54:31PM +0200, Gabriel Dume wrote: ffmpeg | branch: master | Gabriel Dume gabriel.dd...@gmail.com | Mon Sep 1 15:18:57 2014 -0400| [74512f7e369da40e1148c92b58cd8e59f7737b8f] | committer: Diego Biurrun 8svx: Return proper error codes Signed-off-by: Diego

Re: [FFmpeg-devel] [PATCH 1/4] avformat/assdec: UTF-16 support

2014-09-02 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 08:56:09PM +0200, wm4 wrote: Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext. It also can work on a static buffer, needed for format probing. The

Re: [FFmpeg-devel] [PATCH 2/4] avformat/srtdec: UTF-16 support

2014-09-02 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 08:56:10PM +0200, wm4 wrote: --- libavformat/srtdec.c| 22 +++-- libavformat/subtitles.c | 51 +++-- libavformat/subtitles.h | 33 +--- 3 files changed, 91 insertions(+), 15

Re: [FFmpeg-devel] [PATCH 3/4] avformat/samidec: UTF-16 support

2014-09-02 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 08:56:11PM +0200, wm4 wrote: ff_smil_extract_next_chunk() is still used by RealText. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat/subtitles.h | 5 + 3 files changed, 27 insertions(+), 10

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-09-03 Thread Clément Bœsch
On Wed, Sep 03, 2014 at 04:38:15PM +0200, Clément Bœsch wrote: On Sun, Aug 31, 2014 at 02:35:40PM +0200, Andreas Cadhalpun wrote: On 28.08.2014 20:06, Nicolas George wrote: Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : I like this colour of the shed. ;) * move lena.pnm

Re: [FFmpeg-devel] [PATCH] SSE2 version of vf_idet's filter_line()

2014-09-03 Thread Clément Bœsch
On Wed, Sep 03, 2014 at 07:05:48PM +0200, Pascal Massimino wrote: [...] +punpcklbw m3, m_zero +punpckhbw m4, m_zero + +paddswm0, m3 +paddswm1, m4 + +movq m3, [bq+indexq*1] +movq m4, m3 +punpcklbw m3, m_zero +punpckhbw

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Clément Bœsch
On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi, should have been LGPL. Kudos to Carl for spotting that. /skal From cd067fd2f7ae77b6be950fe2e841b9c285208612 Mon Sep 17 00:00:00 2001 From: skal pascal.massim...@gmail.com While at it, can you fix your username in git

Re: [FFmpeg-devel] [PATCH] fix vf_idet new files' license header - LGPL

2014-09-05 Thread Clément Bœsch
On Fri, Sep 05, 2014 at 12:01:20PM +0200, Pascal Massimino wrote: On Fri, Sep 5, 2014 at 11:57 AM, Stefano Sabatini stefa...@gmail.com wrote: On date Friday 2014-09-05 11:50:35 +0200, Clément Bœsch encoded: On Fri, Sep 05, 2014 at 11:47:42AM +0200, Pascal Massimino wrote: Hi

Re: [FFmpeg-devel] [PATCH v2 1/6] avformat/assdec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:22PM +0200, wm4 wrote: Use the UTF-16 BOM to detect UTF-16 encoding. Convert the file contents to UTF-8 on the fly using FFTextReader, which acts as converting wrapper around AVIOContext. It also can work on a static buffer, needed for format probing. The

Re: [FFmpeg-devel] [PATCH v2 3/6] avformat/samidec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:24PM +0200, wm4 wrote: ff_smil_extract_next_chunk() is still used by RealText. --- No changes. --- libavformat/samidec.c | 17 ++--- libavformat/subtitles.c | 15 --- libavformat/subtitles.h | 5 + 3 files changed, 27

Re: [FFmpeg-devel] [PATCH v2 4/6] avformat/realtextdec: UTF-16 support

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:25PM +0200, wm4 wrote: Also remove ff_smil_extract_next_chunk - this was the last user of it. --- No changes. --- libavformat/realtextdec.c | 17 ++--- libavformat/subtitles.c | 9 - libavformat/subtitles.h | 5 - 3 files

Re: [FFmpeg-devel] [PATCH v2 5/6] avformat/srtdec: speed up probing

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:26PM +0200, wm4 wrote: --- As suggested by ubitux. Note that the change to ff_subtitles_read_line() already cuts down the probing time a lot, so maybe this is not needed. --- libavformat/srtdec.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [FFmpeg-devel] [PATCH v2 6/6] avformat/subtitles: guess UTF-16 if no BOM present

2014-09-05 Thread Clément Bœsch
On Thu, Sep 04, 2014 at 10:40:27PM +0200, wm4 wrote: --- Follows reimar's suggestion for detecting UTF-16. If the detection goes wrong, probing the format won't succeed, so this should not break anything. I didn't use ffio_ensure_seekback), because this apparently reallocates the buffer -

Re: [FFmpeg-devel] [PATCH 3/3] web/documentation: Simplify Components Documentation links

2014-09-06 Thread Clément Bœsch
On Sat, Sep 06, 2014 at 09:12:47AM -0800, Lou Logan wrote: On Sat, Sep 6, 2014, at 04:37 AM, db0company wrote: --- src/documentation | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) I don't have a strong opinion about this patch but there are two things I

[FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-07 Thread Clément Bœsch
--- libavformat/matroska.c| 6 libavformat/matroskadec.c | 80 --- libavformat/matroskaenc.c | 79 -- libavformat/version.h | 3 -- 4 files changed, 168 deletions(-) diff --git

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-09 Thread Clément Bœsch
On Sun, Sep 07, 2014 at 10:18:50PM +0200, wm4 wrote: On Sun, 7 Sep 2014 22:03:26 +0200 Clément Bœsch u...@pkh.me wrote: --- libavformat/matroska.c| 6 libavformat/matroskadec.c | 80 --- libavformat/matroskaenc.c | 79

[FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 4fa1174..44273cb 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -80,7 +80,9 @@ static const int

[FFmpeg-devel] [PATCH 2/3] avfilter/ass: better log level mapping

2014-09-11 Thread Clément Bœsch
--- libavfilter/vf_subtitles.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c index 44273cb..c9c21a7 100644 --- a/libavfilter/vf_subtitles.c +++ b/libavfilter/vf_subtitles.c @@ -68,14 +68,14 @@ typedef

Re: [FFmpeg-devel] suggested patch: avfilter/vf_subtitles: add support for subtitles font scaling

2014-09-11 Thread Clément Bœsch
On Thu, Sep 11, 2014 at 09:21:03PM +0200, wm4 wrote: On Thu, 11 Sep 2014 03:58:07 +0400 Sergey serg...@mail.ru wrote: Hello. Recently I used ffmpeg to embed subtitles, and I needed to scale them. I thought original_size option scales subtitles, but it does not. So I wrote a short

Re: [FFmpeg-devel] [FFmpeg-cvslog] doc: document the addition of the AVProbeData.mime_type field and it' s implications

2014-09-13 Thread Clément Bœsch
On Sat, Sep 13, 2014 at 12:53:21AM +0200, Andreas Cadhalpun wrote: ffmpeg | branch: master | Andreas Cadhalpun andreas.cadhal...@googlemail.com | Fri Sep 12 18:18:42 2014 +0200| [d5e802609a0046441798cdbd137c96e4aa912390] | committer: Michael Niedermayer doc: document the addition of the

[FFmpeg-devel] [PATCH] avformat/assenc: honor ReadOrder

2014-09-13 Thread Clément Bœsch
--- libavformat/assenc.c | 89 +--- 1 file changed, 85 insertions(+), 4 deletions(-) diff --git a/libavformat/assenc.c b/libavformat/assenc.c index 9fb9f23..acc21b9 100644 --- a/libavformat/assenc.c +++ b/libavformat/assenc.c @@ -19,12 +19,22 @@

Re: [FFmpeg-devel] [PATCH] avformat/assenc: honor ReadOrder

2014-09-13 Thread Clément Bœsch
On Sat, Sep 13, 2014 at 10:10:39PM +0200, Clément Bœsch wrote: --- Now looking from the last one added instead of the beginning for ZOMG speed --- libavformat/assenc.c | 107 +-- 1 file changed, 103 insertions(+), 4 deletions(-) Applied

[FFmpeg-devel] [PATCH] Changelog/ReleaseNotes: prepare for 2.4

2014-09-14 Thread Clément Bœsch
From: Clément Bœsch clem...@stupeflix.com --- Changelog | 2 +- RELEASE_NOTES | 19 ++- 2 files changed, 15 insertions(+), 6 deletions(-) diff --git a/Changelog b/Changelog index c5185f7..480a298 100644 --- a/Changelog +++ b/Changelog @@ -5,7 +5,7 @@ version next

[FFmpeg-devel] [PATCH] doc/APIchanges: attempt to split releases

2014-09-14 Thread Clément Bœsch
Based on several git show origin/release/X.Y:doc/APIchanges. --- doc/APIchanges | 42 +- 1 file changed, 37 insertions(+), 5 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 1a0ba78..eaac107 100644 --- a/doc/APIchanges +++ b/doc/APIchanges

[FFmpeg-devel] [PATCH 1/2] Kill timed SRT

2014-09-14 Thread Clément Bœsch
--- libavcodec/srtdec.c | 33 +++ libavcodec/srtenc.c | 49 ++- libavformat/matroska.c| 1 - libavformat/matroskaenc.c | 45 +-- libavformat/srtenc.c | 11

Re: [FFmpeg-devel] [PATCH 17/18] avfilter: remove obsolete FF_API_DRAWTEXT_OLD_TIMELINE cruft

2014-09-14 Thread Clément Bœsch
On Sun, Sep 14, 2014 at 10:46:19PM -0300, James Almer wrote: Signed-off-by: James Almer jamr...@gmail.com --- libavfilter/version.h | 3 --- libavfilter/vf_drawtext.c | 43 --- 2 files changed, 46 deletions(-) I don't mind but we *may* want to

Re: [FFmpeg-devel] [WIP] lavfi: port mp/eq to a regular filter

2014-09-15 Thread Clément Bœsch
On Mon, Sep 15, 2014 at 12:46:14AM +0200, James Darnley wrote: --- Still a couple of things on the todo list (see the top of the new file) but I thought I would send the basics along for people to nit pick. configure| 1 + libavfilter/Makefile | 1 +

Re: [FFmpeg-devel] Branchpoint tags

2014-09-15 Thread Clément Bœsch
On Mon, Sep 15, 2014 at 02:13:27PM +0200, wm4 wrote: On Sun, 14 Sep 2014 23:36:14 +0200 Michael Niedermayer michae...@gmx.at wrote: On Sun, Sep 14, 2014 at 07:31:42PM +0200, wm4 wrote: On Sun, 14 Sep 2014 19:12:56 +0200 Michael Niedermayer michae...@gmx.at wrote: Hi

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-15 Thread Clément Bœsch
On Thu, Sep 11, 2014 at 11:53:57PM +0200, Michael Niedermayer wrote: On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote: --- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_subtitles.c b/libavfilter

Re: [FFmpeg-devel] [PATCH] build: simplify libwebp check

2014-09-15 Thread Clément Bœsch
On Tue, Sep 02, 2014 at 06:59:42AM +0200, Clément Bœsch wrote: On Mon, Sep 01, 2014 at 11:33:34PM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: From: Clément Bœsch clement at stupeflix.com Check is simpler, faster, and explicits the required version

[FFmpeg-devel] [PATCH 1/2] avformat/flacenc: use av_clip() instead of av_clip_c()

2014-09-16 Thread Clément Bœsch
--- libavformat/flacenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/flacenc.c b/libavformat/flacenc.c index b3695a2..0eea942 100644 --- a/libavformat/flacenc.c +++ b/libavformat/flacenc.c @@ -95,7 +95,7 @@ static int flac_write_header(struct AVFormatContext

[FFmpeg-devel] [PATCH 2/2] sws: use av_clip() instead of av_clip_c()

2014-09-16 Thread Clément Bœsch
--- libswscale/swscale.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 59ead12..e54d448 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -804,9 +804,9 @@ static void xyz12Torgb48(struct SwsContext

Re: [FFmpeg-devel] [PATCH 1/2] avformat/flacenc: use av_clip() instead of av_clip_c()

2014-09-16 Thread Clément Bœsch
On Tue, Sep 16, 2014 at 04:53:42PM +0200, Michael Niedermayer wrote: On Tue, Sep 16, 2014 at 10:15:41AM +0200, Clément Bœsch wrote: --- libavformat/flacenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM Pushed, thanks -- Clément B. pgpnAHRJ8G715.pgp Description: PGP

Re: [FFmpeg-devel] [PATCH 1/3] avfilter/ass: make sure the log level are in available range

2014-09-16 Thread Clément Bœsch
On Mon, Sep 15, 2014 at 08:59:06PM +0200, Clément Bœsch wrote: On Thu, Sep 11, 2014 at 11:53:57PM +0200, Michael Niedermayer wrote: On Thu, Sep 11, 2014 at 09:13:29PM +0200, Clément Bœsch wrote: --- libavfilter/vf_subtitles.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion

Re: [FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-16 Thread Clément Bœsch
On Tue, Sep 16, 2014 at 10:44:32PM +0200, wm4 wrote: On Tue, 16 Sep 2014 21:37:08 +0200 Clément Bœsch u...@pkh.me wrote: On Sun, Sep 14, 2014 at 10:01:55AM +0200, Clément Bœsch wrote: After this the order from the original file is stored through readorder when doing ffmpeg -i input.ass

Re: [FFmpeg-devel] [FFmpeg-cvslog] x86/me_cmp: port mmxext and sse2 sad functions to yasm

2014-09-17 Thread Clément Bœsch
On Wed, Sep 17, 2014 at 11:41:32AM +0200, James Almer wrote: ffmpeg | branch: master | James Almer jamr...@gmail.com | Tue Sep 16 21:41:47 2014 -0300| [0456d169c469a79e305813d14c873fe698c8c572] | committer: Michael Niedermayer x86/me_cmp: port mmxext and sse2 sad functions to yasm Also

Re: [FFmpeg-devel] [PATCH] postproc: add basic deblock filter visualization support

2014-09-19 Thread Clément Bœsch
On Fri, Sep 19, 2014 at 03:44:31PM +0200, Michael Niedermayer wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libpostproc/postprocess.c | 36 +++- libpostproc/postprocess_internal.h |1 + libpostproc/postprocess_template.c |

[FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-19 Thread Clément Bœsch
After this the order from the original file is stored through readorder when doing ffmpeg -i input.ass -c copy output.mkv. And now that the ASS muxer honors the ReadOrder, extracting the ass back (without transcoding) restores the original order. TODO: micro bump --- libavformat/assdec.c | 25

Re: [FFmpeg-devel] [PATCH] avformat/assdec: output ASS packets

2014-09-19 Thread Clément Bœsch
On Fri, Sep 19, 2014 at 09:00:28PM +0200, wm4 wrote: On Fri, 19 Sep 2014 20:49:49 +0200 Clément Bœsch u...@pkh.me wrote: After this the order from the original file is stored through readorder when doing ffmpeg -i input.ass -c copy output.mkv. And now that the ASS muxer honors

Re: [FFmpeg-devel] [PATCH] avformat/assenc: mux all extradata at once

2014-09-20 Thread Clément Bœsch
On Sat, Sep 20, 2014 at 06:40:57PM +0200, Clément Bœsch wrote: Before this commit, the code was muxing up to the 2nd line after [Event] (assuming it to be the Format: line). The remaining are EventS* typo fixed locally generally Comment: directives which can stay in that place. mkvextract

[FFmpeg-devel] [PATCH 01/17] avcodec/jacosubdec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/jacosubdec.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/libavcodec/jacosubdec.c b/libavcodec/jacosubdec.c index b64fac8..ef847e2 100644 --- a/libavcodec/jacosubdec.c +++ b/libavcodec/jacosubdec.c @@ -168,6 +168,7 @@ static void

[FFmpeg-devel] in preparation of changing the internal form of decoded text subtitles

2014-09-20 Thread Clément Bœsch
Just a patchset to ease the transition to a sane internal form for the decoded text subtitles. [PATCH 01/17] avcodec/jacosubdec: add some memory checks [PATCH 02/17] avcodec/microdvddec: add some memory checks [PATCH 03/17] avcodec/movtextdec: add some memory checks [PATCH 04/17] avcodec/mpl2dec:

[FFmpeg-devel] [PATCH 03/17] avcodec/movtextdec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/movtextdec.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/libavcodec/movtextdec.c b/libavcodec/movtextdec.c index 05ff53a..deb7553 100644 --- a/libavcodec/movtextdec.c +++ b/libavcodec/movtextdec.c @@ -60,7 +60,7 @@ static int

[FFmpeg-devel] [PATCH 02/17] avcodec/microdvddec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/microdvddec.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/libavcodec/microdvddec.c b/libavcodec/microdvddec.c index 7df3e48..9db44e0 100644 --- a/libavcodec/microdvddec.c +++ b/libavcodec/microdvddec.c @@ -261,7 +261,6 @@ static int

[FFmpeg-devel] [PATCH 05/17] avcodec/samidec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/samidec.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavcodec/samidec.c b/libavcodec/samidec.c index 39ac608..ce05cf6 100644 --- a/libavcodec/samidec.c +++ b/libavcodec/samidec.c @@ -114,6 +114,7 @@ end: static int sami_decode_frame(AVCodecContext

[FFmpeg-devel] [PATCH 06/17] avcodec/realtextdec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/realtextdec.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/realtextdec.c b/libavcodec/realtextdec.c index 4578897..603c3c9 100644 --- a/libavcodec/realtextdec.c +++ b/libavcodec/realtextdec.c @@ -59,6 +59,7 @@ static int

[FFmpeg-devel] [PATCH 08/17] avcodec/subviewerdec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/subviewerdec.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/subviewerdec.c b/libavcodec/subviewerdec.c index 63be418..470cd7b 100644 --- a/libavcodec/subviewerdec.c +++ b/libavcodec/subviewerdec.c @@ -51,6 +51,7 @@ static int

[FFmpeg-devel] [PATCH 10/17] avcodec/webvttdec: add some memory checks

2014-09-20 Thread Clément Bœsch
--- libavcodec/webvttdec.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/webvttdec.c b/libavcodec/webvttdec.c index 6b86bed..411abf0 100644 --- a/libavcodec/webvttdec.c +++ b/libavcodec/webvttdec.c @@ -74,6 +74,7 @@ static int

[FFmpeg-devel] [PATCH 11/17] avformat/assenc: mux all extradata at once

2014-09-20 Thread Clément Bœsch
Before this commit, the code was muxing up to the 2nd line after [Events] (assuming it to be the Format: line). The remaining are generally Comment: directives which can stay in that place. mkvextract behaves that way so it seems there is no reason for that extra complexity. ---

[FFmpeg-devel] [PATCH 13/17] avformat/assdec: make sure Comments: events end up in the header

2014-09-20 Thread Clément Bœsch
--- libavformat/assdec.c | 26 -- 1 file changed, 8 insertions(+), 18 deletions(-) diff --git a/libavformat/assdec.c b/libavformat/assdec.c index 5480171..2187c7d 100644 --- a/libavformat/assdec.c +++ b/libavformat/assdec.c @@ -104,7 +104,7 @@ static int

[FFmpeg-devel] [PATCH 15/17] avformat/assdec: add myself to Copyright

2014-09-20 Thread Clément Bœsch
+ * Copyright (c) 2014 Clément Bœsch * * This file is part of FFmpeg. * -- 2.1.0 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 17/17] avformat/assenc: return correct error code

2014-09-20 Thread Clément Bœsch
--- libavformat/assenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/assenc.c b/libavformat/assenc.c index 5779e92..8593b6c 100644 --- a/libavformat/assenc.c +++ b/libavformat/assenc.c @@ -45,7 +45,7 @@ static int write_header(AVFormatContext *s) if

Re: [FFmpeg-devel] in preparation of changing the internal form of decoded text subtitles

2014-09-21 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 08:38:58AM +0200, wm4 wrote: On Sat, 20 Sep 2014 22:27:40 +0200 Clément Bœsch u...@pkh.me wrote: Just a patchset to ease the transition to a sane internal form for the decoded text subtitles. [PATCH 01/17] avcodec/jacosubdec: add some memory checks [PATCH

Re: [FFmpeg-devel] [PATCH 10/17] avcodec/webvttdec: add some memory checks

2014-09-21 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 11:20:26AM +0200, Nicolas George wrote: Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : --- libavcodec/webvttdec.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) Patches 5-10 look right, except most of them could benefit from

Re: [FFmpeg-devel] [PATCH 17/17] avformat/assenc: return correct error code

2014-09-21 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 11:12:41AM +0200, Nicolas George wrote: Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : --- libavformat/assenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM, of course. Applied -- Clément B. pgpPlGTYrOkFV.pgp Description: PGP

Re: [FFmpeg-devel] [PATCH 10/17] avcodec/webvttdec: add some memory checks

2014-09-21 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 07:19:43PM +0200, Nicolas George wrote: Le jour de la Récompense, an CCXXII, Clément Bœsch a écrit : So I did that and pushed that part of the patchset, adjusted to use a bprint helper. Unfortunately, this raises a warning because av_bprint_is_complete() doesn't

Re: [FFmpeg-devel] [PATCH 10/17] avcodec/webvttdec: add some memory checks

2014-09-21 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 07:36:19PM +0200, Nicolas George wrote: Le jour de la Récompense, an CCXXII, Clément Bœsch a écrit : Doesn't this need at least a minor bump? I consider it a bug fix, not an API change. Can you imagine a case where people would need to check for it? One case

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 02:53:12PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: +1 Applied We at XBMC got the first complaints about broken subtitles. Why was this removed? Hi, We just made the ASS packets from the Matroska output them verbatim

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 06:55:54PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer Hochecker wrote: [...] Hi, Thank you very much for this explanation and sorry for not having read release notes properly. subs

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-22 Thread Clément Bœsch
On Mon, Sep 22, 2014 at 09:13:18PM +0200, wm4 wrote: On Mon, 22 Sep 2014 21:05:19 +0200 Clément Bœsch u...@pkh.me wrote: On Mon, Sep 22, 2014 at 06:55:54PM +, Rainer Hochecker wrote: Clément Bœsch u at pkh.me writes: On Mon, Sep 22, 2014 at 03:12:02PM +, Rainer

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace missing PTS by DTS

2014-09-22 Thread Clément Bœsch
On Tue, Sep 23, 2014 at 03:21:30AM +0200, Michael Niedermayer wrote: This fixes the case where muxing fails hard in case of stream copying damaged input Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffmpeg.c |8 1 file changed, 8 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH] ffmpeg: replace missing PTS by DTS

2014-09-22 Thread Clément Bœsch
On Tue, Sep 23, 2014 at 07:45:56AM +0200, Clément Bœsch wrote: On Tue, Sep 23, 2014 at 03:21:30AM +0200, Michael Niedermayer wrote: This fixes the case where muxing fails hard in case of stream copying damaged input Signed-off-by: Michael Niedermayer michae...@gmx.at --- ffmpeg.c

Re: [FFmpeg-devel] [PATCH 16/17] avformat/assenc: mux an event header if none are found in extradata

2014-09-24 Thread Clément Bœsch
On Wed, Sep 24, 2014 at 01:20:36PM +0200, Nicolas George wrote: Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : --- libavformat/assenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/assenc.c b/libavformat/assenc.c index 236d021..5779e92 100644

Re: [FFmpeg-devel] [PATCH 12/17] avformat/assdec: output ASS packets

2014-09-24 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 08:32:59AM +0200, wm4 wrote: On Sat, 20 Sep 2014 22:27:52 +0200 Clément Bœsch u...@pkh.me wrote: After this the order from the original file is stored through readorder when doing ffmpeg -i input.ass -c copy output.mkv. And now that the ASS muxer honors

Re: [FFmpeg-devel] OPW, MPEG-4 Audio Lossless Coding (ALS) encoder

2014-09-24 Thread Clément Bœsch
Hi Thilo, On Wed, Sep 24, 2014 at 03:30:41PM +0200, Thilo Borgmann wrote: Hi Juliet, I am Juliet Fru, a sophomore, Computer Engineering student of the University of Buea. I'm intererested in working on the MPEG-4 Audio Lossless Coding (ALS) encoder project. I'll like to be added as a

Re: [FFmpeg-devel] [PATCH] avformat: remove FF_API_ASS_SSA dead code

2014-09-25 Thread Clément Bœsch
On Thu, Sep 25, 2014 at 12:41:50PM +, Rainer Hochecker wrote: Rainer Hochecker fernetmenta at online.de writes: It's me again. I got a matroshka sample for which I get ASS subs without timestamps (NOPTS_VALUE):

Re: [FFmpeg-devel] [PATCH v7] Add SUP/PGS subtitle demuxer

2014-09-26 Thread Clément Bœsch
On Fri, Sep 26, 2014 at 10:13:34AM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: Thank you for the patch. Looking forward the muxer, and a FATE test. Is there a reason why you didn't merge the muxer? http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/182167 I

Re: [FFmpeg-devel] [PATCH v7] Add SUP/PGS subtitle demuxer

2014-09-26 Thread Clément Bœsch
On Fri, Sep 26, 2014 at 01:32:09PM +0300, Petri Hintukainen wrote: On pe, 2014-09-26 at 12:26 +0200, Clément Bœsch wrote: On Fri, Sep 26, 2014 at 10:13:34AM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: Thank you for the patch. Looking forward the muxer, and a FATE

Re: [FFmpeg-devel] [PATCH 16/17] avformat/assenc: mux an event header if none are found in extradata

2014-09-27 Thread Clément Bœsch
On Wed, Sep 24, 2014 at 01:23:08PM +0200, Clément Bœsch wrote: On Wed, Sep 24, 2014 at 01:20:36PM +0200, Nicolas George wrote: Le jour de la Raison, an CCXXII, Clément Bœsch a écrit : --- libavformat/assenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat

Re: [FFmpeg-devel] [PATCH 15/17] avformat/assdec: add myself to Copyright

2014-09-27 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 01:31:22PM +0200, Michael Niedermayer wrote: On Sat, Sep 20, 2014 at 10:27:55PM +0200, Clément Bœsch wrote: --- libavformat/assdec.c | 1 + 1 file changed, 1 insertion(+) ok Thanks, pushed. -- Clément B. pgpPlCp_WlXrB.pgp Description: PGP signature

Re: [FFmpeg-devel] [PATCH 12/17] avformat/assdec: output ASS packets

2014-09-27 Thread Clément Bœsch
On Wed, Sep 24, 2014 at 01:24:17PM +0200, Clément Bœsch wrote: [...] I think this was ok. Note: I still haven't pushed this because it breaks when line are in old ASS (with Marked=0 prefix). I'll fix that ASAP. Fixed and applied. Thanks. -- Clément B. pgpGfb51Kfbbc.pgp Description

Re: [FFmpeg-devel] [PATCH 13/17] avformat/assdec: make sure Comments: events end up in the header

2014-09-27 Thread Clément Bœsch
On Sun, Sep 21, 2014 at 08:35:51AM +0200, wm4 wrote: On Sat, 20 Sep 2014 22:27:53 +0200 Clément Bœsch u...@pkh.me wrote: --- libavformat/assdec.c | 26 -- 1 file changed, 8 insertions(+), 18 deletions(-) diff --git a/libavformat/assdec.c b/libavformat

Re: [FFmpeg-devel] [PATCH 14/17] avformat/assdec: reindent after previous commit

2014-09-27 Thread Clément Bœsch
On Sat, Sep 20, 2014 at 10:27:54PM +0200, Clément Bœsch wrote: --- libavformat/assdec.c | 31 +++ 1 file changed, 15 insertions(+), 16 deletions(-) diff --git a/libavformat/assdec.c b/libavformat/assdec.c index 2187c7d..d33830e 100644 --- a/libavformat/assdec.c

Re: [FFmpeg-devel] [PATCH 1/2] Kill timed SRT

2014-09-27 Thread Clément Bœsch
On Sun, Sep 14, 2014 at 09:03:39PM +0200, Clément Bœsch wrote: --- libavcodec/srtdec.c | 33 +++ libavcodec/srtenc.c | 49 ++- libavformat/matroska.c| 1 - libavformat/matroskaenc.c | 45

Re: [FFmpeg-devel] [PATCH 2/2] Kill timed SSA

2014-09-27 Thread Clément Bœsch
On Sun, Sep 14, 2014 at 09:03:40PM +0200, Clément Bœsch wrote: --- libavcodec/Makefile | 4 ++-- libavcodec/assdec.c | 60 +++- libavcodec/assenc.c | 14 ++-- libavformat/assenc.c | 14 libavformat/nut.c| 1

[FFmpeg-devel] [PATCH] avcodec/srtdec: use AVBPrint API

2014-09-28 Thread Clément Bœsch
--- Can only be applied after timed SRT is dropped. --- libavcodec/srtdec.c | 78 - 1 file changed, 35 insertions(+), 43 deletions(-) diff --git a/libavcodec/srtdec.c b/libavcodec/srtdec.c index cc17486..d9cc16a 100644 --- a/libavcodec/srtdec.c

Re: [FFmpeg-devel] IRC meeting

2014-09-29 Thread Clément Bœsch
On Mon, Sep 29, 2014 at 05:39:23PM +0200, Stefano Sabatini wrote: Hi, I want to propose to have an FFmpeg IRC meeting the next Saturday, 4th October, UTC 16. Alternatively, I propose the Saturday of the next week, Saturday October 11, same time. Candidate topics of the day: - VDD 2014

Re: [FFmpeg-devel] [PATCH 1/2] Kill timed SRT

2014-09-29 Thread Clément Bœsch
On Mon, Sep 29, 2014 at 05:42:09PM +0200, Nicolas George wrote: L'octidi 28 fructidor, an CCXXII, Clément Bœsch a écrit : --- libavcodec/srtdec.c | 33 +++ libavcodec/srtenc.c | 49 ++- libavformat

Re: [FFmpeg-devel] [PATCH 1/2] Kill timed SRT

2014-09-30 Thread Clément Bœsch
On Mon, Sep 29, 2014 at 07:27:02PM +0200, Clément Bœsch wrote: On Mon, Sep 29, 2014 at 05:42:09PM +0200, Nicolas George wrote: L'octidi 28 fructidor, an CCXXII, Clément Bœsch a écrit : --- libavcodec/srtdec.c | 33 +++ libavcodec/srtenc.c | 49

Re: [FFmpeg-devel] [PATCH] avcodec/srtdec: use AVBPrint API

2014-09-30 Thread Clément Bœsch
On Mon, Sep 29, 2014 at 05:36:15PM +0200, Nicolas George wrote: Le septidi 7 vendémiaire, an CCXXIII, Clément Bœsch a écrit : --- Can only be applied after timed SRT is dropped. --- libavcodec/srtdec.c | 78 - 1 file changed, 35

Re: [FFmpeg-devel] [PATCH] libavfilter: increase vf_drawtext line height

2014-10-01 Thread Clément Bœsch
On Wed, Oct 01, 2014 at 10:10:34PM +0400, Михаил Муковников wrote: But on the other way, I doubt someone could ever need any other value, 1.2 is kind of standard already, e.g. every major desktop browser uses this value as the default line-height. [citation needed] If you don't have any

Re: [FFmpeg-devel] Facebook page

2014-10-04 Thread Clément Bœsch
On Sat, Oct 04, 2014 at 10:12:49PM +0200, Thomas Volkert wrote: Hallo @ll, On Facebook, there is no official ffmpeg presence at the moment. But maybe it would be interesting to have a page with announcements about: * new releases * API/ABI changes * ffmpeg related events * new bounties

[FFmpeg-devel] [PATCH] avfilter: add w/h parameters in timeline

2014-10-06 Thread Clément Bœsch
Fixes Ticket #4008. --- libavfilter/avfilter.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c index 7b11467..3805912 100644 --- a/libavfilter/avfilter.c +++ b/libavfilter/avfilter.c @@ -381,8 +381,23 @@

Re: [FFmpeg-devel] [PATCH] swscale: support internal scaler cascades

2014-10-10 Thread Clément Bœsch
On Wed, Oct 08, 2014 at 02:11:06AM +0200, Michael Niedermayer wrote: Fixes Ticket3170 Signed-off-by: Michael Niedermayer michae...@gmx.at --- libswscale/swscale.c | 13 ++ libswscale/swscale_internal.h |6 + libswscale/utils.c| 54

Re: [FFmpeg-devel] [PATCH] avfilter: add w/h parameters in timeline

2014-10-10 Thread Clément Bœsch
On Mon, Oct 06, 2014 at 05:02:30PM +0200, Stefano Sabatini wrote: On date Monday 2014-10-06 13:11:03 +0200, Clément Bœsch encoded: Fixes Ticket #4008. --- libavfilter/avfilter.c | 21 +++-- 1 file changed, 19 insertions(+), 2 deletions(-) Missing docs updates

[FFmpeg-devel] [PATCH] fate: make subtitles tests output to stdout instead of md5

2014-10-11 Thread Clément Bœsch
This makes tracking subtitles changes simpler. --- tests/fate-run.sh | 6 ++ tests/fate/subtitles.mak | 38 tests/ref/fate/sub-aqtitle| 45 +- tests/ref/fate/sub-charenc| 62 - tests/ref/fate/sub-jacosub| 23 -

Re: [FFmpeg-devel] [PATCH] fate: make subtitles tests output to stdout instead of md5

2014-10-11 Thread Clément Bœsch
On Sat, Oct 11, 2014 at 02:53:24PM +0200, wm4 wrote: On Sat, 11 Oct 2014 13:56:16 +0200 Clément Bœsch u...@pkh.me wrote: This makes tracking subtitles changes simpler. --- ... Seems like a good idea. I'm unfortunately kind of uncomfortable with this: [~]☭ printf x\r\n /tmp

[FFmpeg-devel] [PATCH 1/7] fate: add rawdiff and use it for subtitles

2014-10-11 Thread Clément Bœsch
We want subtitles tests to match exactly line endings and trailing characters. --- tests/fate-run.sh| 1 + tests/fate/subtitles.mak | 1 + 2 files changed, 2 insertions(+) diff --git a/tests/fate-run.sh b/tests/fate-run.sh index 224da8b..2f18fa3 100755 --- a/tests/fate-run.sh +++

[FFmpeg-devel] [PATCH 3/7] avformat/assenc: support remuxing old V4 ASS form (SSA)

2014-10-11 Thread Clément Bœsch
--- libavformat/assenc.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavformat/assenc.c b/libavformat/assenc.c index d495ce3..8225967 100644 --- a/libavformat/assenc.c +++ b/libavformat/assenc.c @@ -35,6 +35,7 @@ typedef struct ASSContext{ DialogueLine

[FFmpeg-devel] [PATCH 2/7] fate: fix some subtitles tests dependencies

2014-10-11 Thread Clément Bœsch
--- tests/fate/subtitles.mak | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/fate/subtitles.mak b/tests/fate/subtitles.mak index 2c6edc6..9243178 100644 --- a/tests/fate/subtitles.mak +++ b/tests/fate/subtitles.mak @@ -37,7 +37,7 @@ fate-sub-sami: CMD = fmtstdout ass

[FFmpeg-devel] [PATCH 5/7] avcodec/ass_split: extend recognized fields in ASS splitter

2014-10-11 Thread Clément Bœsch
This simplifies the logic for the fix in the next commit. --- libavcodec/ass_split.c | 31 ++- libavcodec/ass_split.h | 20 2 files changed, 50 insertions(+), 1 deletion(-) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index

[FFmpeg-devel] [PATCH 6/7] avcodec/ass_split: assume default order when Format: line is not found

2014-10-11 Thread Clément Bœsch
Fixes the decoding of the ASS stream in a mkv from a fansub release from Coalgirls. --- libavcodec/ass_split.c | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index fea38e7..bd7db93 100644 ---

[FFmpeg-devel] [PATCH 4/7] avcodec/ass: output missing fields in AVSubtitles and output files

2014-10-11 Thread Clément Bœsch
Fixes Ticket #3207 --- libavcodec/ass.c| 29 +-- libavcodec/version.h| 2 +- tests/ref/fate/sub-aqtitle | 72 +-- tests/ref/fate/sub-charenc | 106 tests/ref/fate/sub-jacosub |

[FFmpeg-devel] [PATCH 7/7] avcodec/ass: add FFmpeg signature

2014-10-11 Thread Clément Bœsch
--- libavcodec/ass.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/ass.c b/libavcodec/ass.c index ac1bfae..62981b6 100644 --- a/libavcodec/ass.c +++ b/libavcodec/ass.c @@ -34,6 +34,7 @@ int ff_ass_subtitle_header(AVCodecContext *avctx, { avctx-subtitle_header =

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/ass_split: assume default order when Format: line is not found

2014-10-11 Thread Clément Bœsch
On Sat, Oct 11, 2014 at 10:53:28PM +0200, wm4 wrote: On Sat, 11 Oct 2014 19:59:11 +0200 Clément Bœsch u...@pkh.me wrote: Fixes the decoding of the ASS stream in a mkv from a fansub release from Coalgirls. --- libavcodec/ass_split.c | 26 +- 1 file changed, 25

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/ass: add FFmpeg signature

2014-10-12 Thread Clément Bœsch
On Sat, Oct 11, 2014 at 10:53:47PM -0300, James Almer wrote: On 11/10/14 2:59 PM, Clément Bœsch wrote: --- libavcodec/ass.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/ass.c b/libavcodec/ass.c index ac1bfae..62981b6 100644 --- a/libavcodec/ass.c +++ b

<    1   2   3   4   5   6   7   8   9   10   >