Re: [FFmpeg-devel] [PATCH 1/1] Add Sega FILM muxer

2018-04-06 Thread Misty De Meo
On Thu, Apr 5, 2018 at 2:06 PM, Josh de Kock wrote: > Thanks, pushed. I also clarified with wm4 on IRC that while he was against > it he wasn't blocking the muxer if someone else pushes it. Thank you! ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.or

Re: [FFmpeg-devel] lavfi/silencedetect v3

2018-04-06 Thread James Almer
On 4/1/2018 8:26 AM, Paul B Mahol wrote: > On 3/5/18, Gaullier Nicolas wrote: >> Hello, >> I have not received any comment yet on my patchset v3 ("add mono mode" new >> feature + 4 fixes including a ticket), does it look good to you, could it be >> committed ? >> >> http://ffmpeg.org/pipermail/ffm

Re: [FFmpeg-devel] [PATCH 1/3] avformat/matroskadec: use refcounted buffers in EbmlBin

2018-04-06 Thread James Almer
On 4/4/2018 8:43 PM, James Almer wrote: > Data in EbmlBin objects is never changed after being read from the > input file (save for two specific cases with encoded CodePrivate), so > using AVBufferRef we can prevent unnecessary copy of data by instead > creating new references to said constant data

Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/matroskadec: reference the existing data buffer when creating packets

2018-04-06 Thread James Almer
On 4/6/2018 8:30 PM, Michael Niedermayer wrote: > On Thu, Apr 05, 2018 at 12:32:47PM -0300, James Almer wrote: >> Newly allocated data buffers (wavpack, prores, compressed buffers) >> are padded to meet the requirements of AVPacket. >> > >> About 10x speed up in matroska_parse_frame(). > > thats

Re: [FFmpeg-devel] [PATCH 2/3 v2] avformat/matroskadec: factor the prores packet parsing code out

2018-04-06 Thread James Almer
On 4/6/2018 8:29 PM, Michael Niedermayer wrote: > On Thu, Apr 05, 2018 at 12:30:59PM -0300, James Almer wrote: >> Simplifies code in matroska_parse_frame(). This is in preparation for >> the following patch. >> >> Signed-off-by: James Almer >> --- >> Not overloading dst this time... >> >> libavfo

Re: [FFmpeg-devel] Patch for seg fault in swr_convert_internal() -> sum2_float during dithering

2018-04-06 Thread Michael Niedermayer
On Thu, Apr 05, 2018 at 02:38:03PM +0200, Hendrik Schreiber wrote: > Hey there, > > I have recently switched to using FFmpeg for conversions of 24bit stereo WAV > to 16bit stereo WAV (with dithering). > > For some very large files, I occasionally encountered a segmentation fault in > _sum2_floa

Re: [FFmpeg-devel] [PATCH 1/3] avcodec: add siren audio decoder

2018-04-06 Thread Compn
On Wed, 4 Apr 2018 16:09:35 +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > libavcodec/Makefile | 1 + > libavcodec/allcodecs.c | 1 + > libavcodec/avcodec.h| 1 + > libavcodec/codec_desc.c | 8 + > libavcodec/siren.c | 847 >

Re: [FFmpeg-devel] [PATCH 3/3 v2] avformat/matroskadec: reference the existing data buffer when creating packets

2018-04-06 Thread Michael Niedermayer
On Thu, Apr 05, 2018 at 12:32:47PM -0300, James Almer wrote: > Newly allocated data buffers (wavpack, prores, compressed buffers) > are padded to meet the requirements of AVPacket. > > About 10x speed up in matroska_parse_frame(). thats a nice speedup patch seems to work fine thx [...] -- Mic

Re: [FFmpeg-devel] [PATCH 2/3 v2] avformat/matroskadec: factor the prores packet parsing code out

2018-04-06 Thread Michael Niedermayer
On Thu, Apr 05, 2018 at 12:30:59PM -0300, James Almer wrote: > Simplifies code in matroska_parse_frame(). This is in preparation for > the following patch. > > Signed-off-by: James Almer > --- > Not overloading dst this time... > > libavformat/matroskadec.c | 50 > +

Re: [FFmpeg-devel] [PATCH 3/3] avformat/vivo: add support for siren codec

2018-04-06 Thread Compn
On Wed, 4 Apr 2018 16:09:37 +0200, Paul B Mahol wrote: > +} else { > +ast->codecpar->codec_id = AV_CODEC_ID_SIREN; > +ast->codecpar->bits_per_coded_sample = 16; > +ast->codecpar->block_align = 40; > +ast->codecpar->bit_rate = 6400; wow great! now there is o

Re: [FFmpeg-devel] [PATCH] doc/developer: remove merge request method of contributing

2018-04-06 Thread Lou Logan
On Thu, Apr 5, 2018, at 10:17 AM, Lou Logan wrote: > This seems to confuse Github users into thinking that we may accept pull > requests. We do not accept pull requests. > > Sending patches to the ffmpeg-devel mailing list is our preferred method > for users to contribute code. > > Signed-off-by:

Re: [FFmpeg-devel] [PATCH] avformat/rtpenc_chain: use the proper function to free AVFormatContext

2018-04-06 Thread Rostislav Pehlivanov
On 6 April 2018 at 17:59, Vittorio Giovara wrote: > Yes the order of operations is a problem in a generic matrix, but for a > display matrix the order is more or less consolidated in a defacto > standard: > check for flip first, then rotation. We have the same pattern in h264 and > hevc decoder f

[FFmpeg-devel] [PATCH] avformat/rtpenc_chain: use the proper function to free AVFormatContext

2018-04-06 Thread Vittorio Giovara
Yes the order of operations is a problem in a generic matrix, but for a display matrix the order is more or less consolidated in a defacto standard: check for flip first, then rotation. We have the same pattern in h264 and hevc decoder for the rotation side data. You are right that the description

Re: [FFmpeg-devel] [PATCH] lavc/amfenc: Retain a reference to D3D frames used as input during the encoding process

2018-04-06 Thread Alexander Kravchenko
> >> This breaks the testcase described in > >> https://trac.ffmpeg.org/ticket/6990 which is basically the same as the > >> one you described in this patch. > >> > >> I get the following spammed repeatedly: > >> > >> [AVHWFramesContext @ 0502d340] Static surface pool size exceeded. > >> [m

Re: [FFmpeg-devel] [PATCH] avutil/buffer: add a dynamic size buffer pool API

2018-04-06 Thread James Almer
On 3/28/2018 10:42 AM, James Almer wrote: > On 3/28/2018 6:59 AM, wm4 wrote: >> On Tue, 27 Mar 2018 23:11:27 -0300 >> James Almer wrote: >> >>> diff --git a/libavutil/buffer.h b/libavutil/buffer.h >>> index 73b6bd0b14..d06b301fe5 100644 >>> --- a/libavutil/buffer.h >>> +++ b/libavutil/buffer.h >>>

Re: [FFmpeg-devel] [PATCH] avformat/rawenc: check stream type

2018-04-06 Thread Gyan Doshi
On 4/5/2018 12:09 AM, Michael Niedermayer wrote: This does not work breaks fate-unknown_layout-ac3 also the tests mayb too strict, there are similar codec_ids like mpeg1/2 or jpeg variants which are all basically the same from a muxers point of view Revised patch passes FATE. Set looser cond

[FFmpeg-devel] FFmpeg filter for real time captioning.

2018-04-06 Thread ANURAG SINGH IIT BHU
How to write a ffmpeg-libavfilter filter which outputs a "Hello World minute:sec" subtitle each second.The filter should produce packets/frames of subtitles in a AVMEDIA_TYPE_SUBTITLE stream output from the avfilter.The output from the filter should be passed through the filter chain back into the

Re: [FFmpeg-devel] [PATCH] lavc/amfenc: Retain a reference to D3D frames used as input during the encoding process

2018-04-06 Thread James Almer
On 4/6/2018 7:25 AM, Alexander Kravchenko wrote: >> >> This breaks the testcase described in >> https://trac.ffmpeg.org/ticket/6990 which is basically the same as the >> one you described in this patch. >> >> I get the following spammed repeatedly: >> >> [AVHWFramesContext @ 0502d340] Stati

Re: [FFmpeg-devel] [PATCH] doc/developer: remove merge request method of contributing

2018-04-06 Thread Rostislav Pehlivanov
On 6 April 2018 at 12:22, Josh de Kock wrote: > On 2018/04/05 19:17, Lou Logan wrote: > >> This seems to confuse Github users into thinking that we may accept pull >> requests. We do not accept pull requests. >> >> Sending patches to the ffmpeg-devel mailing list is our preferred method >> for us

Re: [FFmpeg-devel] [PATCH] doc/developer: remove merge request method of contributing

2018-04-06 Thread Josh de Kock
On 2018/04/05 19:17, Lou Logan wrote: This seems to confuse Github users into thinking that we may accept pull requests. We do not accept pull requests. Sending patches to the ffmpeg-devel mailing list is our preferred method for users to contribute code. Signed-off-by: Lou Logan --- doc/dev

[FFmpeg-devel] [PATCH] avformat/hlsenc: fix handling of delete_segments when %v is present

2018-04-06 Thread Bodecs Bela
Dear All, when var_stream_map option is used, %v must appear either in segment name template or in the directory path. This latter case currently is not handled and using delete_segments flag of hls_flags is broken now. This patch fixes this issue. The root cause of the bug was that HLSSegment

Re: [FFmpeg-devel] [PATCH] lavc/amfenc: Retain a reference to D3D frames used as input during the encoding process

2018-04-06 Thread Alexander Kravchenko
> > This breaks the testcase described in > https://trac.ffmpeg.org/ticket/6990 which is basically the same as the > one you described in this patch. > > I get the following spammed repeatedly: > > [AVHWFramesContext @ 0502d340] Static surface pool size exceeded. > [mpeg2video @

Re: [FFmpeg-devel] [PATCH] avcodec/openh264enc.c: generate IDR frame in response to I frame pict_type

2018-04-06 Thread Valery Kot
On Thu, Apr 5, 2018 at 9:02 PM, James Almer wrote: > On 4/5/2018 3:25 PM, Lou Logan wrote: >> On Sun, Mar 25, 2018, at 10:51 PM, Valery Kot wrote: >>> >>> Just wondering: is there an active maintainer for avcodec/libopenh264? >> >> Doesn't appear to be so. >> >>> If yes - can he or she please revi

Re: [FFmpeg-devel] [PATCH] [RFC]doc/examples: alternative input handler

2018-04-06 Thread Bodecs Bela
2018.04.06. 1:34 keltezéssel, wm4 írta: On Fri, 30 Mar 2018 14:47:25 +0200 Bodecs Bela wrote: Hi All, regularly, on different forums and mailing lists a requirement popups for a feature to automatically failover switching between main input and a secondary input in case of main input unavai