Re: [FFmpeg-devel] [PATCH] avfilter/af_loudnorm: correctly initialize PTS

2018-02-04 Thread Muhammad Faiz
On Sun, Feb 4, 2018 at 3:32 PM, Paul B Mahol wrote: > On 2/4/18, Muhammad Faiz wrote: >> On Sat, Feb 3, 2018 at 9:22 PM, Niklas Haas wrote: >>> From: Niklas Haas >>> >>> Right now, the PTS always starts out as 0, which

Re: [FFmpeg-devel] [PATCH] avcodec/me_cmp: remove ff_me_cmp_init_static()

2018-02-04 Thread Hendrik Leppkes
On Sun, Feb 4, 2018 at 2:47 PM, Michael Niedermayer wrote: > On Sun, Feb 04, 2018 at 05:51:13AM +0700, Muhammad Faiz wrote: >> Precalculate and constify ff_square_tab. >> >> Signed-off-by: Muhammad Faiz >> --- >> libavcodec/me_cmp.c | 51 >>

Re: [FFmpeg-devel] [PATCH] gitignore: add OpenCL generated kernels

2018-02-04 Thread Hendrik Leppkes
On Sun, Feb 4, 2018 at 3:55 PM, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > .gitignore | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/.gitignore b/.gitignore > index 0e57cb0b4c..c4135b252d 100644 > ---

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1: Support for RGBA64 and GBRAP16

2018-02-04 Thread Michael Niedermayer
On Thu, Feb 01, 2018 at 01:43:00PM +0100, Jerome Martinez wrote: > Add support for 16-bit/component RGB with Alpha encoding and decoding in > FFV1, both RGBA64 and GBRAP16 for encoding, GBRAP16 for decoding. > > Resulting bitstream was tested about lossless encoding/decoding by the > compression

Re: [FFmpeg-devel] [PATCH v4 2/7] lavf/rtp: replace linked list with array

2018-02-04 Thread Muhammad Faiz
On Sat, Feb 3, 2018 at 2:44 AM, Josh de Kock wrote: > --- > libavformat/allformats.c | 4 -- > libavformat/rdt.c| 9 +--- > libavformat/rdt.h| 5 -- > libavformat/rtpdec.c | 138 > ++- > libavformat/rtpdec.h

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (4th attempt)

2018-02-04 Thread Dr. Alan Barclay
On 17/01/2018 11:15, Carl Eugen Hoyos wrote: 2018-01-17 11:56 GMT+01:00 Dr. Alan Barclay : Attached in a further patch - adding the error checks. Please merge this patch into your previous patch. Both patches updated. Alan. And please avoid top-posting here, Carl

Re: [FFmpeg-devel] [PATCH V3] example/vaapi_transcode: Add a VA-API transcode example.

2018-02-04 Thread Mark Thompson
On 29/01/18 02:59, Jun Zhao wrote: > V3: - Add error check when av_buffer_ref fail > - Fix exec encode/write but output file hasn't been open lead to crash > issue > - Add build check > V2: - deduce output format from file extension, because VP8/VP9 encoder > need to a muxer (e,g ivf

Re: [FFmpeg-devel] [PATCH] avcodec/me_cmp: remove ff_me_cmp_init_static()

2018-02-04 Thread Muhammad Faiz
On Sun, Feb 4, 2018 at 9:26 AM, James Almer wrote: > On 2/3/2018 7:51 PM, Muhammad Faiz wrote: >> Precalculate and constify ff_square_tab. >> >> Signed-off-by: Muhammad Faiz >> --- >> libavcodec/me_cmp.c | 51 >>

Re: [FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

2018-02-04 Thread Nicolas George
Muhammad Faiz (2018-02-04): > What about av*iterate(int index)? I like the idea of an index better than the other options evoked, especially the linked-list-like APIs. It is also more similar to the APIs for enumerated types. Another option would be to return the whole list in a mallocated

Re: [FFmpeg-devel] [PATCH] avcodec/me_cmp: remove ff_me_cmp_init_static()

2018-02-04 Thread Michael Niedermayer
On Sun, Feb 04, 2018 at 05:51:13AM +0700, Muhammad Faiz wrote: > Precalculate and constify ff_square_tab. > > Signed-off-by: Muhammad Faiz > --- > libavcodec/me_cmp.c | 51 > +--- > libavcodec/me_cmp.h | 4 +--- >

[FFmpeg-devel] [PATCH] gitignore: add OpenCL generated kernels

2018-02-04 Thread Rostislav Pehlivanov
Signed-off-by: Rostislav Pehlivanov --- .gitignore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.gitignore b/.gitignore index 0e57cb0b4c..c4135b252d 100644 --- a/.gitignore +++ b/.gitignore @@ -36,3 +36,5 @@ /lcov/ /src /mapfile +/libavfilter/opencl/unsharp.c

Re: [FFmpeg-devel] [PATCH] rtsp: rename certain options after a deprecation period

2018-02-04 Thread wm4
On Thu, 25 Jan 2018 19:00:43 +0100 wm4 wrote: > The names inherently clash with the meanings of the HTTP libavformat > protocol options. Rename them after a deprecation period to make them > compatible with the HTTP ones. > --- > I see no better way that wouldn't require

Re: [FFmpeg-devel] [PATCH] id3v2: fix unsynchronization

2018-02-04 Thread wm4
On Tue, 30 Jan 2018 13:43:25 +0100 wm4 wrote: > The ID3v2 "unsynchronization scheme" requires replacing any 0xFF 0x00 > sequences with 0xFF. This has to be done on every byte of the source > data, while the current code skipped a byte after a replacement. This > meant 0xFF

Re: [FFmpeg-devel] [PATCH 4/5] aptx: implement the aptX HD bluetooth codec

2018-02-04 Thread Aurelien Jacobs
On Sat, Jan 20, 2018 at 11:20:22PM +, Rostislav Pehlivanov wrote: > On 20 January 2018 at 17:26, Aurelien Jacobs wrote: > > > On Sun, Jan 14, 2018 at 10:54:34PM +0100, Carl Eugen Hoyos wrote: > > > 2018-01-14 14:06 GMT+01:00 Aurelien Jacobs : > > > > > > >

Re: [FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

2018-02-04 Thread wm4
On Sun, 4 Feb 2018 22:29:10 +0100 Nicolas George wrote: > Josh de Kock (2018-02-04): > > If we were to add in APIs which allowed you to register external components > > again, this idea wouldn't work well as indexes wouldn't necessarily > > correspond > > to the component which

Re: [FFmpeg-devel] [PATCH 4/5] aptx: implement the aptX HD bluetooth codec

2018-02-04 Thread Michael Niedermayer
On Sun, Feb 04, 2018 at 04:07:26PM +0100, Aurelien Jacobs wrote: > On Sat, Jan 20, 2018 at 11:20:22PM +, Rostislav Pehlivanov wrote: > > On 20 January 2018 at 17:26, Aurelien Jacobs wrote: > > > > > On Sun, Jan 14, 2018 at 10:54:34PM +0100, Carl Eugen Hoyos wrote: > > > >

Re: [FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

2018-02-04 Thread Josh de Kock
On Sun, Feb 04, 2018 at 02:46:09PM +0100, Nicolas George wrote: > Muhammad Faiz (2018-02-04): > > What about av*iterate(int index)? This makes no sense, it's then not an API for iteration of components. > > I like the idea of an index better than the other options evoked, > especially the

Re: [FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

2018-02-04 Thread Nicolas George
Josh de Kock (2018-02-04): > If we were to add in APIs which allowed you to register external components > again, this idea wouldn't work well as indexes wouldn't necessarily correspond > to the component which it previously did after you register extra components. That is not a problem if the

Re: [FFmpeg-devel] [PATCH v4 7/7] lavc/bsf: make BSF iteration the same as other iterators

2018-02-04 Thread Nicolas George
Josh de Kock (2018-02-04): > The main benefit of the opaque pointers is the flexibility of how components > are stored/represented internally, and being able to change/extend it with > no API changes (only additions). Sure there is the question of 'how efficient > is it?', but it's not really a

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/huffyuvdec: Check input buffer size

2018-02-04 Thread Michael Niedermayer
On Thu, Feb 01, 2018 at 02:36:16AM +0100, Michael Niedermayer wrote: > On Wed, Jan 31, 2018 at 07:56:06PM +0100, Paul B Mahol wrote: > > On 1/31/18, Michael Niedermayer wrote: > > > Fixes: Timeout > > > Fixes: 5487/clusterfuzz-testcase-4696837035393024 > > > > > >

Re: [FFmpeg-devel] [PATCH] avcodec/utvideodec: Fix bytes left check in decode_frame()

2018-02-04 Thread Michael Niedermayer
On Fri, Feb 02, 2018 at 11:18:28PM +0100, Michael Niedermayer wrote: > Fixes: out of array read > Fixes: poc-2017.avi > > Found-by: GwanYeong Kim > Signed-off-by: Michael Niedermayer > --- > libavcodec/utvideodec.c | 2 +- > 1 file changed, 1

Re: [FFmpeg-devel] [PATCH] Adding mkdir option for img2enc (4th attempt)

2018-02-04 Thread Michael Niedermayer
On Sun, Feb 04, 2018 at 12:51:59PM +, Dr. Alan Barclay wrote: > On 17/01/2018 11:15, Carl Eugen Hoyos wrote: > >2018-01-17 11:56 GMT+01:00 Dr. Alan Barclay : > > > >>Attached in a further patch - adding the error checks. > > > >Please merge this patch into your previous

[FFmpeg-devel] [PATCH]lavfi/avfiltergraph: Do not return ENOMEM if filter is missing

2018-02-04 Thread Carl Eugen Hoyos
Hi! OOM is unlikely as a failure for avfilter_graph_alloc_filter(), the patch avoids a surprising error if a filter was not found. Please comment, Carl Eugen From 6587726a5e96570bb54e49ccf0b7fd6d94b929c8 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 5 Feb 2018

Re: [FFmpeg-devel] fail to seek key video packet

2018-02-04 Thread qw
>> > I use following command to test ffmpeg-2.8.1: >> >> This list is for development of ffmpeg, not for user support or bug reports. >> >> Use the ffmpeg-user list for user support, and http://trac.ffmpeg.org/ >> to report bugs. > >And please don't expect anyone to help debug issues with

Re: [FFmpeg-devel] [PATCH]lavfi/avfiltergraph: Do not return ENOMEM if filter is missing

2018-02-04 Thread James Almer
On 2/4/2018 10:33 PM, Carl Eugen Hoyos wrote: > Hi! > > OOM is unlikely as a failure for avfilter_graph_alloc_filter(), the > patch avoids a surprising error if a filter was not found. > > Please comment, Carl Eugen > > > 0001-lavfi-avfiltergraph-Do-not-return-ENOMEM-if-filterch.patch > > >

Re: [FFmpeg-devel] [PATCH]lavfi/avfiltergraph: Do not return ENOMEM if filter is missing

2018-02-04 Thread James Almer
On 2/4/2018 11:05 PM, James Almer wrote: > On 2/4/2018 10:33 PM, Carl Eugen Hoyos wrote: >> Hi! >> >> OOM is unlikely as a failure for avfilter_graph_alloc_filter(), the >> patch avoids a surprising error if a filter was not found. >> >> Please comment, Carl Eugen >> >> >>

Re: [FFmpeg-devel] [PATCH] rtsp: rename certain options after a deprecation period

2018-02-04 Thread Carl Eugen Hoyos
2018-02-04 15:24 GMT+01:00 wm4 : > On Thu, 25 Jan 2018 19:00:43 +0100 > wm4 wrote: > >> The names inherently clash with the meanings of the HTTP libavformat >> protocol options. Rename them after a deprecation period to make them >> compatible with

[FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

2018-02-04 Thread Steven Liu
fix ticket: 7000 Signed-off-by: Steven Liu --- libavformat/isom.c | 1 + libavformat/mov.c | 1 + 2 files changed, 2 insertions(+) diff --git a/libavformat/isom.c b/libavformat/isom.c index 9d9f85885b..95dceaa50b 100644 --- a/libavformat/isom.c +++ b/libavformat/isom.c

Re: [FFmpeg-devel] [PATCH] avfilter/af_loudnorm: correctly initialize PTS

2018-02-04 Thread Paul B Mahol
On 2/4/18, Muhammad Faiz wrote: > On Sat, Feb 3, 2018 at 9:22 PM, Niklas Haas wrote: >> From: Niklas Haas >> >> Right now, the PTS always starts out as 0, which causes problems on a >> seek or when inserting this filter mid-stream. >> >>