Re: [FFmpeg-devel] [PATCH 2/2] decklink: enhancement: mode selection with AVRational

2016-07-09 Thread Marton Balint
On Sat, 9 Jul 2016, Matthias Hunstock wrote: When finding a video mode based on frame size and frame rate, use AVRational methods instead of simple comparison. Also fixes a swap of num and den and add runtime debug level information. I am a bit against renaming num and den to the names in

Re: [FFmpeg-devel] [PATCH 1/2] decklink: fix: set fps on video stream

2016-07-09 Thread Marton Balint
On Sat, 9 Jul 2016, Matthias Hunstock wrote: Provide the stream with the correct fps information when capturing. Signed-off-by: Matthias Hunstock --- libavdevice/decklink_dec.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/libavdevice/decklink_dec.cpp

Re: [FFmpeg-devel] [PATCH] avcodec/vaapi_encode_h264: Use av_clip_uintp2()

2016-07-09 Thread Michael Niedermayer
On Sun, Jul 03, 2016 at 10:41:20PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavcodec/vaapi_encode_h264.c |4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-07-09 Thread Michael Niedermayer
On Sat, Jul 09, 2016 at 10:16:20PM +0200, Paul B Mahol wrote: > On 7/8/16, sami.h...@gmail.com wrote: > > From: Sami Hult > > > > This is an - once again - updated patch, that uses avio_write instead > > of avio_puts to stream clean text output without

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-07-09 Thread James Almer
On 7/9/2016 4:56 PM, Michael Niedermayer wrote: > On Sat, Jul 09, 2016 at 04:07:14PM -0300, James Almer wrote: >> On 7/9/2016 3:28 PM, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Sat, Jul 9, 2016 at 1:38 PM, James Almer wrote: [14:31:33]

Re: [FFmpeg-devel] [PATCH] Changed metadata print option to accept general urls

2016-07-09 Thread Paul B Mahol
On 7/8/16, sami.h...@gmail.com wrote: > From: Sami Hult > > This is an - once again - updated patch, that uses avio_write instead > of avio_puts to stream clean text output without null characters. Works > now for me as intended. > > Changes metadata

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-07-09 Thread Michael Niedermayer
On Sat, Jul 09, 2016 at 04:07:14PM -0300, James Almer wrote: > On 7/9/2016 3:28 PM, Ronald S. Bultje wrote: > > Hi, > > > > On Sat, Jul 9, 2016 at 1:38 PM, James Almer wrote: > >> [14:31:33] > >> https://github.com/mpc-hc/mpc-hc/commit/fe1b4ebd1ab69109c898fd4aa250013e18d2d116

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Thomas Volkert
On 09.07.2016 19:37, Martin Vignali wrote: You're right, this is a very complex file format, with lot of features. indeed.. But i disagree, with the idea, that this decoder have no interest. The "useful question", highly depend of the point of view. I write this decoder, because i sometime

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-07-09 Thread James Almer
On 7/9/2016 3:28 PM, Ronald S. Bultje wrote: > Hi, > > On Sat, Jul 9, 2016 at 1:38 PM, James Almer wrote: >> [14:31:33] >> https://github.com/mpc-hc/mpc-hc/commit/fe1b4ebd1ab69109c898fd4aa250013e18d2d116 >> looks like some projects using ffmpeg are disabling tree

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-07-09 Thread Ronald S. Bultje
Hi, On Sat, Jul 9, 2016 at 1:38 PM, James Almer wrote: > On 5/8/2016 4:00 AM, Reimar Döffinger wrote: > > On 07.05.2016, at 02:56, Hendrik Leppkes wrote: > > > >> On Sat, May 7, 2016 at 2:02 AM, James Almer wrote: > >>> On 5/6/2016

Re: [FFmpeg-devel] latest ffmpeg git master build is broken on osx powerpc

2016-07-09 Thread Michael Niedermayer
adding jingyu (author of 311a953c76081fca99b872629d248f9d69ebc0c3) to CC On Sat, Jul 09, 2016 at 07:47:26AM -0600, Pavel Koshevoy wrote: > Hi, > > PowerPC build on OSX 10.5 (Leopard) is broken: > > $ gcc --version > powerpc-apple-darwin9-gcc-4.2.1 (GCC) 4.2.1 (Apple Inc. build 5577) > Copyright

Re: [FFmpeg-devel] [PATCH 2/2] configure: Enable GCC vectorization on ≥4.9

2016-07-09 Thread James Almer
On 5/8/2016 4:00 AM, Reimar Döffinger wrote: > On 07.05.2016, at 02:56, Hendrik Leppkes wrote: > >> On Sat, May 7, 2016 at 2:02 AM, James Almer wrote: >>> On 5/6/2016 8:48 PM, Timothy Gu wrote: On Fri, May 06, 2016 at 12:08:14PM +0200, Hendrik

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Martin Vignali
Thanks for your comments @James Almer : MKTAG, is maybe more comprehensive, than the 32i value. Modify locally @Carl Eugen Hoyos : > > +{ AV_CODEC_ID_EXR, MKTAG('p', 's', 'd', ' ') }, /* Psd */ > > What produces such files? > I think i do a mistake here. Remove locally >

[FFmpeg-devel] [PATCH 2/2] decklink: enhancement: mode selection with AVRational

2016-07-09 Thread Matthias Hunstock
When finding a video mode based on frame size and frame rate, use AVRational methods instead of simple comparison. Also fixes a swap of num and den and add runtime debug level information. Signed-off-by: Matthias Hunstock --- libavdevice/decklink_common.cpp | 22

[FFmpeg-devel] [PATCH 1/2] decklink: fix: set fps on video stream

2016-07-09 Thread Matthias Hunstock
Provide the stream with the correct fps information when capturing. Signed-off-by: Matthias Hunstock --- libavdevice/decklink_dec.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp index fcb024e..c25579a

Re: [FFmpeg-devel] libavdevice/decklink: advanced options for selecting an input connection

2016-07-09 Thread Matthias Hunstock
Am 03.07.2016 um 18:22 schrieb Michael Niedermayer: >>> Also it adds list_vinputs and list_ainputs >> There is no attachment? > there seems to be a pull req on github about this > https://github.com/FFmpeg/FFmpeg/pull/212 Doesnt really matter anymore, those new functionalities are also

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Paul B Mahol
On 7/9/16, Rostislav Pehlivanov wrote: > On 9 July 2016 at 16:29, Martin Vignali wrote: > >> Hello, >> >> In attach patch to add decoding of .psd file (Photoshop file). >> >> PSD file can contain merge layers data at the end of the file (Image Data

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Rostislav Pehlivanov
On 9 July 2016 at 16:29, Martin Vignali wrote: > Hello, > > In attach patch to add decoding of .psd file (Photoshop file). > > PSD file can contain merge layers data at the end of the file (Image Data > Section) > > This patch add support for this kind of files, using

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Paul B Mahol
On 7/9/16, Martin Vignali wrote: > Hello, > > In attach patch to add decoding of .psd file (Photoshop file). > > PSD file can contain merge layers data at the end of the file (Image Data > Section) > > This patch add support for this kind of files, using uncompress data

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread James Almer
On 7/9/2016 1:09 PM, James Almer wrote: > On 7/9/2016 12:29 PM, Martin Vignali wrote: >> +static int psd_probe(AVProbeData *p) >> +{ >> +const uint8_t *b = p->buf; >> + >> +if (AV_RL32(b) == 1397768760) > > if (AV_RL32(b) == MKTAG('S','P','B','8')) > MKTAG('8','B','P','S'), sorry.

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Carl Eugen Hoyos
Martin Vignali gmail.com> writes: > psd_probe() Please also test for version (and color mode) and increase the probe score / if easily possible, don't detect things that are immediately rejected by the decoder. And don't remove the empty line from Changelog... > +{ AV_CODEC_ID_EXR,

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

2016-07-09 Thread Carl Eugen Hoyos
Josh de Kock itanimul.li> writes: > On Thu, Jul 7, 2016, at 10:47 AM, Carl Eugen Hoyos wrote: > > Josh de Kock itanimul.li> writes: > > > > > +ret = openmpt_could_open_propability( > > > > I don't know anything about openmpt but what was > > wrong with the original probe function sent by

Re: [FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread James Almer
On 7/9/2016 12:29 PM, Martin Vignali wrote: > Hello, > > In attach patch to add decoding of .psd file (Photoshop file). > > PSD file can contain merge layers data at the end of the file (Image Data > Section) > > This patch add support for this kind of files, using uncompress data Image > Data

Re: [FFmpeg-devel] libavcodec/webp : fix return code check in lossy decoding

2016-07-09 Thread Martin Vignali
2016-07-09 12:18 GMT+02:00 Michael Niedermayer : > On Sat, Jul 09, 2016 at 11:53:04AM +0200, Hendrik Leppkes wrote: > > On Sat, Jul 9, 2016 at 11:13 AM, Michael Niedermayer > > wrote: > > > On Sun, Jun 26, 2016 at 10:45:33PM +0200, Martin Vignali

[FFmpeg-devel] libavcodec : add psd image file decoder

2016-07-09 Thread Martin Vignali
Hello, In attach patch to add decoding of .psd file (Photoshop file). PSD file can contain merge layers data at the end of the file (Image Data Section) This patch add support for this kind of files, using uncompress data Image Data Section Support Gray/A, and RGB/A with 8b or 16 bits by

Re: [FFmpeg-devel] [PATCH] vp9: add 16x16 idct avx2 (8-bit).

2016-07-09 Thread James Almer
On 7/8/2016 6:59 PM, Ronald S. Bultje wrote: > +%if ARCH_X86_64 > +INIT_YMM avx2 Add an %if HAVE_AVX2_EXTERNAL check here, because yasm 1.1.0 and older don't support avx2. lgtm aside from that. > +cglobal vp9_idct_idct_16x16_add, 4, 4, 16, dst, stride, block, eob

[FFmpeg-devel] latest ffmpeg git master build is broken on osx powerpc

2016-07-09 Thread Pavel Koshevoy
Hi, PowerPC build on OSX 10.5 (Leopard) is broken: $ gcc --version powerpc-apple-darwin9-gcc-4.2.1 (GCC) 4.2.1 (Apple Inc. build 5577) Copyright (C) 2007 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for

Re: [FFmpeg-devel] [PATCH] lavc/libopenh264enc: update to openh264 1.6

2016-07-09 Thread Ricardo Constantino
On 23 May 2016 at 20:11, Michael Niedermayer wrote: > > it would be great to have support for both 1.6 and older versions > There's a patch by Martin Störjo in libav-devel that seems to allow support for both releases and git master. Tested just now in MinGW with git

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

2016-07-09 Thread Jörn Heusipp
On 06/30/2016 02:49 AM, Josh de Kock wrote: Fixes ticket #5623 TODO: bump lavf minor --- I looked more thoroughly at your patch again. I'm commenting from libopenmpt perspective of course. +openmpt->module = openmpt_module_create_from_memory(buf, size, openmpt_logfunc, s, NULL); +

Re: [FFmpeg-devel] [PATCHv4] lavf: add libopenmpt demuxer

2016-07-09 Thread Jörn Heusipp
On 07/08/2016 10:17 PM, Nicolas George wrote: Le primidi 21 messidor, an CCXXIV, Jörn Heusipp a écrit : Regarding AVProbeData: Looking at AVProbeData, I can see no (optional) field describing the file size: typedef struct AVProbeData { const char *filename; unsigned char *buf; /**<

Re: [FFmpeg-devel] [PATCH 5/5] lavf/segment: write attached pictures to all segments by default

2016-07-09 Thread Michael Niedermayer
On Mon, Jul 04, 2016 at 10:33:20AM -0700, Rodger Combs wrote: > --- > doc/muxers.texi | 4 > libavformat/segment.c | 33 + > libavformat/version.h | 2 +- > 3 files changed, 38 insertions(+), 1 deletion(-) probably ok a fate test for this would be

Re: [FFmpeg-devel] [PATCH 4/5] lavf/segment: copy stream dispositions in output

2016-07-09 Thread Michael Niedermayer
On Mon, Jul 04, 2016 at 10:33:19AM -0700, Rodger Combs wrote: > --- > libavformat/segment.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/segment.c b/libavformat/segment.c > index 9c766a5..5aec018 100644 > --- a/libavformat/segment.c > +++ b/libavformat/segment.c > @@

Re: [FFmpeg-devel] libavcodec/webp : fix return code check in lossy decoding

2016-07-09 Thread Michael Niedermayer
On Sat, Jul 09, 2016 at 11:53:04AM +0200, Hendrik Leppkes wrote: > On Sat, Jul 9, 2016 at 11:13 AM, Michael Niedermayer > wrote: > > On Sun, Jun 26, 2016 at 10:45:33PM +0200, Martin Vignali wrote: > >> Hello, > >> > >> in attach patch to fix ret code check in webp lossy

Re: [FFmpeg-devel] libavcodec/webp : fix return code check in lossy decoding

2016-07-09 Thread Hendrik Leppkes
On Sat, Jul 9, 2016 at 11:13 AM, Michael Niedermayer wrote: > On Sun, Jun 26, 2016 at 10:45:33PM +0200, Martin Vignali wrote: >> Hello, >> >> in attach patch to fix ret code check in webp lossy decoding (when files >> have alpha) >> >> before this patch, if an error

Re: [FFmpeg-devel] [PATCH]Support QT b64a ARGB64 rawvideo

2016-07-09 Thread Michael Niedermayer
On Sun, Jul 03, 2016 at 11:08:15PM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patches fix ticket #5657. > > Please comment, Carl Eugen > libavcodec/raw.c |1 + > libavcodec/rawdec.c |9 + > libavcodec/version.h |2 +- > libavformat/isom.c |1 + > 4 files

Re: [FFmpeg-devel] libavcodec/webp : fix return code check in lossy decoding

2016-07-09 Thread Michael Niedermayer
On Sun, Jun 26, 2016 at 10:45:33PM +0200, Martin Vignali wrote: > Hello, > > in attach patch to fix ret code check in webp lossy decoding (when files > have alpha) > > before this patch, if an error occured during rgb decoding > the return code was ignored. And a useless ret test was made >

Re: [FFmpeg-devel] [PATCH] hwcontext: add a QSV implementation

2016-07-09 Thread Michael Niedermayer
On Wed, Jul 06, 2016 at 03:51:12PM +0700, nablet developer wrote: > > > On 06 Jul 2016, at 15:48, Hendrik Leppkes wrote: > > > > On Wed, Jul 6, 2016 at 9:08 AM, nablet developer wrote: > >> add hwcontext_qsv (Intel QuickSync Video) implementation > >>