Re: [FFmpeg-devel] [PATCH v4 8/9] avcodec/cbs_h2645: vvc, do not skip nals for nuh_layer_id > 0

2021-02-05 Thread Andriy Gelman
On Sat, 06. Feb 11:30, Nuo Mi wrote: > On Mon, Jan 25, 2021 at 10:21 PM Nuo Mi wrote: > > > > > On Mon, Jan 25, 2021 at 10:17 PM Nuo Mi wrote: > > > >> --- > >> libavcodec/cbs_h2645.c | 3 ++- > >> 1 file changed, 2 insertions(+), 1 deletion(-) > >> > >> diff --git a/libavcodec/cbs_h2645.c

Re: [FFmpeg-devel] [PATCH] libavcodec/gifenc: Only write palette entries that actually used

2021-02-05 Thread Nuo Mi
On Fri, Feb 5, 2021 at 12:57 AM Derek Buitenhuis wrote: > GIF palette entries are not compressed, and writing 256 entries, > which can be up to every frame, uses a significant amount of > space, especially in extreme cases, where palettes can be very > small. > > Example, first six seconds of

Re: [FFmpeg-devel] [PATCH v4 7/9] avformat: add h266/vvc muxer

2021-02-05 Thread Nuo Mi
On Wed, Feb 3, 2021 at 9:57 PM James Almer wrote: > On 2/3/2021 10:45 AM, Moritz Barsnick wrote: > > On Mon, Jan 25, 2021 at 22:15:05 +0800, Nuo Mi wrote: > >> +.extensions= "hevc,h266,266", > > > > Is ".hevc" a valid extension for H.266/VVC? > > Most assuredly not, and just a copy

Re: [FFmpeg-devel] [PATCH v4 8/9] avcodec/cbs_h2645: vvc, do not skip nals for nuh_layer_id > 0

2021-02-05 Thread Nuo Mi
On Mon, Jan 25, 2021 at 10:21 PM Nuo Mi wrote: > > On Mon, Jan 25, 2021 at 10:17 PM Nuo Mi wrote: > >> --- >> libavcodec/cbs_h2645.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/cbs_h2645.c b/libavcodec/cbs_h2645.c >> index 6f3abf6b0b..da541407cf

Re: [FFmpeg-devel] [PATCH] cbs_h2645: Implement replace-PS with a table rather than many functions

2021-02-05 Thread Nuo Mi
On Wed, Feb 3, 2021 at 5:07 AM Mark Thompson wrote: > On 30/01/2021 11:01, Nuo Mi wrote: > > On Wed, Jan 27, 2021 at 9:52 PM Nuo Mi wrote: > >> On Wed, Jan 27, 2021 at 7:06 AM Mark Thompson wrote: > >>> > >>> + > >>> +err = ff_cbs_make_unit_refcounted(ctx, unit); > >>> +if (err < 0) >

Re: [FFmpeg-devel] [PATCH] libavcodec/gifenc: Only write palette entries that actually used

2021-02-05 Thread Jean First
Derek Buitenhuis wrote on 04.02.21 19:42: On 04/02/2021 17:26, Paul B Mahol wrote: How would that work? I'm not against if that does not break existing usage. Somethng like '-no_global_paltte 1' to not use a global palette, and write a palette each frame. Default would be off, so curent

[FFmpeg-devel] [PATCH] avcodec: add cri parser

2021-02-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/cri_parser.c | 105 libavcodec/parsers.c| 1 + 3 files changed, 107 insertions(+) create mode 100644 libavcodec/cri_parser.c diff --git a/libavcodec/Makefile

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec: add xbm parser

2021-02-05 Thread James Almer
ffmpeg | branch: master | Paul B Mahol https://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog>> | Wed Feb 3 11:58:47 2021 +0100| [7dfa98665c172a15f2dc6a9424331737360e5138] | committer: Paul B Mahol avcodec: add xbm parser

Re: [FFmpeg-devel] [PATCH] avcodec: add photocd parser

2021-02-05 Thread James Almer
On 2/5/2021 3:51 PM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/parsers.c| 1 + libavcodec/photocd_parser.c | 87 + 3 files changed, 89 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH 3/3] avformat/mov: simplify size code in probing a bit

2021-02-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 70f76caff5..8504e97831 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7127,10 +7127,7 @@ static int

[FFmpeg-devel] [PATCH 1/3] avformat/mov: factor size out of probe code

2021-02-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 8eacf2cc04..9406e42f49 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7108,9 +7108,11 @@ static int

[FFmpeg-devel] [PATCH 2/3] Support size = 1 and size = 0 special cases in probing

2021-02-05 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavformat/mov.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9406e42f49..70f76caff5 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7113,6 +7113,11 @@ static int mov_probe(const

[FFmpeg-devel] [PATCH] avcodec: add photocd parser

2021-02-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/parsers.c| 1 + libavcodec/photocd_parser.c | 87 + 3 files changed, 89 insertions(+) create mode 100644 libavcodec/photocd_parser.c diff --git a/libavcodec/Makefile

Re: [FFmpeg-devel] [PATCH v3] ffmpeg: add -fpsmax to clamp output framerate

2021-02-05 Thread Gyan Doshi
Pushed as d99cc1782563672bcdb46fb5ec51135847db8c99 On 05-02-2021 09:39 am, Gyan Doshi wrote: Will push in 12h if no further changes. On 03-02-2021 07:42 pm, Gyan Doshi wrote: Useful when encoding in batch or with aberrant inputs. ---   doc/ffmpeg.texi  |  7 +++   fftools/ffmpeg.c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_tile: remove extra whitespace in option description

2021-02-05 Thread Paul B Mahol
trivial, so will apply. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avfilter/avfilter: move enable_str expression parsing into avfilter_init_dict()

2021-02-05 Thread Paul B Mahol
Anybody have comments? This fixes crash. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 47/50 v2] doc/examples/vaapi_transcode: use av_packet_alloc() to allocate packets

2021-02-05 Thread James Almer
Signed-off-by: James Almer --- doc/examples/vaapi_transcode.c | 42 ++ 1 file changed, 22 insertions(+), 20 deletions(-) diff --git a/doc/examples/vaapi_transcode.c b/doc/examples/vaapi_transcode.c index 279d20f636..5a1a704a8e 100644 ---

Re: [FFmpeg-devel] [PATCH 47/50] doc/examples/vaapi_transcode: use av_packet_alloc() to allocate packets

2021-02-05 Thread James Almer
On 2/5/2021 1:49 PM, Michael Niedermayer wrote: On Thu, Feb 04, 2021 at 04:10:02PM -0300, James Almer wrote: Signed-off-by: James Almer --- doc/examples/vaapi_transcode.c | 45 -- 1 file changed, 26 insertions(+), 19 deletions(-) maybe i forgot something

Re: [FFmpeg-devel] [PATCH 47/50] doc/examples/vaapi_transcode: use av_packet_alloc() to allocate packets

2021-02-05 Thread Michael Niedermayer
On Thu, Feb 04, 2021 at 04:10:02PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > doc/examples/vaapi_transcode.c | 45 -- > 1 file changed, 26 insertions(+), 19 deletions(-) maybe i forgot something but this breaks build here make -j32

Re: [FFmpeg-devel] [PATCH] libavcodec/gifenc: Only write palette entries that actually used

2021-02-05 Thread Derek Buitenhuis
On 05/02/2021 01:54, Andreas Rheinhardt wrote: > Could AV_CODEC_FLAG_GLOBAL_HEADER be used for this? It could, but I don't think it should. The global header is still written, just not with a global palette. I think it would be, at best, confusing, especially in terms of intent. - Derek

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add xbm parser

2021-02-05 Thread James Almer
On 2/3/2021 10:10 AM, Paul B Mahol wrote: Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/parsers.c| 1 + libavcodec/xbm_parser.c | 101 3 files changed, 103 insertions(+) create mode 100644

[FFmpeg-devel] [PATCH] avcodec/g722enc: Validate parameters before using them

2021-02-05 Thread Andreas Rheinhardt
In case trellis is outside of 0..23, an invalid shift and/or a signed integer overflow happens; furthermore, it can lead to the request to allocate nonsense amounts of memory. So validate first. Signed-off-by: Andreas Rheinhardt --- libavcodec/g722enc.c | 25 - 1 file

Re: [FFmpeg-devel] [PATCH 1/2] avcodec: add xbm parser

2021-02-05 Thread Paul B Mahol
Will apply soon. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] x11grab: capture a window instead of the whole screen

2021-02-05 Thread Paul B Mahol
Looks fine. On Tue, Jan 26, 2021 at 11:24 AM wrote: > In my previous email the patch got mangled by the web mail interface, so I > am > sending it again. Sorry for the duplicate. > > This patch allows ffmpeg to capture a specific window instead of the whole > screen. An example: > > ffmpeg -f

[FFmpeg-devel] [PATCH] avfilter/vf_tile: remove extra whitespace in option description

2021-02-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- libavfilter/vf_tile.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_tile.c b/libavfilter/vf_tile.c index 6278f72abb..7e421a9721 100644 --- a/libavfilter/vf_tile.c +++ b/libavfilter/vf_tile.c @@ -63,7 +63,7 @@ static const

[FFmpeg-devel] [PATCH] avfilter/avfilter: move enable_str expression parsing into avfilter_init_dict()

2021-02-05 Thread Paul B Mahol
This ensures that needed arrays are always allocated and properly initialized. Previously if code would use only avfilter_init_dict() to set options for filters it would not allocate arrays for timeline processing thus it would crash if user supplied enable option for filter(s). Signed-off-by:

Re: [FFmpeg-devel] [PATCH v2] avformat/concatdec: add support for setting input options

2021-02-05 Thread Moritz Barsnick
On Wed, Feb 03, 2021 at 20:37:35 +0200, Jan Ekström wrote: > > +@item @code{input_options @var{key=value:key2=value2}} > +Input options passed on when reading a specific file, using a :-separated > list > +of key=value pairs. Requires @code{safe} to be non-positive. Global options > for > +all