Re: [FFmpeg-devel] [RFC][PATCH] web: News about FFmpeg in Debian

On 2014-10-06 01:35 +, Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > > > The bad news is that the package probably won't migrate to Debian > > testing in time for the upcoming release "Jessie". The Debian release > > and security teams have made this decision because of concerns o

Re: [FFmpeg-devel] [PATCH] web/index: add news about OPW and samsungs donation

Hi Michael! On 2014-10-08 18:58 +0200, Michael Niedermayer wrote: > --- > src/index | 26 ++ > 1 file changed, 26 insertions(+) > > diff --git a/src/index b/src/index > index 8d3f541..f1b5251 100644 > --- a/src/index > +++ b/src/index > @@ -36,6 +36,32 @@ > News >

Re: [FFmpeg-devel] [RFC] Direct Stream Transfer (DST) decoder

On 2014-10-09 08:13 +0200, Reimar Döffinger wrote: > On 7 October 2014 08:41:09 CEST, Peter Ross wrote: [...] > >> > +if ((ret = init_get_bits8(&gb, avpkt->data, avpkt->size)) < 0) > >> > +if ((ret = read_map(&gb, &fsets, map_ch_to_felem, > >avctx->channels)) < 0) > >> > +if ((ret

Re: [FFmpeg-devel] [RFC][PATCH] web: News about FFmpeg in Debian

On 2014-10-07 08:13 -0700, Timothy Gu wrote: > On Oct 6, 2014 3:23 PM, "Alexander Strasser" wrote: [...] > > I have locally changed all other things requested by Timothy > > and Carl Eugen. I do not think the Libav deprecation message > > should be part of this ne

Re: [FFmpeg-devel] [RFC] Direct Stream Transfer (DST) decoder

On 2014-10-10 10:33 +0200, wm4 wrote: > On Fri, 10 Oct 2014 09:57:20 +0200 > Alexander Strasser wrote: > > > On 2014-10-09 08:13 +0200, Reimar Döffinger wrote: > > > On 7 October 2014 08:41:09 CEST, Peter Ross wrote: > > [...] > > > >> > +i

Re: [FFmpeg-devel] Facebook page

On 2014-10-15 10:49 +0200, Thomas Volkert wrote: > On 10/05/2014 09:09 PM, compn wrote: > >On Sun, 05 Oct 2014 20:19:49 +0200 > >Thomas Volkert wrote: > > > >>http://www.facebook.com/ffmpeg.devel, > >>http://www.facebook.com/ffmpeg.framework, > >>http://www.facebook.com/multimedia.framework, ... >

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

On 2014-10-26 02:49 +0200, Michael Niedermayer wrote: > On Sun, Oct 26, 2014 at 02:09:19AM +0200, wm4 wrote: > > On Sun, 26 Oct 2014 00:16:26 +0200 > > Nicolas George wrote: > > > > > Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > > > > This fixes the issue that a not set or

Re: [FFmpeg-devel] [CLT2018] FFmpeg at Chemnitzer Linux-Tage

On 2018-03-15 17:38 +0100, Paul B Mahol wrote: > On 3/13/18, Thilo Borgmann wrote: > > Am 13.03.18 um 19:52 schrieb Paul B Mahol: > >> On 3/13/18, Thilo Borgmann wrote: > >>> Hi, > >>> > once again, FFmpeg has been accepted for CLT 2018 in Chemnitz, Germany! > This "Chemnitzer Linux Tag

Re: [FFmpeg-devel] [CLT2018] FFmpeg at Chemnitzer Linux-Tage

On 2018-03-13 20:08 +0100, Thilo Borgmann wrote: > Am 13.03.18 um 19:52 schrieb Paul B Mahol: > > On 3/13/18, Thilo Borgmann wrote: > >> Hi, > >> > >>> once again, FFmpeg has been accepted for CLT 2018 in Chemnitz, Germany! > >>> This "Chemnitzer Linux Tage" will take place on 10th and 11th of Mar

Re: [FFmpeg-devel] [PATCH] avfilter/af_channelsplit: add channels option

Hi Paul! On 2018-03-20 19:11 +0100, Paul B Mahol wrote: > So user can pick which channels to extract. > > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 15 > libavfilter/af_channelsplit.c | 54 > +++ > 2 files changed,

Re: [FFmpeg-devel] [PATCH] avfilter/af_channelsplit: add channels option

On 2018-03-22 16:04 +0100, Nicolas George wrote: > Paul B Mahol (2018-03-22): > > So user can pick which channels to extract. > > > > Signed-off-by: Paul B Mahol > > --- > > doc/filters.texi | 18 ++ > > libavfilter/af_channelsplit.c | 39

Re: [FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data

Hi Derek, I am happy to see someone trying to extend FFmpeg to support these kind of features in a deeper way. Good luck on this journey! Below I am mostly commenting on typos and rather minor things. So you should for now mostly ignore my comments, but it is easier to write them now that I read

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/mpegaudio_parser: Skip APE tags when parsing mp3 packets.

On 2018-01-30 04:29 +, Dale Curtis wrote: > ffmpeg | branch: master | Dale Curtis | Mon Jan 29 > 15:10:26 2018 -0800| [42323c3e3a600288e4bf1cefe952486ffc29d280] | committer: > Michael Niedermayer > > avcodec/mpegaudio_parser: Skip APE tags when parsing mp3 packets. > > Otherwise the decode

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

Hi! On 2018-02-04 14:22 +, wm4 wrote: > ffmpeg | branch: master | wm4 | Thu Jan 25 18:57:26 > 2018 +0100| [ff46124b0df17a1d35249e09ae8eae9a61f16e04] | committer: wm4 > > rtsp: rename certain options after a deprecation period > > The "timeout" option name inherently clashes with the meanin

Re: [FFmpeg-devel] [PATCH 06/11] [WIP][RFC]tools: add ffserver_config_test

Hi Lukasz! On 2014-11-19 00:06 +0100, Lukasz Marek wrote: > On 18.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: > >Hi. Overall this looks good, thanks. I would move the test to > >tests/ though and integrate it with the build/test system as, > >say, the way tiny_ssim is: > > > >grep -R tiny_s

[FFmpeg-devel] [PATCH 0/2][RFC] xbr, hqx: minor simplification

visual inspection show no differences. It sure is a matter of taste, so I guess it is your call Clément. Alexander Strasser (2): lavfi/vf_xbr: Remove function generator macro and function pointer indirection lavfi/vf_hqx: Remove function generator macro and function pointer indirection

[FFmpeg-devel] [PATCH 1/2][RFC] lavfi/vf_xbr: Remove function generator macro and function pointer indirection

It is easier to read and shorter this way. Signed-off-by: Alexander Strasser --- libavfilter/vf_xbr.c | 33 ++--- 1 file changed, 10 insertions(+), 23 deletions(-) diff --git a/libavfilter/vf_xbr.c b/libavfilter/vf_xbr.c index 47e4b76..a67aa13 100644 --- a

[FFmpeg-devel] [PATCH 2/2][RFC] lavfi/vf_hqx: Remove function generator macro and function pointer indirection

It is easier to read and shorter this way. Signed-off-by: Alexander Strasser --- libavfilter/vf_hqx.c | 35 +++ 1 file changed, 11 insertions(+), 24 deletions(-) diff --git a/libavfilter/vf_hqx.c b/libavfilter/vf_hqx.c index 4783381..d3e65e9 100644 --- a

[FFmpeg-devel] [project server] moving homepage to the new server

Hello all, we are in the process of migrating the FFmpeg website to the new server. The homepage and the git repos are now resolving to the new server. So if you have SSH access to the ffmpeg-web.git or fateserver.git repositories you will get a fat warning that the host changed (possible dns s

Re: [FFmpeg-devel] [PATCH] XMA1 and XMA2 stereo decoders

Hi Paul! On 2015-11-04 14:18 +0100, Paul B Mahol wrote: [...] > diff --git a/libavcodec/wmaprodec.c b/libavcodec/wmaprodec.c > index 19dc335..7471961 100644 > --- a/libavcodec/wmaprodec.c > +++ b/libavcodec/wmaprodec.c > @@ -86,6 +86,8 @@ > * subframe in order to reconstruct the output samples.

Re: [FFmpeg-devel] [PATCH 3/3] web/documentation: Simplify Components Documentation links

Am 6. September 2014 19:23:25 MESZ, schrieb "Clément Bœsch" : >On Sat, Sep 06, 2014 at 09:12:47AM -0800, Lou Logan wrote: >> On Sat, Sep 6, 2014, at 04:37 AM, db0company wrote: >> > --- >> > src/documentation | 18 +- >> > 1 files changed, 9 insertions(+), 9 deletions(-) >> >>

Re: [FFmpeg-devel] [PATCH 3/3] web/documentation: Simplify Components Documentation links

. Alexander Am 10. November 2015 10:01:23 MEZ, schrieb Alexander Strasser : > > >Am 6. September 2014 19:23:25 MESZ, schrieb "Clément Bœsch" : >>On Sat, Sep 06, 2014 at 09:12:47AM -0800, Lou Logan wrote: >>> On Sat, Sep 6, 2014, at 04:37 AM, db0company wrote: >>

Re: [FFmpeg-devel] [PATCH v5] lavf: palettized QuickTime video in Matroska

On 2015-12-23 13:33 +0100, Hendrik Leppkes wrote: > Am 23.12.2015 13:28 schrieb "Mats Peterson" < > matsp888-at-yahoo@ffmpeg.org>: > > > > On 12/23/2015 01:15 PM, Mats Peterson wrote: > >> > >> On 12/23/2015 01:13 PM, Carl Eugen Hoyos wrote: > >>> > >>> Mats Peterson ffmpeg.org> writes: > >>>

Re: [FFmpeg-devel] Offering to help out - keeping the merges going and ML patch handling

Hi Hendrik! On 2015-08-01 17:07 +0200, Hendrik Leppkes wrote: > I've offered on IRC already, but I figured doing it formally on the ML > is probably best. > > I would be willing to take over doing the merges from Libav for the > time being, until we decide otherwise, and on top I can probably spa

Re: [FFmpeg-devel] Gathering opinions from FFmpeg developers about a reunification with Libav developers

On 2015-08-01 11:48 +0200, Hendrik Leppkes wrote: [...] > I don't have a very strong opinion on the bug tracker, I've seen > better and worse systems than trac, but keeping the existing issues is > probably a good idea - even if a lot of them are user support > questions and not actual issues. J

[FFmpeg-devel] [PATCH 1/3] libavutil: av_parse_time: Fix typo in doc

Signed-off-by: Alexander Strasser --- libavutil/parseutils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/parseutils.h b/libavutil/parseutils.h index e66d24b..b04aa89 100644 --- a/libavutil/parseutils.h +++ b/libavutil/parseutils.h @@ -112,7 +112,7 @@ int

[FFmpeg-devel] [PATCH 2/3] RFC: Should be squashed in the previous patch if OK

Signed-off-by: Alexander Strasser --- libavutil/parseutils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/parseutils.h b/libavutil/parseutils.h index b04aa89..cff5123 100644 --- a/libavutil/parseutils.h +++ b/libavutil/parseutils.h @@ -112,7 +112,7 @@ int

[FFmpeg-devel] [PATCH 3/3] doc/metadata: Default timestamp unit is nanoseconds

Signed-off-by: Alexander Strasser --- doc/metadata.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/metadata.texi b/doc/metadata.texi index bddcc99..be91059 100644 --- a/doc/metadata.texi +++ b/doc/metadata.texi @@ -33,7 +33,7 @@ At the beginning of a chapter section

Re: [FFmpeg-devel] [PATCH 2/3] RFC: Should be squashed in the previous patch if OK

On 2015-08-14 17:35 -0400, Ganesh Ajjanagadde wrote: > On Fri, Aug 14, 2015 at 4:46 PM, Alexander Strasser wrote: > > Signed-off-by: Alexander Strasser > > --- > > libavutil/parseutils.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >

Re: [FFmpeg-devel] [PATCH 1/3] libavutil: av_parse_time: Fix typo in doc

On 2015-08-14 16:59 -0400, Ganesh Ajjanagadde wrote: > On Fri, Aug 14, 2015 at 4:45 PM, Alexander Strasser wrote: > > Signed-off-by: Alexander Strasser > > --- > > libavutil/parseutils.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >

[FFmpeg-devel] [CLT2015] FFmpeg at Chemnitzer Linux-Tage

developer/contributor/supporter to participate. Currently Gerion Entrup and Alexander Strasser are registered as booth staff and therefore will attend for sure. If you want to drop by or help out, it would be nice if you let us know about it here. If you have comments, ideas, or help to offer, pl

Re: [FFmpeg-devel] [PATCH 2/3] libx265: Return proper invalid data code

On 2015-02-17 22:15 +, Derek Buitenhuis wrote: > On 2/17/2015 10:10 PM, James Almer wrote: > > Afaik, if the error is based on user input then EINVAL (Invalid argument) > > is correct. > > AVERROR_INVALIDDATA is when the error is in the bitstream/container and not > > an argument the > > use

Re: [FFmpeg-devel] [PATCH] web/index: add link to ff-facebook

On 2015-03-18 20:10 +0100, Michael Niedermayer wrote: > --- > src/index |3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/index b/src/index > index d4fbc37..fbb3c05 100644 > --- a/src/index > +++ b/src/index > @@ -31,7 +31,8 @@ > >   >https://plus.

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: Do not overwrite output if there is no input

tergraphs - 1; > filtergraphs[nb_filtergraphs - 1]->graph_desc = graph_desc; > + > +input_stream_potentially_available = 1; > + > return 0; > } > > -- From dbf318b0068d04fbe5d2890f6a7e376e092a8219 Mon Sep 17 00:00:00 2001 Message-Id: From: Alexander Strass

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

On 2015-03-29 00:32 +0100, Timo Rothenpieler wrote: > > Le nonidi 9 germinal, an CCXXIII, wm4 a écrit : > >> amovie1.zip, and then "/tmp/amovie1.zip//amovie.srt" is a perfectly > >> valid unix path. Multiple slashes are coalesced into 1. > > > > Just as you say it without realizing it, it is valid

Re: [FFmpeg-devel] [PATCH] lavf: add zip protocol

behaviour. Do not expect it to be beautiful and flawless. Also I didn't even attempt to work out details like option name and if we should match first or last .zip occurrence when guessing. Alexander From 0af78adac09fb6b93a4cc39c71d6e562f9689153 Mon Sep 17 00:00:00 2001 Message-Id: <0af78adac09

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

Hi Anton, hi Jun! On 2020-07-01 16:23 +0200, Anton Khirnov wrote: > Quoting Jun Zhao (2020-06-29 15:23:10) > > From: Jun Zhao > > > > Fix the potential overflow. > > > > Suggested-by: Alexander Strasser > > Signed-off-by: Jun Zhao > > ---

Re: [FFmpeg-devel] RfP Candidates

On 2020-06-29 16:04 +0200, Jean-Baptiste Kempf wrote: > Hello, > > The next step in the community voting process is to re-elect the Community > and Technical Committees. > > Therefore, we need candidates. > > Reminders: > The community committee is here to help smooth things down between people in

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add a template for refcounted objects.

Hi all! On 2020-07-01 17:05 +0200, Anton Khirnov wrote: > Quoting Nicolas George (2020-06-27 17:16:44) > > Signed-off-by: Nicolas George > > --- > > libavutil/avrefcount_template.h | 140 > > tests/ref/fate/source | 1 + > > 2 files changed, 141 inser

Re: [FFmpeg-devel] [PATCH v2] avfilter/vf_edgedetect: properly implement double_threshold()

On 2020-07-04 23:19 -0400, Andriy Gelman wrote: > On Sun, 05. Jul 09:37, lance.lmw...@gmail.com wrote: > > On Sat, Jul 04, 2020 at 01:03:48PM -0400, Andriy Gelman wrote: > > > On Mon, 29. Jun 09:26, Valery Kot wrote: > > > > On Sun, Jun 28, 2020 at 12:03 AM Andriy Gelman > > > > wrote: > > > > >

Re: [FFmpeg-devel] [PATCH] avutil: Add NV61 pixel format and enable IO support for NV16 and NV61 pixel formats

285bceefdab9a > > nv12381574979cb04be10c9168540310afad > > +nv16d3a50501d2ea8535489fd5ec49e7866d > > nv210fdeb2cdd56cf5a7147dc273456fa217 > > nv24193b9eadcc06ad5081609f76249b3e47 > > nv421

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

Hi Paul! Only doc comments. Feel free to apply the changes when the real review has completed. On 2020-07-12 13:01 +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 46 > libavfilter/Makefile | 2 + > libavfilter/allfilters.c | 2 + > liba

Re: [FFmpeg-devel] [PATCH 1/4] avformat/au: Store strings instead of pointers to strings in array

Hi Andreas, no objections for the patchset as a whole. Just for the first in the series I have some questions. The commit message is: avformat/au: Store strings instead of pointers to strings in array This tells the what, but not the why. On 2020-07-17 13:14 +0200, Andreas Rheinhardt wro

Re: [FFmpeg-devel] [PATCH] avfilter: add radial and circular blur video filter

Hi all, this mail thread has turned into an example of exactly what I would not like to see in a patch discussion! It doesn't really respect the first sentence of our code of conduct: Be friendly and respectful towards others and third parties. Furthermore it discourages testing, code revi

Re: [FFmpeg-devel] [PATCH v5] avdevice/xcbgrab: check return values of xcb query functions

On 2020-07-16 23:54 -0400, Andriy Gelman wrote: > On Tue, 14. Jul 14:14, Moritz Barsnick wrote: [...] > > Subject: [PATCH] avdevice/xcbgrab: check return values of xcb query > > functions > > > > Fixes #7312, segmentation fault on close of X11 server > > > > xcb_query_pointer_reply() and xcb_get_g

Re: [FFmpeg-devel] [PATCH v5] avdevice/xcbgrab: check return values of xcb query functions

Hi Andriy! On 2020-07-19 19:47 -0400, Andriy Gelman wrote: > On Sun, 19. Jul 23:19, Alexander Strasser wrote: > > On 2020-07-16 23:54 -0400, Andriy Gelman wrote: > > > On Tue, 14. Jul 14:14, Moritz Barsnick wrote: [...] > > > > --- a/libavdevice/xcbgrab.c >

Re: [FFmpeg-devel] [PATCH 1/4] avformat/au: Store strings instead of pointers to strings in array

On 2020-07-19 23:38 +0200, Andreas Rheinhardt wrote: > Alexander Strasser: [...] > > > > On 2020-07-17 13:14 +0200, Andreas Rheinhardt wrote: > >> Andreas Rheinhardt: > >>> Signed-off-by: Andreas Rheinhardt > >>> --- > >>> libavformat

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

On 2020-07-01 21:05 +0200, Alexander Strasser wrote: > On 2020-07-01 16:23 +0200, Anton Khirnov wrote: > > Quoting Jun Zhao (2020-06-29 15:23:10) > > > From: Jun Zhao > > > > > > Fix the potential overflow. > > > > > > Suggested

Re: [FFmpeg-devel] [PATCH v2 1/3] libavutil/imgutils: add utility to get plane sizes

On 2020-07-22 11:56 +0200, Nicolas George wrote: > James Almer (12020-07-20): > > No, i'll push v3 soon if my argumentation below was not enough to > > convince Nicolas or Michael. My intention is to use ints for the new > > function, not to postpone committing it in any form indefinitely. > > Sorr

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/alac: Check decorr_shift to avoid invalid shift

Hi Michael! On 2020-07-24 00:11 +0200, Michael Niedermayer wrote: > Fixes: shift exponent 128 is too large for 32-bit type 'int' > Fixes: > 23860/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ALAC_fuzzer-5751138914402304 > > Found-by: continuous fuzzing process > https://github.com/google/os

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

On 2020-07-24 19:56 +0800, zhilizhao wrote: > > > > On Jul 24, 2020, at 9:40 AM, myp...@gmail.com wrote: > > > > On Fri, Jul 24, 2020 at 4:43 AM Alexander Strasser > <mailto:eclip...@gmx.net>> wrote: > >> > >> On 2020-07-01 21:05 +0200, Ale

Re: [FFmpeg-devel] AVWriter again (was: v2 1/2] avformat/url: check double dot is not to parent directory)

On 2020-07-28 20:43 +0800, zhilizhao wrote: > > On Jul 28, 2020, at 8:02 PM, Nicolas George wrote: > > > > zhilizhao (12020-07-28): > >> I think jb is referring to > >> > >> FILE *open_memstream(char **bufp, size_t *sizep); > >> https://linux.die.net/man/3/open_memstream > >>

[FFmpeg-devel] [PATCH] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

Don't pass a potential NULL pointer to av_log, instead use a default in the rc_eq AVOption definitions. Now the context variable always holds a string; even if it's the default expression. Note this also fixes the evaluation error path, where a similar av_log references the rc_eq string from the c

Re: [FFmpeg-devel] [PATCH] avfilter/vf_xcam: add xcam video filter

Hi! On 2020-07-30 12:16 +, Zane van Iperen wrote: > On Fri, 31 Jul 2020 00:55:56 +0800 > "zongwave" wrote: [...] > > > ++static av_cold int xcam_init(AVFilterContext *ctx) > > ++{ > > ++XCAMContext *s = ctx->priv; > > ++int ret = 0; > > ++ > > ++s->handle = xcam_create_handle(s->n

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

On 2020-07-30 21:18 +0800, myp...@gmail.com wrote: > > After repeated thinking， I can accept this solution, thx Just to avoid confusion. You accept this > > > > avctx->bit_rate += (s->bit_rate - avctx->bit_rate) / s->frame_number; or this > > If you are aiming for improving accuracy, I guess go

Re: [FFmpeg-devel] [PATCH] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

Hi Michael! On 2020-07-31 19:54 +0200, Michael Niedermayer wrote: > On Thu, Jul 30, 2020 at 02:42:30PM +0200, Alexander Strasser wrote: > > Don't pass a potential NULL pointer to av_log, instead use a default > > in the rc_eq AVOption definitions. Now the context variabl

[FFmpeg-devel] [PATCH 1/2] lavc/snowenc: Expose an option to set the rc_eq expression

options definition of rc_eq (libavcodec/mpegvideo.h), with some minor style adjustments to be closer to the other snowenc option initializer expressions. Signed-off-by: Alexander Strasser --- libavcodec/snowenc.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/snowenc.c b

[FFmpeg-devel] [PATCH 2/2] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

tring from the context too. Signed-off-by: Alexander Strasser --- libavcodec/mpegvideo.h | 2 +- libavcodec/ratecontrol.c | 3 +-- libavcodec/snowenc.c | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/mpegvideo.h b/libavcodec/mpegvideo.h index 29e692f245..

Re: [FFmpeg-devel] [PATCH] doc/developer: origin of tables should be documented.

Hi Nicolas! On 2020-08-03 11:54 +0200, Nicolas George wrote: > Nicolas George (12020-07-31): > > Something like this would be acceptable: > > > > /* Reverse-engineered by encoding various files with the reference > >encoder. */ > > > > Reluctance to give this little information is baffling to

Re: [FFmpeg-devel] [PATCH] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

On 2020-08-02 20:49 +0200, Michael Niedermayer wrote: > On Sat, Aug 01, 2020 at 11:23:36PM +0200, Alexander Strasser wrote: > > Hi Michael! > > > > On 2020-07-31 19:54 +0200, Michael Niedermayer wrote: > > > On Thu, Jul 30, 2020 at 02:42:30PM +0200, Alexander Strass

Re: [FFmpeg-devel] [PATCH 4/6] avformat/argo_asf: add version_major and version_minor options

Hi Zane! Am 8. August 2020 10:01:07 MESZ schrieb Zane van Iperen : >Signed-off-by: Zane van Iperen >--- > libavformat/argo_asf.c | 37 +++-- > 1 file changed, 35 insertions(+), 2 deletions(-) > >diff --git a/libavformat/argo_asf.c b/libavformat/argo_asf.c >index 1

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

Hi Nicolas! Am 8. August 2020 23:26:02 MESZ schrieb Mark Thompson : >On 08/08/2020 18:57, Nicolas George wrote: [...] >> Andreas: I'm sure the following was addressed at me. >>> Not sure I agree with your definition of Libre Software and your >exact >> >> I was more trying to express the d

Re: [FFmpeg-devel] [PATCH 2/2] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

Am 9. August 2020 12:56:53 MESZ schrieb Michael Niedermayer : >On Fri, Aug 07, 2020 at 11:26:58PM +0200, Alexander Strasser wrote: >> Don't pass a potential NULL pointer to av_log, instead use a default >> in the rc_eq AVOption definitions. Now the context variable alwa

Re: [FFmpeg-devel] [PATCH 4/6] avformat/argo_asf: add version_major and version_minor options

Am 8. August 2020 14:52:02 MESZ schrieb Zane van Iperen : >On Sat, 08 Aug 2020 13:05:28 +0200 >"Alexander Strasser" wrote: > >> >@@ -296,8 +298,7 @@ static int argo_asf_write_header(AVFormatContext >> >*s) >> > ArgoASFContext *ctx

Re: [FFmpeg-devel] [PATCH v2 1/4] avformat/argo_asf: don't check or probe file version

Am 9. August 2020 14:33:26 MESZ schrieb Zane van Iperen : >On Sun, 9 Aug 2020 09:29:16 +0200 >"Paul B Mahol" wrote: > >> >> This is really bad practice. >> > >Usually I'd agree. However (and I've just checked this) all file >versions are identical. > >Since this is an Argonaut Games format, I

Re: [FFmpeg-devel] [PATCH v3 1/5] avformat/argo_asf: add games to version list

Hi Zane, thanks for addressing my comments in v3. AFAICS you also took Andreas' feedback into account, but I didn't check that. No more remarks from me. Alexander ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/list

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

On 2020-08-09 13:13 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-09): > > >> Andreas: [...] > > > At least the aspect Mark mentioned below. > > > > In general I think it could be worded a bit clearer and easier to grasp. > > Basically it

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

On 2020-08-12 12:32 +0200, Jean-Baptiste Kempf wrote: > On Wed, 12 Aug 2020, at 00:29, Alexander Strasser wrote: > > Definitions of non-obvious data should have a short comment > > explaining their origin. > > > > If the data is of mathematical origin, you c

Re: [FFmpeg-devel] [PATCH 1/2] lavc/aac_ac3_parser: fix the potential overflow

On 2020-07-31 09:45 +0800, myp...@gmail.com wrote: > On Fri, Jul 31, 2020 at 3:47 AM Alexander Strasser wrote: > > > > On 2020-07-30 21:18 +0800, myp...@gmail.com wrote: > > > > > > After repeated thinking， I can accept this solution, thx > > >

Re: [FFmpeg-devel] [PATCH 1/2] lavc/snowenc: Expose an option to set the rc_eq expression

On 2020-08-09 14:54 +0200, Michael Niedermayer wrote: > On Fri, Aug 07, 2020 at 11:26:17PM +0200, Alexander Strasser wrote: > > Snow uses the ratecontrol module, but does not expose a way to set > > the rc_eq expression. The default expression, set in the ratecontrol > > m

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose().

Hi all! On 2020-08-12 14:53 +, Nicolas George wrote: > ffmpeg | branch: master | Nicolas George | Thu Jul 30 > 00:02:10 2020 +0200| [1201687da268c11459891a80ca1972aeaca8db88] | committer: > Nicolas George > > lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose(). > > Also add an

Re: [FFmpeg-devel] [PATCH 2/2] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

On 2020-08-09 14:56 +0200, Michael Niedermayer wrote: > On Sun, Aug 09, 2020 at 02:24:34PM +0200, Alexander Strasser wrote: > > > > > > Am 9. August 2020 12:56:53 MESZ schrieb Michael Niedermayer > > : > > >On Fri, Aug 07, 2020 at 11:26:58PM +0200, Alexander

Re: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse options in openvino backend

Am 14. August 2020 14:07:23 MESZ schrieb "Xu, Guangxin" : > > >> -Original Message- >> From: ffmpeg-devel On Behalf Of >Guo, >> Yejun >> Sent: Friday, August 14, 2020 9:50 AM >> To: ffmpeg-devel@ffmpeg.org >> Subject: [FFmpeg-devel] [PATCH V2] dnn_backend_openvino.c: parse >options >> in

Re: [FFmpeg-devel] [PATCH v2] doc/developer: origin of tables should be documented.

On 2020-08-14 11:34 +0200, Jean-Baptiste Kempf wrote: > On Wed, 12 Aug 2020, at 14:38, Alexander Strasser wrote: > > On 2020-08-12 12:32 +0200, Jean-Baptiste Kempf wrote: > > > On Wed, 12 Aug 2020, at 00:29, Alexander Strasser wrote: > > > > Definitions of non-ob

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose().

On 2020-08-13 00:13 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-12): > > Stupid question: Why do we transform relative URLs at all? > > > > Isn't it normally supposed to work for HTTP on the server-side too? > > > > Many clients seem to do i

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

Hi Derek, hi all! On 2020-08-14 20:22 +0100, Derek Buitenhuis wrote: > On 14/08/2020 20:13, Paul B Mahol wrote: > >> Resending because I accidentally replied to James instead of the list. > >> Woops. > >> > >> I guess it was not clear to me this is not the initial thread, since it is > >> not > >>

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

On 2020-08-16 18:39 +0200, Nicolas George wrote: > Paul B Mahol (12020-08-16): > > I do not like long name either. > > And do you have a better reason than the fact that it comes from me? This was uncalled for and not nice at all! It's neither good for you nor the community. There is nothing wro

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

On 2020-08-16 18:27 +0200, Nicolas George wrote: > Marton Balint (12020-08-16): > > Can we find a shorter name for the option? E.g. simply call the option > > auto_conversion_filters and the user can use -noauto_conversion_filters to > > disable it... In the next patch with the current name you hav

Re: [FFmpeg-devel] [PATCH 2/2] lavc/ratecontrol: Fix error logging for parsing and evaluation of rc_eq

On 2020-08-13 22:32 +0200, Michael Niedermayer wrote: > On Wed, Aug 12, 2020 at 08:58:30PM +0200, Alexander Strasser wrote: > > On 2020-08-09 14:56 +0200, Michael Niedermayer wrote: [...] > > > > > > lib user > > > > > > > If I'm not mist

Re: [FFmpeg-devel] [PATCH v3 5/5] avformat/argo_asf: strip file extension from name

Hi Zane, sorry for this late remark. On 2020-08-09 23:45 +, Zane van Iperen wrote: > Only when the user hasn't manually specified one. > Matches the original files more closely. > > Signed-off-by: Zane van Iperen > --- > libavformat/argo_asf.c | 20 +--- > 1 file changed, 17

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

Hi Nicolas! On 2020-08-16 23:03 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-16): > > This was uncalled for and not nice at all! > > I will be correct, but I'm done being nice with him. It was an unneeded accusation. Thanks for trying to not repeat those. &

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

On 2020-08-16 23:12 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-16): > > I dislike the negative name too, because like mentioned by Marton it > > doesn't work well with overriding the option to turn it off. > > > > On one hand for this option i

Re: [FFmpeg-devel] [PATCH] avcodec/cfhd: add x86 SIMD

Hi Paul, sorry, forgot to reply to this... On 2020-08-15 12:55 +0200, Paul B Mahol wrote: > On 8/15/20, Alexander Strasser wrote: > > On 2020-08-14 20:22 +0100, Derek Buitenhuis wrote: > >> On 14/08/2020 20:13, Paul B Mahol wrote: > >> > What specific insults in th

Re: [FFmpeg-devel] [PATCH V4] dnn_backend_openvino.c: parse options in openvino backend

Hi Yejun! On 2020-08-18 23:08 +0800, Guo, Yejun wrote: > Signed-off-by: Guo, Yejun > --- > v3: use AVOption > v4: don't add new file dnn_common.h/c > > libavfilter/dnn/dnn_backend_openvino.c | 50 > +- > 1 file changed, 49 insertions(+), 1 deletion(-) > > diff --

Re: [FFmpeg-devel] [PATCH V4] dnn_backend_openvino.c: parse options in openvino backend

On 2020-08-20 01:10 +, Guo, Yejun wrote: > > > -Original Message- > > From: ffmpeg-devel On Behalf Of > > Alexander Strasser > > Sent: 2020年8月20日 4:06 > > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH V4

[FFmpeg-devel] Curly braces around single statements (was avformat/avidec: Fix memleak when error) happens after creating DV stream

On 2020-08-20 22:32 +0200, Michael Niedermayer wrote: > On Thu, Aug 20, 2020 at 12:46:12PM +0200, Andreas Rheinhardt wrote: > > Michael Niedermayer: > > > On Wed, Aug 19, 2020 at 12:00:37AM +0200, Andreas Rheinhardt wrote: > > >> Signed-off-by: Andreas Rheinhardt > > >> --- > > >> The memleak can

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: add disable_all_auto_conversion_filters option.

On 2020-08-20 19:49 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-17): > > I think the pendulum can swing in both direction here. So the overall > > effect is not clear to me. E.g. one developer may think > > > > "hey what's this -> i nee

Re: [FFmpeg-devel] Curly braces around single statements (was avformat/avidec: Fix memleak when error) happens after creating DV stream

Hi Mark, thanks for your opinion on this! On 2020-08-20 23:35 +0100, Mark Thompson wrote: > On 20/08/2020 22:20, Alexander Strasser wrote: > > Please pardon me for bringing this up in the context of this patch. > > No objections or particular opinion regarding this instance of

Re: [FFmpeg-devel] [PATCH] libavfilter/vf_overlay_cuda: Pass the main frame to the output if secondary frame is not available. now filter fails if secondary frame is not available due to e.g secondary

On 2020-08-21 20:46 +1000, Alex Pokotilo wrote: > On Fri, Aug 21, 2020 at 7:54 PM Nicolas George wrote: > > > > Alex Pokotilo (12020-08-21): > > > From b3739e33fa04a9292dc6584bd2f31460aa53d478 Mon Sep 17 00:00:00 2001 > > > From: Alex Pokotilo > > > Date: Fri, 21 Aug 2020 09:14:37 + > > > Sub

Re: [FFmpeg-devel] [PATCH] fate: add fate-sub-dvb test

On 2020-08-21 14:32 +0200, Clément Bœsch wrote: > On Tue, Aug 18, 2020 at 11:43:29PM +0200, Michael Niedermayer wrote: > > On Tue, Aug 18, 2020 at 10:45:50PM +0200, Clément Bœsch wrote: > > > On Tue, Aug 11, 2020 at 02:19:01PM +0200, Clément Bœsch wrote: > > > [...] > > > > +FATE_SUBTITLES-$(call A

Re: [FFmpeg-devel] Documentation: proposed changes in the structure

On 2020-08-21 15:02 +0200, Jean-Baptiste Kempf wrote: > > > On Fri, 21 Aug 2020, at 14:35, Nicolas George wrote: > > 2. What would you think about switching from texinfo to a small basic > >subset of HTML for new documentation? > > > >I think most of us are much more familiar with HTML synt

Re: [FFmpeg-devel] Documentation: proposed changes in the structure

On 2020-08-21 14:35 +0200, Nicolas George wrote: > 1. What would you think about putting the documentation for >libavfilter/vf_foobar.c into libavfilter/doc/vf_foobar.texi instead >of into huge doc/filters.texi (25k lines!)? And same for codecs, >formats, etc. > >We can adopt this f

Re: [FFmpeg-devel] Documentation: proposed changes in the structure

Hi Nicolas, hi Clement! Wow the discussion gets up to speed quickly :) Thanks for bringing up all those good points! I'm intentionally top-posting, because I want to say something concerning the points you discuss below. Though I couldn't find a good way to do it in a comment style, so instead I'

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavf/url: rewrite ff_make_absolute_url() using ff_url_decompose().

On 2020-08-22 12:59 +0200, Nicolas George wrote: > Alexander Strasser (12020-08-15): > > If I'm not mistaken that reason for the rewrite didn't make > > it into the commit message: > > You are right. The reason for the complete rewrite was that the original > code

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec: add RPZA encoder

Hi all, hi Paul! Shouldn't it have been also reflected in the commit's metadata, author or message, that the encoder was originally written by Todd Kirby and David Adler? Did you base it on the version from Jaikrishnan Menon? If yes, I think it should have been mentioned in the commit message too

Re: [FFmpeg-devel] Documentation: proposed changes in the structure

Hi Jim! On 2020-08-22 14:09 -0700, Jim DeLaHunt wrote: > On  Fri Aug 21 15:35:38 EEST 2020,  Nicolas George wrote: > > 1. What would you think about putting the documentation for > > libavfilter/vf_foobar.c into libavfilter/doc/vf_foobar.texi … > > 2. What would you think about switching from

Re: [FFmpeg-devel] [RFC] Built-in documentation API

First about this discussion in general: There is a reason this is tagged RFC and the other thread is called proposal. I hope we can go on with the vivid discussions while keeping in mind that it is not yet narrowed down and nothing is decided. Always constraining all ideas, is not a good way to o

Re: [FFmpeg-devel] [PATCH 3/3] web/download: Rename builds anchors so old links posted elsewhere still work

On 2014-07-23 13:42 -0800, Lou Logan wrote: > On Tue, 22 Jul 2014 10:36:31 -0800, Lou Logan wrote: [...] > Actually I'd like to wait on this anchor patch since I contacted > Alexander to add some redirects instead, so the new, cleaner names can > be used. If that's not possible then I'll push this

Re: [FFmpeg-devel] [PATCH v2] libavdevice: Improve example in deprecation message for opengl and sdl

On 2024-04-29 23:53 +0200, Alexander Strasser via ffmpeg-devel wrote: > When piping ffmpeg into ffplay both programs write a status line in > the terminal. That causes flickering and invisibility of one or the > other status line. > > As compromise set ffplay log level to warning

Re: [FFmpeg-devel] [PATCH 1/4] lavu/float_dsp: add double-precision scalar product

On 2024-05-29 18:51 +0300, Rémi Denis-Courmont wrote: > Le keskiviikkona 29. toukokuuta 2024, 18.44.13 EEST Andreas Rheinhardt a écrit > : > > > +static double ff_scalarproduct_double_c(const double *v1, > > > Don't use an ff_ prefix for a static function. > > I can see over 300 such identifiers in

<    1   2   3   >