Re: [FFmpeg-devel] [PATCH] avcodec: Add bits_per_raw_sample to AVCodecParameters

2016-04-14 Thread James Almer
On 4/14/2016 7:30 PM, Michael Niedermayer wrote: > On Tue, Apr 12, 2016 at 01:25:43PM +0200, wm4 wrote: >> On Tue, 12 Apr 2016 13:10:09 +0200 >> Michael Niedermayer wrote: >> >>> On Mon, Apr 11, 2016 at 01:32:07AM +0200, Michael Niedermayer wrote: The

Re: [FFmpeg-devel] [PATCH] avformat/hashenc: simplify hash_write_trailer

2016-04-14 Thread James Almer
On 4/14/2016 1:34 PM, Michael Niedermayer wrote: > On Wed, Apr 13, 2016 at 10:58:22PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavformat/hashenc.c | 28 +--- >> 1 file changed, 5 insertions(+), 23 deletions(-) > > LGTM if it

[FFmpeg-devel] [PATCH 3/3] configure: Remove -Wredundant-decls on OS/2 to suppress noise.

2016-04-14 Thread Dave Yeo
From a417fdf752bd7c704ed5ba0c94c5cea96e1a91ff Mon Sep 17 00:00:00 2001 From: Dmitriy Kuminov Date: Thu, 14 Apr 2016 01:53:46 +0400 Subject: [PATCH 3/3] configure: Remove -Wredundant-decls on OS/2 to suppress noise. This warning causes too much noise in libc 0.6.6- headers due

[FFmpeg-devel] [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2.

2016-04-14 Thread Dave Yeo
From fbff28a583cc9fd7144dd4f9dc37b7b5f8aa Mon Sep 17 00:00:00 2001 From: Dmitriy Kuminov Date: Thu, 14 Apr 2016 01:45:01 +0400 Subject: [PATCH 2/3] configure: Do not create/install versioned DLLs on OS/2. Only one DLL for each module must be present on OS/2:

[FFmpeg-devel] [PATCH 1/3] configure: Support backslashes in TMPDIR on OS/2.

2016-04-14 Thread Dave Yeo
From b322802ecc6d87fae3afb17fbca8217993d8d0c7 Mon Sep 17 00:00:00 2001 From: Dmitriy Kuminov Date: Thu, 14 Apr 2016 01:41:30 +0400 Subject: [PATCH 1/3] configure: Support backslashes in TMPDIR on OS/2. Signed-off-by: Dave Yeo --- configure | 13

Re: [FFmpeg-devel] [PATCH] fate: Add test for ticket 1833 (Ogg/Vorbis Chapters)

2016-04-14 Thread James Almer
On 4/14/2016 10:18 PM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > tests/fate-run.sh |4 > tests/fate/vorbis.mak |7 ++- > tests/ref/fate/vorbis-1833-chapters | 36 >

[FFmpeg-devel] [PATCH] fate: Add test for ticket 1833 (Ogg/Vorbis Chapters)

2016-04-14 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- tests/fate-run.sh |4 tests/fate/vorbis.mak |7 ++- tests/ref/fate/vorbis-1833-chapters | 36 +++ 3 files changed, 46 insertions(+), 1 deletion(-)

Re: [FFmpeg-devel] [PATCH v5 2/3] avformat/tee: Fix leaks in tee muxer when open_slave fails

2016-04-14 Thread Marton Balint
On Thu, 14 Apr 2016, Marton Balint wrote: On Tue, 12 Apr 2016, sebechlebsky...@gmail.com wrote: From: Jan Sebechlebsky Calling close_slave in case error is to be returned from open_slave will free allocated resources. Since failure can happen before bsfs array

[FFmpeg-devel] [PATCH] avcodec/avcodec: Document signed/unsignedness of sample formats in relation to bits_per_raw_sample

2016-04-14 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavcodec/avcodec.h |8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 9e6169f..e5ba9aa 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avcodec.h @@

Re: [FFmpeg-devel] [PATCH] avcodec: Add bits_per_raw_sample to AVCodecParameters

2016-04-14 Thread Michael Niedermayer
On Tue, Apr 12, 2016 at 01:25:43PM +0200, wm4 wrote: > On Tue, 12 Apr 2016 13:10:09 +0200 > Michael Niedermayer wrote: > > > On Mon, Apr 11, 2016 at 01:32:07AM +0200, Michael Niedermayer wrote: > > > The bits_per_raw_sample represents the number of bits of precission per

Re: [FFmpeg-devel] [PATCH]lavc/dds: Fix GRAY8A decoding

2016-04-14 Thread Paul B Mahol
On 4/14/16, Carl Eugen Hoyos wrote: > Paul B Mahol gmail.com> writes: > >> > +else if (bpp == 16 && r == 0xff00 && g == 0 && >> b == 0 && a == 0xff) { >> > +avctx->pix_fmt = AV_PIX_FMT_YA8; >> > +ctx->postproc = DDS_SWAP_ALPHA; >> >> Why? You

Re: [FFmpeg-devel] [PATCH]lavc/dds: Fix GRAY8A decoding

2016-04-14 Thread Carl Eugen Hoyos
Paul B Mahol gmail.com> writes: > > +else if (bpp == 16 && r == 0xff00 && g == 0 && > b == 0 && a == 0xff) { > > +avctx->pix_fmt = AV_PIX_FMT_YA8; > > +ctx->postproc = DDS_SWAP_ALPHA; > > Why? You disabled postproc lines bellow. I don't understand this comment:

Re: [FFmpeg-devel] [PATCH] avcodec: Add bits_per_raw_sample to AVCodecParameters

2016-04-14 Thread Michael Niedermayer
On Tue, Apr 12, 2016 at 01:18:10PM +0200, Dominik 'Rathann' Mierzejewski wrote: > On Tuesday, 12 April 2016 at 13:10, Michael Niedermayer wrote: > > new version of this with improved documentation > > I intend to apply this soon if noone is against > > > > > > From

Re: [FFmpeg-devel] [PATCH]lavc/dds: Fix GRAY8A decoding

2016-04-14 Thread Paul B Mahol
On 4/14/16, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes the original sample of ticket #4667 for me. > > Please comment, Carl Eugen > > diff --git a/libavcodec/dds.c b/libavcodec/dds.c > index 9577b67..f364de7 100644 > --- a/libavcodec/dds.c > +++ b/libavcodec/dds.c >

[FFmpeg-devel] [PATCH]lavc/dds: Fix GRAY8A decoding

2016-04-14 Thread Carl Eugen Hoyos
Hi! Attached patch fixes the original sample of ticket #4667 for me. Please comment, Carl Eugen diff --git a/libavcodec/dds.c b/libavcodec/dds.c index 9577b67..f364de7 100644 --- a/libavcodec/dds.c +++ b/libavcodec/dds.c @@ -356,6 +356,10 @@ static int parse_pixel_format(AVCodecContext *avctx)

Re: [FFmpeg-devel] [PATCH 1/2 v4] avfilter: add readvitc filter

2016-04-14 Thread Michael Niedermayer
On Thu, Apr 14, 2016 at 08:49:58AM +0200, Tobias Rapp wrote: > On 08.04.2016 17:01, Tobias Rapp wrote: > >On 08.04.2016 15:24, Tobias Rapp wrote: > >>On 08.04.2016 14:54, Paul B Mahol wrote: > >>>On 4/8/16, Tobias Rapp wrote: > On 08.04.2016 12:48, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH 1/2 v4] avfilter: add readvitc filter

2016-04-14 Thread Michael Niedermayer
On Thu, Apr 14, 2016 at 08:49:58AM +0200, Tobias Rapp wrote: > On 08.04.2016 17:01, Tobias Rapp wrote: > >On 08.04.2016 15:24, Tobias Rapp wrote: > >>On 08.04.2016 14:54, Paul B Mahol wrote: > >>>On 4/8/16, Tobias Rapp wrote: > On 08.04.2016 12:48, Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCHv2] add signature filter for MPEG7 video signature

2016-04-14 Thread Michael Niedermayer
On Thu, Apr 14, 2016 at 07:06:29PM +0200, Gerion Entrup wrote: > On Montag, 11. April 2016 14:54:57 CEST Michael Niedermayer wrote: > > On Mon, Apr 11, 2016 at 02:30:37PM +0200, Gerion Entrup wrote: > > > On Montag, 11. April 2016 12:57:17 CEST Michael Niedermayer wrote: > > > > On Mon, Apr 11,

Re: [FFmpeg-devel] [PATCHv2] add signature filter for MPEG7 video signature

2016-04-14 Thread Gerion Entrup
On Montag, 11. April 2016 14:54:57 CEST Michael Niedermayer wrote: > On Mon, Apr 11, 2016 at 02:30:37PM +0200, Gerion Entrup wrote: > > On Montag, 11. April 2016 12:57:17 CEST Michael Niedermayer wrote: > > > On Mon, Apr 11, 2016 at 04:25:28AM +0200, Gerion Entrup wrote: > > > > On Donnerstag, 7.

Re: [FFmpeg-devel] [PATCH 1/3] doc/examples: Add fffuzz example

2016-04-14 Thread Michael Niedermayer
On Thu, Apr 14, 2016 at 02:33:14PM +, Kieran Kunhya wrote: > On Thu, 14 Apr 2016 at 15:07 Michael Niedermayer > wrote: > > > On Mon, Apr 11, 2016 at 11:21:55PM +0200, Pawel Golinski wrote: > > > There are some afl specific macros inside, > > > to make the example

Re: [FFmpeg-devel] [PATCH] avformat/hashenc: simplify hash_write_trailer

2016-04-14 Thread Michael Niedermayer
On Wed, Apr 13, 2016 at 10:58:22PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavformat/hashenc.c | 28 +--- > 1 file changed, 5 insertions(+), 23 deletions(-) LGTM if it produces the same output thx [...] -- Michael GnuPG

Re: [FFmpeg-devel] [PATCH 1/3] doc/examples: Add fffuzz example

2016-04-14 Thread Kieran Kunhya
On Thu, 14 Apr 2016 at 15:07 Michael Niedermayer wrote: > On Mon, Apr 11, 2016 at 11:21:55PM +0200, Pawel Golinski wrote: > > There are some afl specific macros inside, > > to make the example usable with afl fuzzer. > > --- > > doc/examples/fffuzz.c | 373 >

Re: [FFmpeg-devel] [PATCH 1/3] doc/examples: Add fffuzz example

2016-04-14 Thread Michael Niedermayer
On Mon, Apr 11, 2016 at 11:21:55PM +0200, Pawel Golinski wrote: > There are some afl specific macros inside, > to make the example usable with afl fuzzer. > --- > doc/examples/fffuzz.c | 373 > ++ > 1 file changed, 373 insertions(+) > create mode

Re: [FFmpeg-devel] [PATCH] lavc/mediacodec: add hwaccel support

2016-04-14 Thread Matthieu Bouron
On Thu, Apr 7, 2016 at 4:18 PM, wm4 wrote: > On Fri, 18 Mar 2016 17:50:39 +0100 > Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > --- > > > > Hello, > > Can't say much about this, so just some minor

Re: [FFmpeg-devel] [PATCH 1/2 v4] avfilter: add readvitc filter

2016-04-14 Thread Tobias Rapp
On 08.04.2016 17:01, Tobias Rapp wrote: On 08.04.2016 15:24, Tobias Rapp wrote: On 08.04.2016 14:54, Paul B Mahol wrote: On 4/8/16, Tobias Rapp wrote: On 08.04.2016 12:48, Carl Eugen Hoyos wrote: Tobias Rapp noa-archive.com> writes: AV_PIX_FMT_YUV440P? Also J