[FFmpeg-devel] [PATCH] avfilter/vsrc_testsrc: add pal75bars and pal100bars video filter sources

2018-05-17 Thread Tobias Rapp
Generates color bar test patterns based on EBU PAL recommendations. Signed-off-by: Tobias Rapp --- Changelog| 1 + doc/filters.texi | 10 +++- libavfilter/Makefile | 2 + libavfilter/allfilters.c | 2 + libavfilter/version.h

Re: [FFmpeg-devel] compound literal in public header file

2018-05-17 Thread wm4
On Thu, 17 May 2018 00:41:10 +0200 Carl Eugen Hoyos wrote: > 2018-05-16 11:47 GMT+02:00, Steinar H. Gunderson > : > > On Wed, May 16, 2018 at 11:41:23AM +0200, Tobias Rapp wrote: > >> Yes, I am referring to usage of the libavutil headers in C. If the macro > >> is > >> only hidden for C++ and

Re: [FFmpeg-devel] [PATCH] avutil/error: hidden macro av_err2str for C++

2018-05-17 Thread wm4
On Thu, 17 May 2018 05:50:55 +0100 Rostislav Pehlivanov wrote: > On 17 May 2018 at 05:46, Zhao Zhili wrote: > > > --- > > libavutil/error.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/libavutil/error.h b/libavutil/error.h > > index 71df4da..8a35fef 100644 > > --- a/libavut

Re: [FFmpeg-devel] [PATCH] cmdutils - don't search for option 'default'

2018-05-17 Thread Michael Niedermayer
On Wed, May 16, 2018 at 09:21:49PM +0530, Gyan Doshi wrote: > > cmdutils.c |2 -- > 1 file changed, 2 deletions(-) > 495ccaae10f8873dc5070858d11c0c3093ce92b4 > 0001-cmdutils-don-t-search-for-option-default.patch > From 20a0544d66b5b3b9ebc3e03049a134d13706bda9 Mon Sep 17 00:00:00 2001 > Fro

[FFmpeg-devel] [PATCH] avutil/error: document av_err2str

2018-05-17 Thread Zhao Zhili
--- libavutil/error.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavutil/error.h b/libavutil/error.h index 71df4da..b357bfa 100644 --- a/libavutil/error.h +++ b/libavutil/error.h @@ -115,6 +115,9 @@ static inline char *av_make_error_string(char *errbuf, size_t errbuf_size, int e /*

Re: [FFmpeg-devel] [PATCH] avutil/error: document av_err2str

2018-05-17 Thread Nicolas George
Zhao Zhili (2018-05-17): > --- > libavutil/error.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/libavutil/error.h b/libavutil/error.h > index 71df4da..b357bfa 100644 > --- a/libavutil/error.h > +++ b/libavutil/error.h > @@ -115,6 +115,9 @@ static inline char *av_make_error_string(c

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/qtrle: Do not output duplicated frames on insufficient input

2018-05-17 Thread Michael Niedermayer
On Wed, May 16, 2018 at 12:53:46AM +0200, Carl Eugen Hoyos wrote: > 2018-05-16 0:29 GMT+02:00, Hendrik Leppkes : > > > It makes no real difference if its less efficient or whatever - > > if a codec specification asks for this behavior, then our > > decoders should act accordingly. > > I wonder wh

Re: [FFmpeg-devel] [PATCH] avutil/error: hidden macro av_err2str for C++

2018-05-17 Thread Timo Rothenpieler
On 17.05.2018 06:46, Zhao Zhili wrote: > --- > libavutil/error.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavutil/error.h b/libavutil/error.h > index 71df4da..8a35fef 100644 > --- a/libavutil/error.h > +++ b/libavutil/error.h > @@ -116,8 +116,10 @@ static inline char *av_make_

Re: [FFmpeg-devel] [PATCH] avformat/hls: tag as AVFMT_TS_DISCONT

2018-05-17 Thread Timo Rothenpieler
> I think I'm against this. HLS streams do not typically contain > timestamp resets (even if they could). Otherwise you might as well add > this flag to the Matroska demuxer. Besides, it would break some of my > code, which uses this flag as a heuristic to detect mpeg-ts style > non-container forma

Re: [FFmpeg-devel] [PATCH] avutil/error: document av_err2str

2018-05-17 Thread Tobias Rapp
On 17.05.2018 11:24, Zhao Zhili wrote: --- libavutil/error.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavutil/error.h b/libavutil/error.h index 71df4da..b357bfa 100644 --- a/libavutil/error.h +++ b/libavutil/error.h @@ -115,6 +115,9 @@ static inline char *av_make_error_string(c

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/qtrle: Do not output duplicated frames on insufficient input

2018-05-17 Thread Hendrik Leppkes
On Thu, May 17, 2018 at 11:49 AM, Michael Niedermayer wrote: > On Wed, May 16, 2018 at 12:53:46AM +0200, Carl Eugen Hoyos wrote: >> 2018-05-16 0:29 GMT+02:00, Hendrik Leppkes : >> >> > It makes no real difference if its less efficient or whatever - >> > if a codec specification asks for this behav

Re: [FFmpeg-devel] [PATCH]lavc/v210dec: Skip Canopus C210 extradata

2018-05-17 Thread Carl Eugen Hoyos
2018-05-10 22:50 GMT+02:00, Carl Eugen Hoyos : > Peter Bubestinger provided a C210 file where every frame starts with > 64 bytes of extradata (24 byte "INFO", 16 byte "RDRT", rest "FIEL"). > Piotr confirmed that the Canopus decoder accepts files without the > extradata but consumes it if present.

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/qtrle: Do not output duplicated frames on insufficient input

2018-05-17 Thread Carl Eugen Hoyos
2018-05-17 12:22 GMT+02:00, Hendrik Leppkes : > On Thu, May 17, 2018 at 11:49 AM, Michael Niedermayer > wrote: >> On Wed, May 16, 2018 at 12:53:46AM +0200, Carl Eugen Hoyos wrote: >>> 2018-05-16 0:29 GMT+02:00, Hendrik Leppkes : >>> >>> > It makes no real difference if its less efficient or whatev

Re: [FFmpeg-devel] [PATCH 1/2] hevcdec: Miss the location of chroma samples when exporting stream parameters

2018-05-17 Thread Hendrik Leppkes
On Thu, May 17, 2018 at 8:08 AM, Xiang, Haihao wrote: > On Wed, 2018-05-16 at 11:27 +0200, Hendrik Leppkes wrote: >> On Wed, May 16, 2018 at 10:49 AM, Xiang, Haihao >> wrote: >> > On Wed, 2018-05-16 at 10:17 +0200, Hendrik Leppkes wrote: >> > > On Wed, May 16, 2018 at 9:19 AM, Haihao Xiang >> >

Re: [FFmpeg-devel] [PATCH] cmdutils - don't search for option 'default'

2018-05-17 Thread Gyan Doshi
On 5/17/2018 2:19 PM, Michael Niedermayer wrote: On Wed, May 16, 2018 at 09:21:49PM +0530, Gyan Doshi wrote: cmdutils.c |2 -- 1 file changed, 2 deletions(-) 495ccaae10f8873dc5070858d11c0c3093ce92b4 0001-cmdutils-don-t-search-for-option-default.patch From 20a0544d66b5b3b9ebc3e0304

[FFmpeg-devel] [PATCH 2/2] avcodec/mlpdec: Only change noise_type if the related fields are valid

2018-05-17 Thread Michael Niedermayer
Fixes: inconsistency Fixes:runtime error: index 8 out of bounds for type 'int32_t [8]' Fixes: 6686/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TRUEHD_fuzzer-5191383498358784 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: M

[FFmpeg-devel] [PATCH 1/2] indeo4: Decode all or nothing of a band header.

2018-05-17 Thread Michael Niedermayer
This avoids inconsistent value combinations. Alternatively it would be possible to add more checks and careful use of temporary variables, but my try of this quickly seemed to become a rather large change. The disadvantage of this, is that the struct is copied back and forth. Fixes: index 6 out of

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/qtrle: Do not output duplicated frames on insufficient input

2018-05-17 Thread Michael Niedermayer
On Thu, May 17, 2018 at 12:22:01PM +0200, Hendrik Leppkes wrote: > On Thu, May 17, 2018 at 11:49 AM, Michael Niedermayer > wrote: > > On Wed, May 16, 2018 at 12:53:46AM +0200, Carl Eugen Hoyos wrote: > >> 2018-05-16 0:29 GMT+02:00, Hendrik Leppkes : > >> > >> > It makes no real difference if its l

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: Check AVPacket->duration for being positive not just non zero

2018-05-17 Thread Derek Buitenhuis
On Thu, May 17, 2018 at 1:22 AM, Michael Niedermayer wrote: > Some demuxers (mov, microdvd at least) set duration negative. > > Signed-off-by: Michael Niedermayer > --- > fftools/ffmpeg.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) The patch itself seems reasonable, though I am

Re: [FFmpeg-devel] [PATCH] mov: Make sure PTS are both monotonically increasing, and unique

2018-05-17 Thread Derek Buitenhuis
On Tue, May 15, 2018 at 8:44 PM, Derek Buitenhuis wrote: > We already did this for audio, but it should be done for video too. > If we don't, seeking back to the start of the file, for example, can > become quite broken, since the first N packets will have repeating > and nonmonotonic PTS, yet the

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread Derek Buitenhuis
On Thu, May 17, 2018 at 1:15 AM, Michael Niedermayer wrote: > I have no magic bullet sadly, which is why i didnt reply. Even lacking a solution, a public agreement that change is needed is a start, isn't it? > But if there is a consensus to enforce some CoC. Then it will be enforced. > I think t

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread Clément Bœsch
On Mon, May 14, 2018 at 05:50:25PM +0100, Derek Buitenhuis wrote: [...] > 1. Implement a formal CoC enforcement system. This has been mostly > copypasted >from VideoLAN's, and is meant as more of a blueprint. This will no > doubt >be controversial. So as mentioned already in

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread Rostislav Pehlivanov
On 17 May 2018 at 15:22, Clément Bœsch wrote: > On Mon, May 14, 2018 at 05:50:25PM +0100, Derek Buitenhuis wrote: > [...] > > 1. Implement a formal CoC enforcement system. This has been mostly > copypasted > >from VideoLAN's, and is meant as more of a blueprint. This will > no doubt >

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread Dave Rice
> On May 17, 2018, at 10:22 AM, Clément Bœsch wrote: > > On Mon, May 14, 2018 at 05:50:25PM +0100, Derek Buitenhuis wrote: > [...] >>1. Implement a formal CoC enforcement system. This has been mostly >> copypasted >> from VideoLAN's, and is meant as more of a blueprint. This will no

Re: [FFmpeg-devel] [PATCH] add support for parsing pssh box

2018-05-17 Thread Jacob Trimble
On Wed, May 16, 2018 at 11:32 PM, Zewu Chen wrote: > Use structure AVEncryptionInitInfo to provide CENC initialization information > for the application. > > Signed-off-by: Chen Zewu > --- > libavformat/isom.h | 1 + > libavformat/mov.c | 64 > +

[FFmpeg-devel] [PATCH] avfilter: add declick and declip audio filters

2018-05-17 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 95 +++ libavfilter/Makefile | 2 + libavfilter/af_declick.c | 699 +++ libavfilter/allfilters.c | 2 + 4 files changed, 798 insertions(+) create mode 100644 libavfilter/af_declick

Re: [FFmpeg-devel] [RFC][ALT PATCHES] Code of Conduct Enforcement

2018-05-17 Thread grady player
There are lots of developers that consume FFMPEG and have a vested interest in its development continuing smoothly... I count myself among them. I dont think it would be hard to impanel a group (3-4) of reasonable and dispassionate people to officiate and moderate public communication. As an obs

Re: [FFmpeg-devel] [PATCH] avfilter: add declick and declip audio filters

2018-05-17 Thread Moritz Barsnick
On Thu, May 17, 2018 at 19:19:06 +0200, Paul B Mahol wrote: Valuable filters. I had compiled your previous separate patches, but didn't have any samples for testing. > +Setting this to very high value increases impulsive noise removal but makes > whole ^ a very high ...

[FFmpeg-devel] [RFC] Proposal for improved remap filter

2018-05-17 Thread Paul Holland
The general feature provided by the remap filter is important in a number of use cases. Lens distortion/correction is one that has been discussed previously. But it is also a key feature in processing video material intended for Virtual Reality and 360 degree playback. These workflows employ non

[FFmpeg-devel] [FFmpeg-devl][RFC] Proposal for improved remap filter

2018-05-17 Thread Paul Holland
The general feature provided by the remap filter is important in a number of use cases. Lens distortion/correction is one that has been discussed previously. But it is also a key feature in processing video material intended for Virtual Reality and 360 degree playback. These workflows employ non

Re: [FFmpeg-devel] [RFC] Proposal for improved remap filter

2018-05-17 Thread Paul B Mahol
On 5/17/18, Paul Holland wrote: > The general feature provided by the remap filter is important in a number > of use cases. Lens distortion/correction is one that has been discussed > previously. But it is also a key feature in processing video material > intended for Virtual Reality and 360 deg

Re: [FFmpeg-devel] [RFC] Proposal for improved remap filter

2018-05-17 Thread Paul Holland
I am sure there are multiple ways to achieve this. My current idea is to teach/enhance swscale to perform the basic logic for it. In that sort of design it would be able to reuse most of the existing, highly optimized filtering functions - as well as all of its to, and from, planar format convers

[FFmpeg-devel] [PATCH v2] avcodec/videotoolbox: fix decoding of some hevc videos

2018-05-17 Thread Aman Gupta
From: Aman Gupta See https://s3.amazonaws.com/tmm1/videotoolbox/germany-hevc-zdf.ts This moves the hw_frames_ctx reference from the AVFrame to a new VTHWFrame which now holds both the CVPixelBufferRef and the AVBuffer. Previously we would set the hw_frames_context on the AVFrame directly, but o

[FFmpeg-devel] [PATCH v3 3/3] avformat/mpegts: add merge_pmt_versions option

2018-05-17 Thread Aman Gupta
From: Aman Gupta This new optional flag makes it easier to deal with mpegts samples where the PMT is updated and elementary streams move to different PIDs in the middle of playback. Previously, new AVStreams were created per PID, and it was up to the user to figure out which streams had migrated

[FFmpeg-devel] [PATCH v3 1/3] avformat: update AVProgram to keep track of version and changes

2018-05-17 Thread Aman Gupta
From: Aman Gupta Signed-off-by: Aman Gupta --- doc/APIchanges | 5 + libavformat/avformat.h | 3 ++- libavformat/utils.c| 1 + libavformat/version.h | 2 +- 4 files changed, 9 insertions(+), 2 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index befa58c84a..bca2e748

[FFmpeg-devel] [PATCH v3 2/3] avformat/mpegts: keep track of AVProgram.pmt_version and set AV_PROGRAM_CHANGED on version updates

2018-05-17 Thread Aman Gupta
From: Aman Gupta This can be used to detect changes to the streams in an AVProgram Signed-off-by: Aman Gupta --- libavformat/mpegts.c | 20 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index fc9bb3940e..e83238d7

[FFmpeg-devel] [PATCH] avcodec/ac3dec: Use frame_size if superframe_size is 0

2018-05-17 Thread Michael Niedermayer
Fixes: Infinite loop Fixes: 7669/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_AC3_FIXED_fuzzer-4689042185650176 Fixes: 7670/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_AC3_fuzzer-4706306762997760 Fixes: 7672/clusterfuzz-testcase-ffmpeg_AV_CODEC_ID_EAC3_fuzzer-4702108499574784 Found-by: continuous fuzzing

[FFmpeg-devel] [PATCH] avformat/mpegts: add skip_unknown_pmt option

2018-05-17 Thread Aman Gupta
From: Aman Gupta Some filtered mpegts streams may erroneously include PMTs for programs that are not advertised in the PAT. This confuses ffmpeg and most players because multiple audio/video streams are created and it is unclear which ones actually contain data. See for example https://tmm1.s3.a

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: add skip_unknown_pmt option

2018-05-17 Thread Aman Gupta
On Thu, May 17, 2018 at 5:04 PM, Aman Gupta wrote: > From: Aman Gupta > > Some filtered mpegts streams may erroneously include PMTs for programs > that are not advertised in the PAT. This confuses ffmpeg and most > players because multiple audio/video streams are created and it is > unclear whic

[FFmpeg-devel] [PATCH v3] avformat: keep track of PMT details in AVStream

2018-05-17 Thread Aman Gupta
From: Aman Gupta With these fields, the user has enough information to make the same decisions externally that the demuxer does internally when merge_pmt_versions=1. An API user would monitor the AVProgram they're interested in to see if (flags&AV_PROGRAM_CHANGED), and would then iterate over th

Re: [FFmpeg-devel] [PATCH] avfilter: add declick and declip audio filters

2018-05-17 Thread James Almer
On 5/17/2018 2:19 PM, Paul B Mahol wrote: > +static int config_input(AVFilterLink *inlink) > +{ > +AVFilterContext *ctx = inlink->dst; > +DeclickContext *s = ctx->priv; > +int i; > + > +s->pts = AV_NOPTS_VALUE; > +s->window_size = inlink->sample_rate * s->w / 1000.; > +if (s

Re: [FFmpeg-devel] [PATCH v3 2/3] avformat/mpegts: keep track of AVProgram.pmt_version and set AV_PROGRAM_CHANGED on version updates

2018-05-17 Thread Aman Gupta
On Thu, May 17, 2018 at 3:49 PM, Aman Gupta wrote: > From: Aman Gupta > > This can be used to detect changes to the streams in an AVProgram > Forgot to add: I have seen two related patches in the wild that attempt to solve this same problem in different ways. The first is in MythTV's ffmpeg fo

Re: [FFmpeg-devel] [PATCH] cmdutils - don't search for option 'default'

2018-05-17 Thread Michael Niedermayer
On Thu, May 17, 2018 at 04:40:44PM +0530, Gyan Doshi wrote: > > On 5/17/2018 2:19 PM, Michael Niedermayer wrote: > >On Wed, May 16, 2018 at 09:21:49PM +0530, Gyan Doshi wrote: > >> > > > >> cmdutils.c |2 -- > >> 1 file changed, 2 deletions(-) > >>495ccaae10f8873dc5070858d11c0c3093ce92b4 >

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/libzvbi-teletextdec: formatted ass output

2018-05-17 Thread Aman Gupta
On Mon, May 7, 2018 at 4:20 PM, Aman Gupta wrote: > > > On Mon, May 7, 2018 at 4:07 PM, Marton Balint wrote: > >> >> >> On Mon, 7 May 2018, Aman Gupta wrote: >> >> >>> >>> On Mon, May 7, 2018 at 12:50 PM, Aman Gupta wrote: >>> >>> >>> On Sun, May 6, 2018 at 2:05 PM, Marton Balint >>> wro

Re: [FFmpeg-devel] [PATCH v3] configure: error out on unsupported MSVC versions

2018-05-17 Thread Rostislav Pehlivanov
On 16 May 2018 at 20:52, Rostislav Pehlivanov wrote: > The FATE tests for MSVC versions older than 2013 are untested in FATE > and apparently are no longer supported. > > This commit makes the configure process error out in case an older version > is used, and suggests to use a supported version

Re: [FFmpeg-devel] [PATCH] doc/developer: update style guidelines to include for loops with declarations

2018-05-17 Thread Rostislav Pehlivanov
On 8 November 2017 at 21:26, Rostislav Pehlivanov wrote: > Signed-off-by: Rostislav Pehlivanov > --- > doc/developer.texi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index a7b4f1d737..de7d887451 100644 > --- a/doc/developer.texi > +++ b/

[FFmpeg-devel] [PATCH] avformat/hls: Properly expose intercepted ID3 tags to the API.

2018-05-17 Thread rshaffer
From: Richard Shaffer The HLS demuxer will process any ID3 tags at the beginning of a segment in order to obtain timestamp data. However, when this data was parsed on a second or subsequent segment, the updated metadata would be discarded. This change preserves the data and also sets the AVSTREAM