Re: [FFmpeg-devel] [PATCH 1/2] lavf/http.c: Free allocated client URLContext in case of error.

2017-05-19 Thread Stephan Holljes
On Sat, Apr 22, 2017 at 3:18 AM, Stephan Holljes wrote: > On Tue, Apr 11, 2017 at 8:05 PM, Stephan Holljes > wrote: >> --- >> >> This version returns 0 in case of success and does not fallthrough to fail:. >> >> libavformat/http.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/l

Re: [FFmpeg-devel] [PATCH] lavf/mov: make invalid mdhd time_scale default to 1 instead of erroring out

2017-05-19 Thread Michael Niedermayer
On Wed, May 17, 2017 at 02:58:12PM +0200, Matthieu Bouron wrote: > On Wed, May 17, 2017 at 01:56:13PM +0200, Matthieu Bouron wrote: > > On Fri, May 12, 2017 at 11:12:12PM +0200, Michael Niedermayer wrote: > > > On Thu, May 11, 2017 at 04:33:50PM +0200, Matthieu Bouron wrote: > > > > Some samples ha

Re: [FFmpeg-devel] [PATCH] avio_read data loss with multiple calls to fill_buffer

2017-05-19 Thread Rob Meyers
I can confirm the current source tree has resolved our issue. On Fri, May 19, 2017 at 12:16 PM Michael Niedermayer wrote: > On Fri, May 19, 2017 at 08:28:23PM +0200, Michael Niedermayer wrote: > > hi > > > > On Thu, May 18, 2017 at 10:58:39PM +, Rob Meyers wrote: > > > avio_read does multipl

Re: [FFmpeg-devel] [PATCH] avfilter: take_samples: do not directly return frame when samples are skipped

2017-05-19 Thread Michael Niedermayer
On Thu, May 18, 2017 at 08:37:21PM +0700, Muhammad Faiz wrote: > Should fix Ticket6349. > Modifying data pointer may make it unaligned. > > Also change frame->nb_samples < max to frame->nb_samples <= max. > This improves performance. Benchmark: > ./ffmpeg -filter_complex "aevalsrc=0:n=1166,firequa

Re: [FFmpeg-devel] [PATCH] Disable MSA optimization for big endian arch

2017-05-19 Thread Michael Niedermayer
On Thu, May 18, 2017 at 12:36:51PM +, Shivraj Patil wrote: > Shivraj: yes, -mmsa flag will be added and should not be an issue for big > endian mips builds. > > > +if enabled bigendian && enabled msa; then > > +disable msa > > +fi > > As currently, MSA optimizations does not

Re: [FFmpeg-devel] [PATCH] avio_read data loss with multiple calls to fill_buffer

2017-05-19 Thread Michael Niedermayer
On Fri, May 19, 2017 at 08:28:23PM +0200, Michael Niedermayer wrote: > hi > > On Thu, May 18, 2017 at 10:58:39PM +, Rob Meyers wrote: > > avio_read does multiple calls to fill_buffer(). Adjusting things in > > fill_buffer was frowned upon; how would you suggest I approach this? > > Its not fr

Re: [FFmpeg-devel] [PATCH] avio_read data loss with multiple calls to fill_buffer

2017-05-19 Thread Michael Niedermayer
hi On Thu, May 18, 2017 at 10:58:39PM +, Rob Meyers wrote: > avio_read does multiple calls to fill_buffer(). Adjusting things in > fill_buffer was frowned upon; how would you suggest I approach this? Its not frowned upon to fix a bug in fill_buffer(), its frowned upon to make a random change

[FFmpeg-devel] [PATCH] Added the RTP Header Extension to AVPacket side data

2017-05-19 Thread Anna Sokol
Added a way to store the RFC 3550 Section 5.3.1 RTP Header Extension in side data of AVPacket. - In libavcodec/avcodec.h AVPacketSideDataType added AV_PKT_DATA_RTP_HEADER_EXTENSION. - In libavcodec/avpacket.c av_packet_side_data_name added a case for the AV_PKT_DATA_RTP_HEADER_EXTENSION. - In libav

Re: [FFmpeg-devel] [PATCH] avfilter: Enable in MP4 container both AMR-NB and AMR-WB

2017-05-19 Thread Bob Kirnum
I tried adding support for MOV to see if I could use unpatched FFmpeg libraries. Although both AMR-NB and AMR-WB do appear to record to MOV, neither result plays properly using QuickTime or Windows 'Movies & TV' app. The MOV (H.264 / AMR-NB) sometimes audio plays, sometimes video. Appears that th

Re: [FFmpeg-devel] [PATCH] avfilter/graphparser: allow specifying filter@id as filter instance

2017-05-19 Thread Muhammad Faiz
On Thu, May 18, 2017 at 11:48 PM, Paul B Mahol wrote: > On 5/18/17, Muhammad Faiz wrote: >> On Thu, May 18, 2017 at 11:11 PM, Paul B Mahol wrote: >>> On 5/17/17, Muhammad Faiz wrote: See http://lists.ffmpeg.org/pipermail/ffmpeg-user/2017-April/035975.html Parsed_filter_X is not intuit

Re: [FFmpeg-devel] [PATCH V3] lavc/vaapi_encode_h264: Enable MB rate control.

2017-05-19 Thread Mark Thompson
On 19/05/17 00:43, Jun Zhao wrote: > > On 2017/5/14 12:26, Jun Zhao wrote: >> V3: - Fix build error with old VAAPI version. >> V2: - Refine the name/value type to mb_rate_control/bool. >> - Only supported GEN9+ (SKL/APL/KBL/...) >> - i965 driver default use frame-level rate control algorit