[FFmpeg-devel] [PATCH] avcodec/utils: only warn when passed invalid lowres value

2016-08-04 Thread Aman Gupta
From: Aman Gupta This makes it easier to use the lowres option when dealing with input files in different codecs. If the codec doesn't support lowres=1 for instance, it will throw a warning and use lowres=0 instead of erroring out completely. --- libavcodec/utils.c | 5 ++--- 1

Re: [FFmpeg-devel] [PATCH] Avoid sending packets to network when multicast ttl is 0 in udp

2016-08-04 Thread Michael Niedermayer
On Fri, Aug 05, 2016 at 01:20:12AM +0430, Omid Ghaffarinia wrote: > Thanks for your comment, actually 'code move' is necessary to make the > code compile because it is needed to use udp_set_url in > udp_set_multicast_ttl and the code is moved to make it possible. > I can make it in two separate

Re: [FFmpeg-devel] [PATCH 3/3] avformat/img2enc: Use AV_FRAME_FILENAME_FLAGS_MULTIPLE, support tee:

2016-08-04 Thread Michael Niedermayer
On Wed, Aug 03, 2016 at 08:55:23PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/img2enc.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 2/3] avformat/hlsenc: Use AV_FRAME_FILENAME_FLAGS_MULTIPLE, support tee:

2016-08-04 Thread Michael Niedermayer
On Wed, Aug 03, 2016 at 08:55:22PM +0200, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavformat/hlsenc.c | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 1/3] avformat: Add av_get_frame_filename2() and AV_FRAME_FILENAME_FLAGS_MULTIPLE

2016-08-04 Thread Michael Niedermayer
On Wed, Aug 03, 2016 at 08:55:21PM +0200, Michael Niedermayer wrote: > This will be used to allow writing file sequences using the tee output onto > multiple places in parallel > > Signed-off-by: Michael Niedermayer > --- > libavformat/avformat.h | 7 +++ >

Re: [FFmpeg-devel] [PATCH] Avoid sending packets to network when multicast ttl is 0 in udp

2016-08-04 Thread Omid Ghaffarinia
Thanks for your comment, actually 'code move' is necessary to make the code compile because it is needed to use udp_set_url in udp_set_multicast_ttl and the code is moved to make it possible. I can make it in two separate patches if needed, first to move udp_set_multicast_ttl without any further

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

2016-08-04 Thread Michael Niedermayer
On Sun, Jul 10, 2016 at 01:02:36PM +0200, Matthias Hunstock wrote: > From: atze This doesnt list your full name is that intended ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those who would give up essential Liberty,

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: clean up vdpau_internal.h

2016-08-04 Thread James Almer
On 8/4/2016 4:32 AM, Carl Eugen Hoyos wrote: > 2016-08-03 6:30 GMT+02:00 James Almer : >> Also don't include it on files that don't need it. >> >> This reduces differences with libav >> >> Signed-off-by: James Almer >> --- >> Untested with VDPAU enabled. > >

Re: [FFmpeg-devel] [PATCH] avformat: add a TTA Muxer

2016-08-04 Thread James Almer
On 8/4/2016 3:38 PM, Michael Niedermayer wrote: > On Sun, Jul 31, 2016 at 01:03:45PM -0300, James Almer wrote: >> Signed-off-by: James Almer > [...] >> +#include "apetag.h" >> +#include "avformat.h" >> +#include "avio_internal.h" >> +#include "internal.h" >> + >> +typedef

Re: [FFmpeg-devel] [PATCH] avformat: add a TTA Muxer

2016-08-04 Thread Michael Niedermayer
On Sun, Jul 31, 2016 at 01:03:45PM -0300, James Almer wrote: > Signed-off-by: James Almer [...] > +#include "apetag.h" > +#include "avformat.h" > +#include "avio_internal.h" > +#include "internal.h" > + > +typedef struct TTAMuxContext { > +AVIOContext *seek_table; > +

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

2016-08-04 Thread Martin Vignali
> > I think you should fail probe with less than 26 bytes > or are there smaller files that can be valid ? > > > Fix > > this is not based on git master > New patch in attach Martin From 7af7660813e2abe7174d12fef80f9198ee612192 Mon Sep 17 00:00:00 2001 From: Martin Vignali

Re: [FFmpeg-devel] [PATCH] avformat: add a TTA Muxer

2016-08-04 Thread James Almer
On 7/31/2016 1:03 PM, James Almer wrote: > Signed-off-by: James Almer > --- > Changelog | 1 + > doc/general.texi| 2 +- > libavformat/Makefile| 1 + > libavformat/allformats.c| 2 +- > libavformat/avio_internal.h | 2 + >

Re: [FFmpeg-devel] [PATCH] avcodec: don't include vdpau_compat.h when vdpau is not enabled

2016-08-04 Thread James Almer
On 8/4/2016 4:23 AM, Carl Eugen Hoyos wrote: > 2016-08-04 1:35 GMT+02:00 James Almer : >> On 8/3/2016 7:59 PM, Carl Eugen Hoyos wrote: >>> Hi! >>> >>> 2016-08-04 0:33 GMT+02:00 James Almer : This removes unnecessary header dependencies. >>> >>> Why is

[FFmpeg-devel] [PATCH v2 10/11] avformat/fifo: Add AVFMT_FLAG_NONBLOCK support

2016-08-04 Thread sebechlebskyjan
From: Jan Sebechlebsky Add support for nonblocking calls. Signed-off-by: Jan Sebechlebsky --- Changes from the last version: - boolean flags accessed from both threads are ints now and are accessed with atomic operations. -

[FFmpeg-devel] [PATCH v2 06/11] avformat: add avformat_write_abort() function

2016-08-04 Thread sebechlebskyjan
From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- Changes from last version of patch: - removed AVFMT_FLAG_NONBLOCK check and modified comment so it states how function behaves with both blocking / non-blocking muxer

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

2016-08-04 Thread Matthias Hunstock
Am 10.07.2016 um 13:02 schrieb 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

Re: [FFmpeg-devel] Log level of message "Increasing reorder buffer to ..."

2016-08-04 Thread Robert Krüger
On Thu, Aug 4, 2016 at 1:55 PM, Carl Eugen Hoyos wrote: > 2016-08-04 13:50 GMT+02:00 Robert Krüger : > > the log level of INFO does not seem appropriate for the message > (h264dec.c, > > line 531). If it is a condition that shouldn't really happen,

Re: [FFmpeg-devel] FFmpeg 3.1.2

2016-08-04 Thread Mark Thompson
On 04/08/16 11:25, Michael Niedermayer wrote: > ill soon make FFmpeg 3.1.2 > that is if someone wants something backported, do it now! We should fix in ffmpeg - Debian carrying the patch is not helpful. Nicolas, would you (or someone

Re: [FFmpeg-devel] [Libav-user] [PATCH]Fix a crash in vaapi_encode if "." is not the decimal separator

2016-08-04 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 08:56:46PM +0100, Mark Thompson wrote: > On 02/08/16 18:39, Nicolas George wrote: > > Le sextidi 16 thermidor, an CCXXIV, Carl Eugen Hoyos a écrit : > >> -{ "i_qfactor", "1.0" }, > >> -{ "i_qoffset", "0.0" }, > >> -{ "b_qfactor", "1.2" }, > >> -

[FFmpeg-devel] [PATCH v5 01/11] avformat: Add fifo pseudo-muxer

2016-08-04 Thread sebechlebskyjan
From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- Changes from the last version of patch: - boolean AVOptions are now ints, this was the cause of fate test segfault reported by Michael Changelog| 1 +

Re: [FFmpeg-devel] Log level of message "Increasing reorder buffer to ..."

2016-08-04 Thread Carl Eugen Hoyos
2016-08-04 13:50 GMT+02:00 Robert Krüger : > the log level of INFO does not seem appropriate for the message (h264dec.c, > line 531). If it is a condition that shouldn't really happen, it should be It does happen, see at least two related tickets iirc. > a warning,

[FFmpeg-devel] Log level of message "Increasing reorder buffer to ..."

2016-08-04 Thread Robert Krüger
Hi, the log level of INFO does not seem appropriate for the message (h264dec.c, line 531). If it is a condition that shouldn't really happen, it should be a warning, otherwise it should be debug (especially for library users this floods the logs as this seems to happen for many files. Would a

Re: [FFmpeg-devel] [PATCH 03/11] avformat/fifo: Add fate test

2016-08-04 Thread Jan Sebechlebsky
On 08/02/2016 10:08 PM, Michael Niedermayer wrote: On Tue, Aug 02, 2016 at 03:24:14PM +0200, sebechlebsky...@gmail.com wrote: segfaults on x86-32 Thanks for testing, I wrongly used uint8_t for boolean AVOptions instead of int. I'll send fixed patch soon. Regards, Jan

Re: [FFmpeg-devel] [PATCH v3 1/2] avcodec/bsf: Set EOF flag only in pkt == NULL

2016-08-04 Thread Jan Sebechlebsky
On 07/26/2016 12:40 PM, sebechlebsky...@gmail.com wrote: From: Jan Sebechlebsky Set BSF EOF flag only if pkt == NULL in av_bsf_send_packet(). Signed-off-by: Jan Sebechlebsky --- libavcodec/bsf.c | 2 +- 1 file changed, 1 insertion(+),

Re: [FFmpeg-devel] [PATCH 06/11] avformat: add av_abort_output() function

2016-08-04 Thread Jan Sebechlebsky
On 08/03/2016 03:15 PM, Nicolas George wrote: Le sextidi 16 thermidor, an CCXXIV, sebechlebsky...@gmail.com a écrit : From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- libavformat/avformat.h | 14 ++

[FFmpeg-devel] FFmpeg 3.1.2

2016-08-04 Thread Michael Niedermayer
Hi all ill soon make FFmpeg 3.1.2 that is if someone wants something backported, do it now! thx -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The bravest are surely those who have the clearest vision of what is before them, glory and danger alike, and yet

Re: [FFmpeg-devel] [PATCH] avcodec/alsdec: implement floating point decoding

2016-08-04 Thread Thilo Borgmann
> [...] > I had sent the old patch file by mistake. This is the correct one. LGTM. Pushing in a few days if there are no further comments. -Thilo ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: clean up vdpau_internal.h

2016-08-04 Thread Timothy Gu
On Thu, Aug 4, 2016 at 12:57 AM Carl Eugen Hoyos wrote: > 2016-08-03 6:30 GMT+02:00 James Almer : > > Also don't include it on files that don't need it. > > > > This reduces differences with libav > > > > Signed-off-by: James Almer > >

Re: [FFmpeg-devel] [PATCH] avcodec/vdpau: clean up vdpau_internal.h

2016-08-04 Thread Carl Eugen Hoyos
2016-08-03 6:30 GMT+02:00 James Almer : > Also don't include it on files that don't need it. > > This reduces differences with libav > > Signed-off-by: James Almer > --- > Untested with VDPAU enabled. The patch is probably ok if it does not brake compilation

Re: [FFmpeg-devel] why prebuilt windows ffmpeg use rtmpdump instead of its native rtmp plugin

2016-08-04 Thread Carl Eugen Hoyos
2016-08-04 8:53 GMT+02:00 qw : > But why does official prebuilt windows ffmpeg use rtmpdump instead of its > native rtmp plugin? I don't think there are "official" windows binaries but feel free to ask Zeranoe why he does things the way he does them, there is a forum. Carl

Re: [FFmpeg-devel] [PATCH] avcodec: don't include vdpau_compat.h when vdpau is not enabled

2016-08-04 Thread Carl Eugen Hoyos
2016-08-04 1:35 GMT+02:00 James Almer : > On 8/3/2016 7:59 PM, Carl Eugen Hoyos wrote: >> Hi! >> >> 2016-08-04 0:33 GMT+02:00 James Almer : >>> This removes unnecessary header dependencies. >> >> Why is this an advantage? > > h263, vc1 and mpeg decoder were

[FFmpeg-devel] why prebuilt windows ffmpeg use rtmpdump instead of its native rtmp plugin

2016-08-04 Thread qw
Hi, Previously, I sent email to ask which rtmp plugin is better. The answer is ffmpeg's native rtmp plugin is better than rtmpdump. But why does official prebuilt windows ffmpeg use rtmpdump instead of its native rtmp plugin? Thanks! Regards andrew