Re: [FFmpeg-devel] [PATCH] checkasm: Add vf_blend tests

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > --- > tests/checkasm/Makefile | 1 + > tests/checkasm/checkasm.c | 3 ++ > tests/checkasm/checkasm.h | 1 + > tests/checkasm/vf_blend.c | 130 > ++ > 4 files changed, 135 insertions(+) > create mode 100644 tests/

Re: [FFmpeg-devel] [PATCH 1/2] vf_blend: Templatize identity function and use a better name

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > --- > libavfilter/vf_blend.c | 22 ++ > 1 file changed, 14 insertions(+), 8 deletions(-) > lgtm ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] vf_blend: Use memcpy when opacity is 0

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > For xor mode, 1.8x faster on Haswell. > --- > libavfilter/vf_blend.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c > index c24013d..599084f 100644 > --- a/libavfilter/vf_blend.c > +++ b/libavfilter/vf

Re: [FFmpeg-devel] [PATCH] vf_blend: Reduce number of arguments for kernel function

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > --- > libavfilter/blend.h | 2 +- > libavfilter/vf_blend.c | 27 ++- > libavfilter/x86/vf_blend.asm| 3 +-- > libavfilter/x86/vf_blend_init.c | 2 +- > 4 files changed, 17 insertions(+), 17 deletions(-) > ok if i

Re: [FFmpeg-devel] [PATCH] vf_blend: Use integers for divide mode

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > 2.5x faster for 8-bit mode without autovectorization in GCC, 2x > slower with it on x86. However, since the platforms we enable GCC > autovectorization on most probably has support for SSE2 > optimization (added in the subsequent commit), this commit should > in gen

Re: [FFmpeg-devel] [PATCH] x86/vf_blend: Add SSE2 optimization for divide

2016-02-14 Thread Paul B Mahol
On 2/14/16, Timothy Gu wrote: > On Sat, Feb 13, 2016 at 07:21:25PM -0800, Timothy Gu wrote: >> --- >> libavfilter/x86/vf_blend.asm| 30 ++ >> libavfilter/x86/vf_blend_init.c | 2 ++ >> 2 files changed, 32 insertions(+) > > Locally added commit message: > > 4.

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

2016-02-14 Thread Mats Peterson
On 02/14/2016 07:03 AM, Mats Peterson wrote: Old stuff, new shape. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Just as a parenthesis, regardless of whether this patch is applied or not, th

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

2016-02-14 Thread Mats Peterson
On 02/14/2016 11:45 AM, Mats Peterson wrote: On 02/14/2016 07:03 AM, Mats Peterson wrote: Old stuff, new shape. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Just as a parenthesis, regardles

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

2016-02-14 Thread Mats Peterson
On 02/14/2016 11:45 AM, Mats Peterson wrote: Just as a parenthesis, regardless of whether this patch is applied or not, the only way to create a "spec" compliant 1 bpp raw file from a 1 bpp raw AVI or QuickTime file Make that any type of source file, not only AVI or QuickTime. -- Mats Peterson

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

2016-02-14 Thread Paul B Mahol
On 2/13/16, Stefano Sabatini wrote: > On date Friday 2016-02-12 15:28:15 +0100, Paul B Mahol encoded: >> On 2/11/16, Paul B Mahol wrote: >> > Hi, >> > >> > patch attached. >> > >> >> Better version attached. > >> From d953f78bffbf3db8f0209b41b189ece12b402afa Mon Sep 17 00:00:00 2001 >> From: Paul

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

2016-02-14 Thread Paul B Mahol
On 2/12/16, Paul B Mahol wrote: > On 2/11/16, Paul B Mahol wrote: >> Hi, >> >> patch attached. >> > > Better version attached. > Even better version attached. From 5a96e8103b2556291e1fe1922adb16945e70f64b Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Thu, 11 Feb 2016 22:05:54 +0100 Subject:

Re: [FFmpeg-devel] [PATCH 4/4] avfilter/f_metadata: whitespace clean-up

2016-02-14 Thread Paul B Mahol
All aplied. Thanks. ___ 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-14 Thread Michael Niedermayer
On Sat, Jan 23, 2016 at 10:12:30AM +0100, Andreas Cadhalpun wrote: > VLC uses hwaccel with frame threads and it works fine, but returning > an error here made it fail. > > This regression was introduced in commit 31741ae. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/utils.c | 5 ++---

Re: [FFmpeg-devel] [PATCH 1/3] lavf/mpjpeg: Trim quotes on MIME boundary, if present.

2016-02-14 Thread Michael Niedermayer
On Sat, Feb 13, 2016 at 11:47:20PM -0500, Alexander Agranovsky wrote: > > mpjpegdec.c |7 +++ > 1 file changed, 7 insertions(+) > 7be9bc0f4e4e369bc7bf45dd1ce8a3b97e960f4e > 0001-lavf-mpjpeg-Trim-quotes-on-MIME-boundary-if-present.patch > From f2d8bc033cc6ab6e9a42472c99bdfce8305a13db Mo

Re: [FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format

2016-02-14 Thread Christophe Gisquet
Hi, 2016-02-14 6:49 GMT+01:00 Timothy Gu : > %if HAVE_MMX_INLINE Isn't that macro meant for C code (and in config.asm without much of a purpose)? I suspect it is not useful, but I haven't dug into that. > +; XXX some of these macros are not used right now, but they will in the > future > +;

Re: [FFmpeg-devel] [PATCH]levc/hevc_cabac Optimise ff_hevc_hls_residual_coding (v2)

2016-02-14 Thread Christophe Gisquet
Hi, 2016-02-14 0:43 GMT+01:00 Michael Niedermayer : > i can test and commit the code, it seems everyone who wanted to > comment did comment Yes, nothing really worth postponing the patchset from my side, except maybe cleaner splitting of the bypass stuff, as agreed by the author. I don't mind if

[FFmpeg-devel] [PATCH] avutil/parseutils: remove 2112 date from fate test

2016-02-14 Thread Marton Balint
It is not supported on 32bit. Signed-off-by: Marton Balint --- libavutil/parseutils.c| 2 +- tests/ref/fate/parseutils | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c index 16c8b6a..0097bec 100644 --- a/libavutil/parseuti

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: av_image_get_buffer_size(): do not insert padding between stride*height and palette

2016-02-14 Thread Stefano Sabatini
On date Saturday 2016-02-13 23:08:50 +0100, wm4 encoded: > On Sat, 13 Feb 2016 21:51:48 +0100 > Michael Niedermayer wrote: > > > On Sat, Feb 13, 2016 at 08:46:34PM +0100, wm4 wrote: > > > On Sat, 13 Feb 2016 19:38:01 +0100 > > > Michael Niedermayer wrote: > > > > > > > This fixes the layout t

Re: [FFmpeg-devel] Fwd: Fwd: libavformat/segment : add option to increment timecode

2016-02-14 Thread Stefano Sabatini
On date Saturday 2016-02-13 22:00:27 +0100, Martin Vignali encoded: [...] > From 8395ed7c66327f754e656622bacb5c2dd91d00ca Mon Sep 17 00:00:00 2001 > From: Martin Vignali > Date: Sat, 13 Feb 2016 21:46:38 +0100 > Subject: [PATCH] lavf/segment: add increment timecode option > > for example you can

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as, part of the returned frame

2016-02-14 Thread Michael Niedermayer
On Sat, Feb 13, 2016 at 11:49:06PM -0500, Alexander Agranovsky wrote: > > mpjpegdec.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > 0c761f422ba83446d20728449c0c6813e4e11560 > 0003-lavf-mpjpeg-do-not-include-CRLF-preceding-boundary-a.patch > From b2ef061db32933e896dbeecb68268

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-14 Thread Paul B Mahol
On 2/10/16, Derek Buitenhuis wrote: > * Upstream libutvideo doesn't build as a lib, and Linux is not supported. > It has no API. > * The fork that was used as an argument to keep support is *also* broken and > unmaintained now, and it's maintainer/author is fine with removal. > * A third fork

Re: [FFmpeg-devel] Fwd: Fwd: libavformat/segment : add option to increment timecode

2016-02-14 Thread Martin Vignali
2016-02-14 14:10 GMT+01:00 Stefano Sabatini : > On date Saturday 2016-02-13 22:00:27 +0100, Martin Vignali encoded: > [...] > > From 8395ed7c66327f754e656622bacb5c2dd91d00ca Mon Sep 17 00:00:00 2001 > > From: Martin Vignali > > Date: Sat, 13 Feb 2016 21:46:38 +0100 > > Subject: [PATCH] lavf/segme

[FFmpeg-devel] [PATCH] lavc/rawdec: Retrieve nut palette from packet data

2016-02-14 Thread Mats Peterson
Since a new frame is created due to the line alignment requirement of 16 bytes, the nut palette has to be retrieved from the packet data. Mats -- Mats Peterson http://matsp888.no-ip.org/~mats/ >From d8687d9a6db9630876619e802086c78d82f5c661 Mon Sep 17 00:00:00 2001 From: Mats Peterson Date: Sun

Re: [FFmpeg-devel] [PATCH] x86/vf_blend: Add SSE2 optimization for divide

2016-02-14 Thread Henrik Gramner
You could try doing 8 or 16 bytes per iteration instead of 4, it might be faster depending on how good your cpu is at OOE. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-14 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > Because there is not more maintainer for this, I will > push this very soon. I will try to maintain it for the foreseeable future. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/m

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-14 Thread James Almer
On 2/14/2016 11:53 AM, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> Because there is not more maintainer for this, I will >> push this very soon. > > I will try to maintain it for the foreseeable future. > > Carl Eugen How do you plan to get around the fact the only working

[FFmpeg-devel] [PATCH v2 0/3] Improve os2threads

2016-02-14 Thread KO Myung-Hun
Hi/2. These are v2 patches for os2threads. Replaced spin-locks with an event semaphore. [PATCH v2 1/3] compat/os2threads: Improve pthread_cond_xxx() [PATCH v2 2/3] compat/os2threads: support the return value of joined [PATCH v2 3/3] compat/os2threads: split long lines ___

[FFmpeg-devel] [PATCH v2 3/3] compat/os2threads: split long lines

2016-02-14 Thread KO Myung-Hun
--- compat/os2threads.h | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/compat/os2threads.h b/compat/os2threads.h index 3b289df..40a119f 100644 --- a/compat/os2threads.h +++ b/compat/os2threads.h @@ -71,7 +71,10 @@ static void thread_entry(void *arg) thr

[FFmpeg-devel] [PATCH v2 1/3] compat/os2threads: Improve pthread_cond_xxx() functions

2016-02-14 Thread KO Myung-Hun
1. Manipulate waiting count in pthread_cond_wait() 2. Use builtin atomic functions to manipulate waiting count --- compat/os2threads.h | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/compat/os2threads.h b/compat/os2threads.h index 7c0fe13..12cb7b0 1

[FFmpeg-devel] [PATCH v2 2/3] compat/os2threads: support the return value of joined thread

2016-02-14 Thread KO Myung-Hun
--- compat/os2threads.h | 39 ++- 1 file changed, 18 insertions(+), 21 deletions(-) diff --git a/compat/os2threads.h b/compat/os2threads.h index 12cb7b0..3b289df 100644 --- a/compat/os2threads.h +++ b/compat/os2threads.h @@ -35,9 +35,15 @@ #include #include

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-14 Thread Carl Eugen Hoyos
James Almer gmail.com> writes: > How do you plan to get around the fact the only working > version of this library is an obscure outdated fork that > doesn't even compile on most OSes I am sorry, I did not realize this: On which system does it not compile? Carl Eugen

Re: [FFmpeg-devel] [PATCH] avcodec: Remove libutvideo support

2016-02-14 Thread James Almer
On 2/14/2016 12:20 PM, Carl Eugen Hoyos wrote: > James Almer gmail.com> writes: > >> How do you plan to get around the fact the only working >> version of this library is an obscure outdated fork that >> doesn't even compile on most OSes > > I am sorry, I did not realize this: > On which syste

[FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Michael Niedermayer
This was suggested by wm4 and stefano. After this patch using align=1 the size used by various functions would not contain padding, while the palette would be aligned at align>1 This patch makes it required to use align>=4 if the palette is to be accessed as uint32 As a side-effect It fixes stori

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Mats Peterson
On 02/14/2016 04:25 PM, Michael Niedermayer wrote: This was suggested by wm4 and stefano. After this patch using align=1 the size used by various functions would not contain padding, while the palette would be aligned at align>1 This patch makes it required to use align>=4 if the palette is to b

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Mats Peterson
On 02/14/2016 04:25 PM, Michael Niedermayer wrote: As a side-effect It fixes storing pal8 in nut with odd with&height It worked fine before this patch (palette starting at a 4-byte boundary). ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: av_image_get_buffer_size(): do not insert padding between stride*height and palette

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 02:00:24PM +0100, Stefano Sabatini wrote: > On date Saturday 2016-02-13 23:08:50 +0100, wm4 encoded: > > On Sat, 13 Feb 2016 21:51:48 +0100 > > Michael Niedermayer wrote: > > > > > On Sat, Feb 13, 2016 at 08:46:34PM +0100, wm4 wrote: > > > > On Sat, 13 Feb 2016 19:38:01 +0

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Mats Peterson
On 02/14/2016 04:28 PM, Mats Peterson wrote: On 02/14/2016 04:25 PM, Michael Niedermayer wrote: As a side-effect It fixes storing pal8 in nut with odd with&height It worked fine before this patch (palette starting at a 4-byte boundary). OK, sorry, I get it. Mats -- Mats Peterson http://

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Mats Peterson
On 02/14/2016 04:30 PM, Mats Peterson wrote: On 02/14/2016 04:28 PM, Mats Peterson wrote: On 02/14/2016 04:25 PM, Michael Niedermayer wrote: As a side-effect It fixes storing pal8 in nut with odd with&height It worked fine before this patch (palette starting at a 4-byte boundary). OK, so

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Henrik Gramner
On Fri, Jan 1, 2016 at 3:19 PM, Michael Niedermayer wrote: > Hi all > > Its a while since 2.8 so unless there are objections i will make a > 2.9 or if people prefer a 3.0 within the next month or so The Ubuntu 16.04 LTS feature freeze is coming up next week, so it'd be nice to have a release befo

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread James Almer
On 2/14/2016 12:54 PM, Henrik Gramner wrote: > On Fri, Jan 1, 2016 at 3:19 PM, Michael Niedermayer > wrote: >> Hi all >> >> Its a while since 2.8 so unless there are objections i will make a >> 2.9 or if people prefer a 3.0 within the next month or so > > The Ubuntu 16.04 LTS feature freeze is co

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Carl Eugen Hoyos
Henrik Gramner gramner.com> writes: > The Ubuntu 16.04 LTS feature freeze is coming up next week, > so it'd be nice to have a release before that. (I am all for it, I am not sure if it is still possible though.) The following tickets are showstoppers imo: #5215 #5090 and #5216 I wonder if the

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread James Almer
On 2/14/2016 1:04 PM, Carl Eugen Hoyos wrote: > Henrik Gramner gramner.com> writes: > >> The Ubuntu 16.04 LTS feature freeze is coming up next week, >> so it'd be nice to have a release before that. > > (I am all for it, I am not sure if it is still possible though.) > > The following tickets

Re: [FFmpeg-devel] [PATCH] checkasm: Add vf_blend tests

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 10:06:16AM +0100, Paul B Mahol wrote: > On 2/14/16, Timothy Gu wrote: > > + > > +#include > > +#include "checkasm.h" > > +#include "libavfilter/blend.h" > > +#include "libavutil/common.h" > > +#include "libavutil/internal.h" > > +#include "libavutil/intreadwrite.h" > > + >

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread wm4
On Sun, 14 Feb 2016 13:07:04 -0300 James Almer wrote: > On 2/14/2016 1:04 PM, Carl Eugen Hoyos wrote: > > Henrik Gramner gramner.com> writes: > > > >> The Ubuntu 16.04 LTS feature freeze is coming up next week, > >> so it'd be nice to have a release before that. > > > > (I am all for it,

Re: [FFmpeg-devel] [PATCH 1/2] vf_blend: Templatize identity function and use a better name

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 10:07:39AM +0100, Paul B Mahol wrote: > On 2/14/16, Timothy Gu wrote: > > --- > > libavfilter/vf_blend.c | 22 ++ > > 1 file changed, 14 insertions(+), 8 deletions(-) > > > > lgtm Set pushed. Timothy ___ ff

Re: [FFmpeg-devel] [PATCH] vf_blend: Reduce number of arguments for kernel function

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 10:12:21AM +0100, Paul B Mahol wrote: > On 2/14/16, Timothy Gu wrote: > > --- > > libavfilter/blend.h | 2 +- > > libavfilter/vf_blend.c | 27 ++- > > libavfilter/x86/vf_blend.asm| 3 +-- > > libavfilter/x86/vf_blend_init.

Re: [FFmpeg-devel] [PATCH] checkasm: Add vf_blend tests

2016-02-14 Thread Michael Niedermayer
On Sat, Feb 13, 2016 at 03:04:58PM -0800, Timothy Gu wrote: > --- > tests/checkasm/Makefile | 1 + > tests/checkasm/checkasm.c | 3 ++ > tests/checkasm/checkasm.h | 1 + > tests/checkasm/vf_blend.c | 130 > ++ > 4 files changed, 135 insertions(+

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 04:04:29PM +, Carl Eugen Hoyos wrote: > Henrik Gramner gramner.com> writes: > > > The Ubuntu 16.04 LTS feature freeze is coming up next week, > > so it'd be nice to have a release before that. > > (I am all for it, I am not sure if it is still possible though.) > >

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Γιώργος Μεταξάκης
I have also fixed the #5008 have anyone checked it ? [patch] gdigrab-mouse-dpi-awareness On Sun, Feb 14, 2016 at 8:03 PM Michael Niedermayer wrote: > On Sun, Feb 14, 2016 at 04:04:29PM +, Carl Eugen Hoyos wrote: > > Henrik Gramner gramner.com> writes: > > > > > The Ubuntu 16.04 LTS feature

Re: [FFmpeg-devel] [PATCH v4] libavcodec/dnxhd_parser: add parser and probe support raw 444 and dnxhr formats

2016-02-14 Thread Michael Niedermayer
On Sat, Feb 13, 2016 at 09:44:32PM -0800, Mark Reid wrote: > --- > libavcodec/dnxhd_parser.c | 7 +++ > libavcodec/dnxhddata.c| 8 > libavcodec/dnxhddata.h| 18 +- > libavcodec/dnxhddec.c | 12 > libavformat/dnxhddec.c| 7 --- > 5 fi

Re: [FFmpeg-devel] [PATCH v2 1/3] compat/os2threads: Improve pthread_cond_xxx() functions

2016-02-14 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 12:20:33AM +0900, KO Myung-Hun wrote: > 1. Manipulate waiting count in pthread_cond_wait() > 2. Use builtin atomic functions to manipulate waiting count > --- > compat/os2threads.h | 26 +++--- > 1 file changed, 15 insertions(+), 11 deletions(-) applied

Re: [FFmpeg-devel] [PATCH v2 3/3] compat/os2threads: split long lines

2016-02-14 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 12:20:35AM +0900, KO Myung-Hun wrote: > --- > compat/os2threads.h | 17 - > 1 file changed, 12 insertions(+), 5 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Observe your enemies, for they f

Re: [FFmpeg-devel] [PATCH v2 2/3] compat/os2threads: support the return value of joined thread

2016-02-14 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 12:20:34AM +0900, KO Myung-Hun wrote: > --- > compat/os2threads.h | 39 ++- > 1 file changed, 18 insertions(+), 21 deletions(-) applied btw, you should add yourself to the MAINTAINERS file i think, as you de facto maintain this code ...

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 06:04:36PM +, Γιώργος Μεταξάκης wrote: > I have also fixed the #5008 have anyone checked it ? > [patch] gdigrab-mouse-dpi-awareness maybe matt or someone else who worked on similar stuff can take a look at the code > > On Sun, Feb 14, 2016 at 8:03 PM Michael Niederma

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 04:04:29PM +, Carl Eugen Hoyos wrote: > Henrik Gramner gramner.com> writes: > > > The Ubuntu 16.04 LTS feature freeze is coming up next week, > > so it'd be nice to have a release before that. > > (I am all for it, I am not sure if it is still possible though.) > >

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 07:34:26PM +0100, Michael Niedermayer wrote: > On Sun, Feb 14, 2016 at 04:04:29PM +, Carl Eugen Hoyos wrote: > > Henrik Gramner gramner.com> writes: > > > > > The Ubuntu 16.04 LTS feature freeze is coming up next week, > > > so it'd be nice to have a release before th

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

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 02:54:29AM +0100, Marton Balint wrote: > > I have applied the series except the last one patching ffmpeg_opt. This set seems to cause some issues in some FATE stations: http://fatebeta.ffmpeg.org/report/armel5tej-qemu-debian-gcc4.4/20160214170733#failed_tests http://fateb

Re: [FFmpeg-devel] [PATCH] x86/vc1dsp: Port vc1_*_hor_16b_shift2 to NASM format

2016-02-14 Thread Timothy Gu
On Sun, Feb 14, 2016 at 01:21:57PM +0100, Christophe Gisquet wrote: > Hi, > > 2016-02-14 6:49 GMT+01:00 Timothy Gu : > > %if HAVE_MMX_INLINE > > Isn't that macro meant for C code (and in config.asm without much of a > purpose)? Yes, but this code isn't used unless inline asm is enabled so I don

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

2016-02-14 Thread Marton Balint
On Sun, 14 Feb 2016, Timothy Gu wrote: On Sun, Feb 14, 2016 at 02:54:29AM +0100, Marton Balint wrote: I have applied the series except the last one patching ffmpeg_opt. This set seems to cause some issues in some FATE stations: http://fatebeta.ffmpeg.org/report/armel5tej-qemu-debian-gcc4.4

Re: [FFmpeg-devel] [PATCH] avformat/asfenc: write group_mutual_exclusion_objects on multiple languages

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 05:03:26AM +0100, Marton Balint wrote: > > On Sun, 14 Feb 2016, Michael Niedermayer wrote: > > >On Sun, Feb 07, 2016 at 11:02:56PM +0100, Marton Balint wrote: > >>Improves streaming compatibility with Windows Media Services. Also tested > >>for > >>compatilbility in Windo

[FFmpeg-devel] [PATCH] avcodec/cfhd: clear idwt_buf on allocation

2016-02-14 Thread Michael Niedermayer
This avoids use of uninitialized variables and might make bugs in general easier to reproduce Signed-off-by: Michael Niedermayer --- libavcodec/cfhd.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index 5c15d9b..d8e183b 100644 --- a/

[FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/cfhd.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index 5c15d9b..97da320 100644 --- a/libavcodec/cfhd.c +++ b/libavcodec/cfhd.c @@ -756,6 +756,6 @@ AVCodec ff_cfhd_decoder = {

Re: [FFmpeg-devel] [PATCH] avformat/genh: Mark coef_splitted as av_unused

2016-02-14 Thread Michael Niedermayer
On Sun, Oct 18, 2015 at 08:42:37PM +0200, Michael Niedermayer wrote: > From: Michael Niedermayer > > This avoid "libavformat/genh.c:43:14: warning: variable coef_splitted set but > not used" > Fewer warnings makes it easier to see new and important warnings applied [..] -- Michael GnuPG f

Re: [FFmpeg-devel] [PATCH] avcodec/h264_cabac: Check decode_cabac_mb_mvd() for failure

2016-02-14 Thread Michael Niedermayer
On Tue, Jan 26, 2016 at 02:47:43AM +0100, Michael Niedermayer wrote: > From: Michael Niedermayer > > Fixes harmless integer overflow > Fixes Ticket5150 > > No speedloss measured, actually its slightly faster, but please benchmark & > double check this > > Signed-off-by: Michael Niedermayer >

Re: [FFmpeg-devel] [PATCH] doc/protocols: document protocol_whitelist

2016-02-14 Thread Michael Niedermayer
On Sat, Feb 13, 2016 at 06:01:36PM +0100, Stefano Sabatini wrote: > On date Saturday 2016-02-13 17:01:59 +0100, Michael Niedermayer encoded: > > Signed-off-by: Michael Niedermayer > > --- > > doc/protocols.texi | 19 +++ > > 1 file changed, 19 insertions(+) > > > > diff --git a

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

2016-02-14 Thread Paul B Mahol
Hi, patch attached. From 851f9cb101d8a23b8f35679ca83fce5a7c122101 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sun, 14 Feb 2016 19:02:42 +0100 Subject: [PATCH] avfilter: add fieldhint filter Signed-off-by: Paul B Mahol --- libavfilter/Makefile | 1 + libavfilter/allfilters.c |

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > > The following tickets are showstoppers imo: > > #5090 and #5216 > > 5090 and 5216 should be fixed by: > 0212 11:21 Rodger Combs(2.0K) └─&─&─&─&─&─> > Re: [FFmpeg-devel] [PATCH] lavf/mov: add > support for sidx fragment indexes > please test an

[FFmpeg-devel] [PATCHv2] avformat/asfenc: write group_mutual_exclusion_objects for audio on multiple languages

2016-02-14 Thread Marton Balint
Improves streaming compatibility with Windows Media Services. Also tested for compatilbility in Windows Media Player, Windows Media ASF Viewer and VLC. This version of the patch only writes exclusion among audio streams, therefore choosing a subtitle language should be possible independently of au

Re: [FFmpeg-devel] [PATCH 2/2] x86: use the new helper macros where useful

2016-02-14 Thread James Almer
On 2/11/2016 10:09 PM, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/x86/hevcdsp_init.c | 11 ++- > libavcodec/x86/huffyuvencdsp_mmx.c | 2 +- > libavcodec/x86/jpeg2000dsp_init.c | 2 +- > libavcodec/x86/mlpdsp_init.c

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Fri, Jan 01, 2016 at 03:19:21PM +0100, Michael Niedermayer wrote: > Hi all > > Its a while since 2.8 so unless there are objections i will make a > 2.9 or if people prefer a 3.0 within the next month or so > > Also ill likely make another round of point releases from the > 2.8/2.7/2.6/2.5 bran

Re: [FFmpeg-devel] [PATCH 2/2] x86: use the new helper macros where useful

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 07:04:44PM -0300, James Almer wrote: > On 2/11/2016 10:09 PM, James Almer wrote: > > Signed-off-by: James Almer > > --- > > libavcodec/x86/hevcdsp_init.c | 11 ++- > > libavcodec/x86/huffyuvencdsp_mmx.c | 2 +- > > libavcodec/x86/jpeg2000dsp

Re: [FFmpeg-devel] [PATCH 2/2] x86: use the new helper macros where useful

2016-02-14 Thread James Almer
On 2/14/2016 7:50 PM, Michael Niedermayer wrote: > On Sun, Feb 14, 2016 at 07:04:44PM -0300, James Almer wrote: >> On 2/11/2016 10:09 PM, James Almer wrote: >>> Signed-off-by: James Almer >>> --- >>> libavcodec/x86/hevcdsp_init.c | 11 ++- >>> libavcodec/x86/huffyuvencdsp_mm

Re: [FFmpeg-devel] [patch] gdigrab-mouse-dpi-awareness

2016-02-14 Thread Matt Oliver
On 13 February 2016 at 05:33, Γιώργος Μεταξάκης wrote: > checked with patcheck also. > only "problem" is > "Missing changelog entry (ignore if minor change)' > LGTM, applied. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailm

[FFmpeg-devel] [PATCH/RFC]lavf/mpegts: Detect Private Stream 1 as teletext

2016-02-14 Thread Carl Eugen Hoyos
Hi! Attached patch fixes teletext detection for a sample that was uploaded by a user in the thread "Which command line can do dvb_teletext from ts to mp4" on the user mailing list: http://we.tl/Zg2yq1H136 I am glad if somebody could explain how this should be done differently for the given str

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread Carl Eugen Hoyos
Michael Niedermayer niedermayer.cc> writes: > -.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS, > +.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS > | AV_CODEC_CAP_EXPERIMENTAL, Please edit the Changelog accordingly: - Experimental Cineform HD decoder Th

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread James Almer
On 2/14/2016 8:59 PM, Carl Eugen Hoyos wrote: > Michael Niedermayer niedermayer.cc> writes: > >> -.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS, >> +.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_FRAME_THREADS >> | AV_CODEC_CAP_EXPERIMENTAL, > > Please edit the Chan

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread Carl Eugen Hoyos
James Almer gmail.com> writes: > The patch was rejected on irc and dropped. Reviews do not take place on the mailing list anymore? Or are the crashes fixed? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/l

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread James Almer
On 2/14/2016 9:08 PM, Carl Eugen Hoyos wrote: > James Almer gmail.com> writes: > >> The patch was rejected on irc and dropped. > > Reviews do not take place on the mailing list anymore? > > Or are the crashes fixed? Afaik, When that happens the author of the patch usually replies announcing th

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 08:50:28PM +0100, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/cfhd.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) patch droped as there where objections against it see IRC logs of today [...] -- Michael GnuPG fing

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: mark as experimental, there are some unsolved crashes

2016-02-14 Thread Ronald S. Bultje
Hi, On Sun, Feb 14, 2016 at 7:08 PM, Carl Eugen Hoyos wrote: > James Almer gmail.com> writes: > > > The patch was rejected on irc and dropped. > > Reviews do not take place on the mailing list anymore? Sorry, my fault, I should've copied that comment on the ML. Most crashes are fixed, one is

Re: [FFmpeg-devel] Fwd: Fwd: libavformat/segment : add option to increment timecode

2016-02-14 Thread Stefano Sabatini
On date Sunday 2016-02-14 15:41:05 +0100, Martin Vignali encoded: [...] > Hello, > > New patch attach, i move variable declaration at the top (for > AVDictionnaryEntry too). > I also remove the return err, in case of an error occured in the > av_timecode_init_from_string function. > Instead i put

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Stefano Sabatini
On date Sunday 2016-02-14 16:25:02 +0100, Michael Niedermayer encoded: > This was suggested by wm4 and stefano. > After this patch using align=1 the size used by various functions would not > contain padding, while the palette would be aligned at align>1 > > This patch makes it required to use ali

Re: [FFmpeg-devel] [FFmpeg-mentors] New task ideas for GSoC and Outreachy

2016-02-14 Thread Stefano Sabatini
On date Wednesday 2016-02-10 16:21:09 +, Kieran Kunhya encoded: > Hello, > > It seems that we're trotting out the same old GSoC and Outreachy tasks that > students haven't picked for years. > > So, in the spirit of open source, I'd like to ask you, the developers and > users what students sho

Re: [FFmpeg-devel] [PATCH] avutil/imgutils: remove special case for aligning the palette

2016-02-14 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 02:44:26AM +0100, Stefano Sabatini wrote: > On date Sunday 2016-02-14 16:25:02 +0100, Michael Niedermayer encoded: > > This was suggested by wm4 and stefano. > > After this patch using align=1 the size used by various functions would not > > contain padding, while the palett

[FFmpeg-devel] [PATCH v3 2/2] lavc/ccaption_dec: implement special and extended character sets

2016-02-14 Thread Aman Gupta
From: Aman Gupta character sets implemented as defined in https://en.wikipedia.org/wiki/EIA-608#Characters --- libavcodec/ccaption_dec.c | 152 +- 1 file changed, 149 insertions(+), 3 deletions(-) diff --git a/libavcodec/ccaption_dec.c b/libavcodec/c

Re: [FFmpeg-devel] [PATCH/RFC]lavf/mpegts: Detect Private Stream 1 as teletext

2016-02-14 Thread Marton Balint
On Mon, 15 Feb 2016, Carl Eugen Hoyos wrote: Hi! Attached patch fixes teletext detection for a sample that was uploaded by a user in the thread "Which command line can do dvb_teletext from ts to mp4" on the user mailing list: http://we.tl/Zg2yq1H136 I am glad if somebody could explain how this

[FFmpeg-devel] [PATCH v3 1/2] lavc/ccaption_dec: do not ignore repeated character commands

2016-02-14 Thread Aman Gupta
From: Aman Gupta control codes in a cc stream can be repeated, and must be ignored. however, repeated characters must not be ignored. the code attempted to wipe prev_cmd in handle_char to allow repeated characters to be processed, but prev_cmd would previously get reset _after_ handle_char() i a

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as, part of the returned frame

2016-02-14 Thread Alexander Agranovsky
Thanks - any problems with 2/3? Or is it still pending? On 2/14/16 8:53 AM, Michael Niedermayer wrote: On Sat, Feb 13, 2016 at 11:49:06PM -0500, Alexander Agranovsky wrote: mpjpegdec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 0c761f422ba83446d20728449c0c6813e4e11560 000

Re: [FFmpeg-devel] 2.9/3.0, 2.8.5, ...

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 11:08:35PM +0100, Michael Niedermayer wrote: > On Fri, Jan 01, 2016 at 03:19:21PM +0100, Michael Niedermayer wrote: > > Hi all > > > > Its a while since 2.8 so unless there are objections i will make a > > 2.9 or if people prefer a 3.0 within the next month or so > > > > A

Re: [FFmpeg-devel] [PATCH 3/3] lavf/mpjpeg: do not include CRLF preceding boundary as, part of the returned frame

2016-02-14 Thread Michael Niedermayer
On Sun, Feb 14, 2016 at 10:22:30PM -0500, Alexander Agranovsky wrote: > Thanks - any problems with 2/3? Or is it still pending? i saw a case where it caused much more data to be read in probing i didnt had time to look yet so thats not a too informative bug report, dont even know if it is a bug .

Re: [FFmpeg-devel] [PATCH v2 2/3] compat/os2threads: support the return value of joined thread

2016-02-14 Thread KO Myung-Hun
Michael Niedermayer wrote: > On Mon, Feb 15, 2016 at 12:20:34AM +0900, KO Myung-Hun wrote: >> --- >> compat/os2threads.h | 39 ++- >> 1 file changed, 18 insertions(+), 21 deletions(-) > > applied > > btw, you should add yourself to the MAINTAINERS file i thi

[FFmpeg-devel] Help needed with muxing issues pertaining to raw AVI and QuickTime video

2016-02-14 Thread Mats Peterson
I'm not very confident on the muxing side, therefore I would be grateful if anyone able to look outside of his own agenda could look into possible solutions in order to solve the following issues pertaining to writing AVI and QuickTime files created with "-vcodec rawvideo": 1. Incorrect stride

Re: [FFmpeg-devel] Help needed with muxing issues pertaining to raw AVI and QuickTime video

2016-02-14 Thread Mats Peterson
On 02/15/2016 05:38 AM, Mats Peterson wrote: 2. Palette being included in the frames. This is OK for nut, but it's clearly non-standard behaviour for AVI and QuickTime, and it makes the files unplayable with anything else than FFplay. Well, it's perhaps not so much the inclusion of the palette

Re: [FFmpeg-devel] Help needed with muxing issues pertaining to raw AVI and QuickTime video

2016-02-14 Thread Mats Peterson
On 02/15/2016 05:38 AM, Mats Peterson wrote: I'm not very confident on the muxing side, therefore I would be grateful if anyone able to look outside of his own agenda could look into possible solutions in order to solve the following issues pertaining to writing AVI and QuickTime files created wi

Re: [FFmpeg-devel] Help needed with muxing issues pertaining to raw AVI and QuickTime video

2016-02-14 Thread Mats Peterson
On 02/15/2016 07:00 AM, Mats Peterson wrote: Files for testing are available at the links below. Get the *bpp_129.* files. They are odd-width files (129x97 pixels). https://drive.google.com/open?id=0B3_pEBoLs0faMnU1NUVMdXdsOE0 https://drive.google.com/open?id=0B3_pEBoLs0faWElmM2FnLTZYNlk The

[FFmpeg-devel] Codec for GPS

2016-02-14 Thread Chau Pham
Hi Seniors what codec does ffmpeg use for GPS? Thank in advance Tim ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel