Re: [FFmpeg-devel] IRC meeting

2014-10-01 Thread Stefano Sabatini
On date Tuesday 2014-09-30 20:57:33 -0700, Timothy Gu encoded: > Hi, > > On Mon, Sep 29, 2014 at 8:39 AM, 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,

Re: [FFmpeg-devel] [PATCH v2] Add SUP/PGS subtitle muxer

2014-10-01 Thread Petri Hintukainen
On ti, 2014-09-30 at 15:56 +0200, wm4 wrote: > On Tue, 30 Sep 2014 11:01:17 +0300 > phint...@gmail.com wrote: > > > From: Petri Hintukainen > > > > Fixes ticket #2208 > > --- > > Changelog| 2 +- > > doc/general.texi | 2 +- > > libavformat/Makefile | 1 + > > lib

Re: [FFmpeg-devel] [PATCH 4/4] hevc: reuse edge emu buffer for coefficients

2014-10-01 Thread Michael Niedermayer
On Tue, Sep 23, 2014 at 09:36:43PM +, Christophe Gisquet wrote: > Kind of hackish but... > --- > libavcodec/hevc.c | 8 > libavcodec/hevc.h | 1 - > libavcodec/hevc_cabac.c | 4 ++-- > 3 files changed, 6 insertions(+), 7 deletions(-) reviewed by Mickael Raulet applied th

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

2014-10-01 Thread Petri Hintukainen
On ti, 2014-09-30 at 16:09 +0200, wm4 wrote: > On Tue, 30 Sep 2014 12:01:33 +0300 > Petri Hintukainen wrote: > > > Hello, > > > > Missing DTS combined with non-monotonic PTS seems to cause some > > problems. This is with the file from ticket #2208 (I added dumping > > timestamps to demuxer and m

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

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 01:36:58PM +0300, Petri Hintukainen wrote: > On ti, 2014-09-30 at 16:09 +0200, wm4 wrote: > > On Tue, 30 Sep 2014 12:01:33 +0300 > > Petri Hintukainen wrote: > > > > > Hello, > > > > > > Missing DTS combined with non-monotonic PTS seems to cause some > > > problems. This

Re: [FFmpeg-devel] [PATCH] x86/mpegvideoencdsp: improve ff_pix_sum16_sse2

2014-10-01 Thread Michael Niedermayer
On Tue, Sep 30, 2014 at 10:24:28PM -0300, James Almer wrote: > ~15 faster. > > Also add an mmxext version that takes advantage of the new code, and > build it alongside with the mmx version only on x86_32. > > Signed-off-by: James Almer > --- > libavcodec/x86/mpegvideoencdsp.asm| 51 >

Re: [FFmpeg-devel] [PATCH 1/18] avcodec: remove obsolete FF_API_FAST_MALLOC cruft

2014-10-01 Thread Andreas Cadhalpun
On 01.10.2014 04:34, Michael Niedermayer wrote: On Mon, Sep 15, 2014 at 10:59:39PM +0200, Andreas Cadhalpun wrote: On 15.09.2014 22:03, Michael Niedermayer wrote: On Mon, Sep 15, 2014 at 04:08:49PM -0300, James Almer wrote: On 15/09/14 6:07 AM, Michael Niedermayer wrote: On Sun, Sep 14, 2014

Re: [FFmpeg-devel] [PATCH] x86/mpegvideoencdsp: improve ff_pix_sum16_sse2

2014-10-01 Thread James Darnley
On 2014-10-01 03:24, James Almer wrote: > ~15 faster. 15 what faster? 15 cycles? 15%? 15 times? signature.asc Description: OpenPGP digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-de

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

2014-10-01 Thread wm4
On Wed, 1 Oct 2014 12:58:32 +0200 Michael Niedermayer wrote: > On Wed, Oct 01, 2014 at 01:36:58PM +0300, Petri Hintukainen wrote: > > On ti, 2014-09-30 at 16:09 +0200, wm4 wrote: > > > On Tue, 30 Sep 2014 12:01:33 +0300 > > > Petri Hintukainen wrote: > > > > > > > Hello, > > > > > > > > Missin

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

2014-10-01 Thread Михаил Муковников
It’s a common practice to add some space between lines (typically 0.2 * font-height) as it dramatically increases readability. In my opinion, the default behaviour increasing the height just by max_glyph_h is not a very good one. diff --git a/libavfilter/vf_drawtext.c b/libavfilter/vf_drawtext.

Re: [FFmpeg-devel] [PATCH] avformat/movenc: AVC Intra support

2014-10-01 Thread Michael Niedermayer
On Tue, Sep 30, 2014 at 11:07:28AM +0100, Kieran Kunhya wrote: > > Some of the DPP AVCI100 test samples were created using x264* so could > > be used for reference. Or I could knock up something if I can find my > > reference CL I used last time. > > www.obe.tv/about-us/obe-blog/item/1-oss-dpp-cre

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

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 06:07:02PM +0400, Михаил Муковников wrote: > It’s a common practice to add some space between lines (typically 0.2 * > font-height) as it dramatically increases readability. In my opinion, the > default behaviour increasing the height just by max_glyph_h is not a very > g

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

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 03:07:19PM +0200, wm4 wrote: > On Wed, 1 Oct 2014 12:58:32 +0200 > Michael Niedermayer wrote: > > > On Wed, Oct 01, 2014 at 01:36:58PM +0300, Petri Hintukainen wrote: > > > On ti, 2014-09-30 at 16:09 +0200, wm4 wrote: > > > > On Tue, 30 Sep 2014 12:01:33 +0300 > > > > Petr

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

2014-10-01 Thread wm4
On Wed, 1 Oct 2014 17:11:17 +0200 Michael Niedermayer wrote: > > Ouch. Yeah, I think this function used to corrupt perfectly fine h264 > > mpegts timestamps in the past or so... > > which trac ticket is that ? > or do you mean by "past" that this is not relevant anymore as it has > been fixed al

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

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 05:32:52PM +0200, wm4 wrote: > On Wed, 1 Oct 2014 17:11:17 +0200 > Michael Niedermayer wrote: > > > > Ouch. Yeah, I think this function used to corrupt perfectly fine h264 > > > mpegts timestamps in the past or so... > > > > which trac ticket is that ? > > or do you mean

Re: [FFmpeg-devel] [PATCH] x86/mpegvideoencdsp: improve ff_pix_sum16_sse2

2014-10-01 Thread James Almer
On 01/10/14 8:22 AM, Michael Niedermayer wrote: > On Tue, Sep 30, 2014 at 10:24:28PM -0300, James Almer wrote: >> ~15 faster. >> >> Also add an mmxext version that takes advantage of the new code, and >> build it alongside with the mmx version only on x86_32. >> >> Signed-off-by: James Almer >> --

Re: [FFmpeg-devel] [PATCH] x86/mpegvideoencdsp: improve ff_pix_sum16_sse2

2014-10-01 Thread James Almer
On 01/10/14 9:57 AM, James Darnley wrote: > On 2014-10-01 03:24, James Almer wrote: >> ~15 faster. > > 15 what faster? 15 cycles? 15%? 15 times? Percent, thanks for noticing. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mai

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

2014-10-01 Thread Михаил Муковников
Sure: http://i.imgur.com/FEaJKJo.png (153 KiB). Current behavior is on the left, the propositional one is on the right. Both are generated by a filter string "color=s=512x512,drawtext=fontfile=/Library/Fonts/Arial.ttf:text=123\n456\n789:fontsize=130:fontcolor=white:x=256-text_w/2:y=256-text_h/2”

[FFmpeg-devel] [PATCH]Stop demuxing wtv on eof

2014-10-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes the wtv regression described in tickets #3991 and #3995 for me. Please review, Carl Eugen diff --git a/libavformat/wtvdec.c b/libavformat/wtvdec.c index 4cb3295..674ff6e 100644 --- a/libavformat/wtvdec.c +++ b/libavformat/wtvdec.c @@ -789,6 +789,8 @@ static int parse_chu

[FFmpeg-devel] [PATCH 1/3] lavf/webm_dash: fix hardcode in cues_end computation

2014-10-01 Thread Vignesh Venkatasubramanian
Fix an incorrect hard code in cues_end computation. Updating the fate test reference files related to the fix as well. The earlier computation was clearly wrong as the cues_end field was greater than the file size itself in some cases. Signed-off-by: Vignesh Venkatasubramanian --- libavformat/ma

[FFmpeg-devel] [PATCH]Check for OOM in the dirac parser

2014-10-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes two possible null pointer dereferences on oom as described in ticket #3996. Please comment, Carl Eugen diff --git a/libavcodec/dirac_parser.c b/libavcodec/dirac_parser.c index 4119e3b..8b7c55e 100644 --- a/libavcodec/dirac_parser.c +++ b/libavcodec/dirac_parser.c @@ -139

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

2014-10-01 Thread Timothy Gu
On Wed, Oct 1, 2014 at 7:07 AM, Михаил Муковников wrote: > It’s a common practice to add some space between lines (typically 0.2 * > font-height) as it dramatically increases readability. In my opinion, the > default behaviour increasing the height just by max_glyph_h is not a very > good one.

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

2014-10-01 Thread Михаил Муковников
Yes, that would be awesome. On 01 окт. 2014 г., at 21:32, Timothy Gu wrote: > I think the line spacing should be user-changeable through an AVOption. > > Timothy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/

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

2014-10-01 Thread Михаил Муковников
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. On 01 окт. 2014 г., at 22:04, Михаил Муковников wrote: > Yes, that would be awesome. > > On 01 окт. 2014 г., at

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 an

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

2014-10-01 Thread Михаил Муковников
I suppose https://developer.mozilla.org/en-US/docs/Web/CSS/line-height could stand as a citation. Anyway, all right, let’s make this option tunable, no problem. I hope I would be able to make other patch in a day or something. On 01 окт. 2014 г., at 22:31, Clément Bœsch wrote: > [citation need

Re: [FFmpeg-devel] [PATCH]Check for OOM in the dirac parser

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 07:32:36PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes two possible null pointer dereferences on oom as > described in ticket #3996. > > Please comment, Carl Eugen > dirac_parser.c |4 > 1 file changed, 4 insertions(+) > b813326b16996c7744858d

Re: [FFmpeg-devel] [PATCH 1/3] lavf/webm_dash: fix hardcode in cues_end computation

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 10:13:30AM -0700, Vignesh Venkatasubramanian wrote: > Fix an incorrect hard code in cues_end computation. Updating the fate > test reference files related to the fix as well. The earlier computation > was clearly wrong as the cues_end field was greater than the file size > i

Re: [FFmpeg-devel] [PATCH 2/2] av_lockmgr_register defines behavior on failure.

2014-10-01 Thread Manfred Georg
On Tue, Sep 30, 2014 at 8:16 PM, Michael Niedermayer wrote: > On Tue, Sep 30, 2014 at 07:34:28PM -0700, Manfred Georg wrote: > > Yeah, that seemed a bit odd to meI guess I get to go correct some > > calling code. > > > > Here is yet another update with a comment which tells you not to use a >

[FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread James Almer
This makes the addition of arch optimized functions easier. Signed-off-by: James Almer --- libavcodec/Makefile | 2 +- libavcodec/jpeg2000dec.c | 71 +-- libavcodec/jpeg2000dsp.c | 98 libavcodec/jpeg2000dsp.h

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread Carl Eugen Hoyos
James Almer gmail.com> writes: > This makes the addition of arch optimized functions easier. Do you think that the functions in question are bitexact wrt the specification? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmp

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread James Almer
On 01/10/14 5:22 PM, Carl Eugen Hoyos wrote: > James Almer gmail.com> writes: > >> This makes the addition of arch optimized functions easier. > > Do you think that the functions in question are bitexact > wrt the specification? > > Carl Eugen The 53 is afaik. The 97 float isn't, which is why

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread Carl Eugen Hoyos
James Almer gmail.com> writes: > On 01/10/14 5:22 PM, Carl Eugen Hoyos wrote: > > James Almer gmail.com> writes: > > > >> This makes the addition of arch optimized > >> functions easier. > > > > Do you think that the functions in question are > > bitexact wrt the specification? > The 53 is

Re: [FFmpeg-devel] [PATCH 2/2] av_lockmgr_register defines behavior on failure.

2014-10-01 Thread Manfred Georg
As a further note. I'm pretty sure that none of the tests make use of av_lockmgr_register...so there's very little guarantee that this function is working correctly (particularly now that it has gotten so complicated). Manfred ___ ffmpeg-devel mailing l

Re: [FFmpeg-devel] [PATCH]Stop demuxing wtv on eof

2014-10-01 Thread Peter Ross
On Wed, Oct 01, 2014 at 06:18:56PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes the wtv regression described in tickets #3991 and #3995 > for me. > > Please review, Carl Eugen > diff --git a/libavformat/wtvdec.c b/libavformat/wtvdec.c > index 4cb3295..674ff6e 100644 > --- a/lib

Re: [FFmpeg-devel] [PATCH 2/2] av_lockmgr_register defines behavior on failure.

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 01:01:33PM -0700, Manfred Georg wrote: > On Tue, Sep 30, 2014 at 8:16 PM, Michael Niedermayer > wrote: > > > On Tue, Sep 30, 2014 at 07:34:28PM -0700, Manfred Georg wrote: > > > Yeah, that seemed a bit odd to meI guess I get to go correct some > > > calling code. > > >

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 05:14:38PM -0300, James Almer wrote: > This makes the addition of arch optimized functions easier. > > Signed-off-by: James Almer > --- > libavcodec/Makefile | 2 +- > libavcodec/jpeg2000dec.c | 71 +-- > libavcodec/jpeg2000dsp.c | 98

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread James Almer
On 01/10/14 8:04 PM, Michael Niedermayer wrote: > On Wed, Oct 01, 2014 at 05:14:38PM -0300, James Almer wrote: >> This makes the addition of arch optimized functions easier. >> >> Signed-off-by: James Almer >> --- >> libavcodec/Makefile | 2 +- >> libavcodec/jpeg2000dec.c | 71 +

Re: [FFmpeg-devel] [PATCH 2/2] av_lockmgr_register defines behavior on failure.

2014-10-01 Thread Manfred Georg
[snip] > @@ -3457,22 +3457,53 @@ AVHWAccel *av_hwaccel_next(const AVHWAccel > *hwaccel) > > int av_lockmgr_register(int (*cb)(void **mutex, enum AVLockOp op)) > > { > > if (lockmgr_cb) { > > -if (lockmgr_cb(&codec_mutex, AV_LOCK_DESTROY)) > > -return -1; > > -if

Re: [FFmpeg-devel] [PATCH 2/2] av_lockmgr_register defines behavior on failure.

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 04:37:21PM -0700, Manfred Georg wrote: > [snip] > > > @@ -3457,22 +3457,53 @@ AVHWAccel *av_hwaccel_next(const AVHWAccel > > *hwaccel) > > > int av_lockmgr_register(int (*cb)(void **mutex, enum AVLockOp op)) > > > { > > > if (lockmgr_cb) { > > > -if (lockmgr_

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 08:08:18PM -0300, James Almer wrote: > On 01/10/14 8:04 PM, Michael Niedermayer wrote: > > On Wed, Oct 01, 2014 at 05:14:38PM -0300, James Almer wrote: > >> This makes the addition of arch optimized functions easier. > >> > >> Signed-off-by: James Almer > >> --- > >> libav

[FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread James Almer
This makes the addition of arch optimized functions easier. Signed-off-by: James Almer --- libavcodec/Makefile | 2 +- libavcodec/jpeg2000dec.c | 71 +-- libavcodec/jpeg2000dsp.c | 98 libavcodec/jpeg2000dsp.h

Re: [FFmpeg-devel] patch 1/4: libavcodec/ppc/pixblockdsp.c: fix get_pixels_altivec() and diff_pixels_altivec() for POWER LE

2014-10-01 Thread Michael Niedermayer
On Tue, Sep 30, 2014 at 07:51:38PM +0200, Reimar Döffinger wrote: > On Tue, Sep 30, 2014 at 03:51:55PM +0800, rongyan wrote: > > Hi, > > I present 4 patches to fix bugs for POWER8 little endian. > > I will send 4 patches in 4 different email. This is the first. > > The fate test result after merg

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread Michael Niedermayer
On Wed, Oct 01, 2014 at 10:35:59PM -0300, James Almer wrote: > This makes the addition of arch optimized functions easier. > > Signed-off-by: James Almer > --- > libavcodec/Makefile | 2 +- > libavcodec/jpeg2000dec.c | 71 +-- > libavcodec/jpeg2000dsp.c | 98

Re: [FFmpeg-devel] [PATCH] avformat/mxfenc: add jpeg2000 support

2014-10-01 Thread Michael Niedermayer
On Tue, Sep 30, 2014 at 04:08:23PM +0200, Matthieu Bouron wrote: > On Tue, Sep 30, 2014 at 3:37 PM, Benoit Fouet wrote: > > > Hi, > > > > Hi Benoit, > > Thanks for your patch. > I've reviewed the ULs values and they look good to me. applied thanks > > > > > > this patch adds support for j

Re: [FFmpeg-devel] *** GMX Spamverdacht *** Re: patch 1/4: libavcodec/ppc/pixblockdsp.c: fix get_pixels_altivec() and diff_pixels_altivec() for POWER LE

2014-10-01 Thread Reimar Döffinger
On 02.10.2014, at 03:38, Michael Niedermayer wrote: > On Tue, Sep 30, 2014 at 07:51:38PM +0200, Reimar Döffinger wrote: >> On Tue, Sep 30, 2014 at 03:51:55PM +0800, rongyan wrote: >>> Hi, >>> I present 4 patches to fix bugs for POWER8 little endian. >>> I will send 4 patches in 4 different email.

Re: [FFmpeg-devel] [PATCH] jpeg2000: split off inverse MCT decoding as Jpeg2000DSP

2014-10-01 Thread James Almer
On 01/10/14 11:16 PM, Michael Niedermayer wrote: > On Wed, Oct 01, 2014 at 10:35:59PM -0300, James Almer wrote: >> This makes the addition of arch optimized functions easier. >> >> Signed-off-by: James Almer >> --- >> libavcodec/Makefile | 2 +- >> libavcodec/jpeg2000dec.c | 71 +---

Re: [FFmpeg-devel] [PATCH]Stop demuxing wtv on eof

2014-10-01 Thread Paul B Mahol
On 10/1/14, Peter Ross wrote: > On Wed, Oct 01, 2014 at 06:18:56PM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes the wtv regression described in tickets #3991 and >> #3995 >> for me. >> >> Please review, Carl Eugen > >> diff --git a/libavformat/wtvdec.c b/libavformat/wtvdec.c >>