Re: [FFmpeg-devel] [ANN] Poll Results: GA voters list updates

2023-11-06 Thread Alexander Strasser
Hi all, hi J-B! On 2023-11-06 07:10 +0100, Jean-Baptiste Kempf wrote: > Yo, > > Time is up, results are here: > https://vote.ffmpeg.org/cgi-bin/civs/results.pl?id=E_029f7195fed7aadf Should I have been mailed about this vote? I'm pretty sure I could vote in 2020. Or am I just missing something?

Re: [FFmpeg-devel] [PATCH v2] doc: add spi.txt

2023-11-08 Thread Alexander Strasser
On 2023-10-16 00:49 +0200, Michael Niedermayer wrote: > This explains how to request refunds and what can be funded by SPI > > Co-Author: Stefano Sabatini > --- > doc/spi.txt | 73 + > 1 file changed, 73 insertions(+) > create mode 100644 doc/s

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo: Remove spec-incompliant inverse quantisation

2023-11-08 Thread Alexander Strasser
On 2023-11-08 12:40 +0100, Anton Khirnov wrote: > Quoting Michael Niedermayer (2023-10-31 09:40:44) > > On Mon, Oct 30, 2023 at 02:11:27PM +0100, Andreas Rheinhardt wrote: > > > Section 7.4.4 of the MPEG-2 specifications requires that the > > > last bit of the last coefficient be toggled so that th

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo: Remove spec-incompliant inverse quantisation

2023-11-09 Thread Alexander Strasser
On 2023-11-09 11:13 +0100, Anton Khirnov wrote: > Quoting Alexander Strasser (2023-11-08 21:55:10) > > On 2023-11-08 12:40 +0100, Anton Khirnov wrote: > > > Quoting Michael Niedermayer (2023-10-31 09:40:44) > > > > On Mon, Oct 30, 2023 at 02:11:27PM +

Re: [FFmpeg-devel] [PATCH] avcodec/mpegvideo: Remove spec-incompliant inverse quantisation

2023-11-09 Thread Alexander Strasser
On 2023-11-08 17:58 -0500, Vittorio Giovara wrote: > On Wed, Nov 8, 2023 at 3:46 PM Alexander Strasser wrote: > > > On 2023-11-08 12:40 +0100, Anton Khirnov wrote: > > > Quoting Michael Niedermayer (2023-10-31 09:40:44) > > > > On Mon, Oct 30, 2023 at 02:11:27P

Re: [FFmpeg-devel] [ANN] Poll Results: GA voters list updates

2023-11-09 Thread Alexander Strasser
Hi Anton! On 2023-11-09 13:15 +0100, Anton Khirnov wrote: > Quoting Alexander Strasser (2023-11-06 16:56:55) > > On 2023-11-06 07:10 +0100, Jean-Baptiste Kempf wrote: > > > Yo, > > > > > > Time is up, results are here: > > > https://vote.ffmpeg.org/

Re: [FFmpeg-devel] [ANNOUNCE] upcoming vote: extra members for GA

2023-11-09 Thread Alexander Strasser
gt; FFmpeg) > Moritz Barsnick(Member of the 2020 GA but was not on jbs list) > Lauri Kasanen (Linux / PowerPC maintainer) > Dale Curtis(14 commits in 2020 was in the 2020-GA) > Alexander Strasser (Root admin, just recently reported that he could not vote > even

Re: [FFmpeg-devel] [PATCH 1/2] tools/general_assembly: implement extra GA members

2023-11-26 Thread Alexander Strasser
On 2023-11-26 10:18 +0100, Anton Khirnov wrote: > Set pushed. > > The general_assembly.pl script should now be usable as the authoritative > source for GA members. The patches mostly LGTM. My Perl knowledge in general is really mostly from 20 years ago. So if there is any Perl-ish devil in the de

Re: [FFmpeg-devel] [PATCH] fate.sh: Allow overriding what targets to make for running the tests

2023-11-27 Thread Alexander Strasser
Hi Martin, patch looks technically good to me. On 2023-11-27 17:46 +0200, Rémi Denis-Courmont wrote: > Le maanantaina 27. marraskuuta 2023, 14.31.18 EET Martin Storsjö a écrit : > > This can be useful if doing testing of uncommon CPU extensions by > > running tests with QEMU (by configuring with

Re: [FFmpeg-devel] [ANNOUNCE] upcoming vote: TC/CC elections

2023-11-30 Thread Alexander Strasser
On 2023-11-30 00:14 +0100, Michael Niedermayer wrote: > On Tue, Nov 28, 2023 at 02:23:13PM +0100, Anton Khirnov wrote: > > For the record, I've edited the vote description to make it more clear. > > It now looks like this: > > > > Five people from the list below will become the members of the Tech

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: Return FFERROR_REDO when receiving a frame of unexpected size

2019-03-27 Thread Alexander Strasser
Hi Stephan! On 2019-03-25 20:32 +0100, Stephan Hilb wrote: > Alexander Strasser wrote on 21.03.2019 at 23:34: > > > 3. Return zero-sized packets => This works and is consistent with how > >we handle frames flagged to be corrupted (V4L2_BUF_FLAG_ERRO

Re: [FFmpeg-devel] [PATCH] avcodec/ccaption_dec: Add a blank like at the end to avoid rollup reading from outside

2019-04-22 Thread Alexander Strasser
Hi Michael! On 2019-04-20 18:11 +0200, Michael Niedermayer wrote: > Fixes: index 20 out of bounds for type 'const char *[4][128]' Somehow I don't understand this diagnostic message. It says "index 20 out of bounds for [4][128]". You're change looks like an off-by-one fix. Sorry; I'm surely miss

Re: [FFmpeg-devel] [PATCH] avfilter/af_astats: count number of NaNs/Infs/denormals for floating-point audio too

2019-04-22 Thread Alexander Strasser
Hi Paul, just three small comments from me... On 2019-04-22 11:51 +0200, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi| 6 +++ > libavfilter/af_astats.c | 86 ++--- > 2 files changed, 86 insertions(+), 6 deletions(-) I w

Re: [FFmpeg-devel] [PATCH V2 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-04-22 Thread Alexander Strasser
Hi! On 2019-04-15 21:23 +0800, Guo, Yejun wrote: > take decoder names an example, with the default page length, shell command > 'pr' needs two pages for all the decoder names. The names are firstly printed > in the first page, then in the second page. So, as a whole, the names are > sorted neither

Re: [FFmpeg-devel] [PATCH] avfilter/af_astats: count number of NaNs/Infs/denormals for floating-point audio too

2019-04-27 Thread Alexander Strasser
On 2019-04-25 18:49 +0200, Paul B Mahol wrote: > On 4/23/19, Alexander Strasser wrote: [...] > > > > On 2019-04-22 11:51 +0200, Paul B Mahol wrote: > >> Signed-off-by: Paul B Mahol > >> --- > >> doc/filters.texi|

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-04-27 Thread Alexander Strasser
831,8 +3831,24 @@ die_unknown(){ } print_in_columns() { -cols=$(expr $ncols / 24) -cat | tr ' ' '\n' | sort | pr -r "-$cols" -w $ncols -t +sort | tr '\n' ' ' | awk -v width="$ncols" ' +{ + num_cols = width /

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-04-28 Thread Alexander Strasser
On 2019-04-28 03:11 +, avih wrote: > > What do you think about using awk instead of shell? > > No objection here, especially if it's more robust in some ways than this > or other shell code (though personally I'm not fluent in awk). > > My only concern was preventing a considerable performance

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-04-28 Thread Alexander Strasser
On 2019-04-28 07:42 +, Guo, Yejun wrote: > > -Original Message- > > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > > Alexander Strasser > > Sent: Sunday, April 28, 2019 9:18 AM > > To: FFmpeg development discussions and pat

[FFmpeg-devel] [PATCH 0/2] configure: Replace pr with awk and fix column sorting

2019-05-01 Thread Alexander Strasser
rsion on master. I would appreciate more testing, especially on the less popular platforms. Alexander Strasser (2): configure: print_in_columns: Replace pr with awk configure: log_file: Replace pr with awk invocation configure | 20 +--- 1 file changed, 17 insertions(

[FFmpeg-devel] [PATCH 1/2] configure: print_in_columns: Replace pr with awk

2019-05-01 Thread Alexander Strasser
down one of the columns. Fixes output for sizes with width < column width, too. Fixes part of ticket #5680 Signed-off-by: Alexander Strasser --- configure | 18 -- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/configure b/configure index b122b27268..81e377606

[FFmpeg-devel] [PATCH 2/2] configure: log_file: Replace pr with awk invocation

2019-05-01 Thread Alexander Strasser
Fixes remaining part of ticket #5680 Signed-off-by: Alexander Strasser --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 81e3776060..96ada8f636 100755 --- a/configure +++ b/configure @@ -503,7 +503,7 @@ log(){ log_file(){ log

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-01 Thread Alexander Strasser
On 2019-04-29 00:55 +, Guo, Yejun wrote: [...] > > Wild guess: CR LF line endings are emitted somewhere and the CRs stay in > > the input. Your terminal resets the cursor to the start of the line when > > interpreting the midline CRs. > > > > Does it work if you extend the tr in print_in_column

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-03 Thread Alexander Strasser
Hi Avih! On 2019-05-02 08:55 +, avih wrote: > > It seems awk is unconditionally required already. However I wanted to > > say that it's a very nice dep to have > > While it's possibly nicer than other deps to have, it's still better to use > it IMHO only when it adds some value, like simpler c

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-03 Thread Alexander Strasser
Hi Carl Eugen! On 2019-05-01 22:57 +0200, Carl Eugen Hoyos wrote: > 2019-04-28 3:18 GMT+02:00, Alexander Strasser : > > > What do you think about using awk instead of shell? > > Do we only use awk for --enable-random and the dependency > files so far? Does configure also wor

Re: [FFmpeg-devel] [PATCH V5 2/2] configure: replace 'pr' with printf since busybox does not support pr

2019-05-04 Thread Alexander Strasser
Hi all! On 2019-04-28 00:38 +, Guo, Yejun wrote: > > From: avih [mailto:avih...@yahoo.com] > > Sent: Wednesday, April 24, 2019 9:23 PM > > To: FFmpeg development discussions and patches > > Cc: Guo, Yejun > > Subject: Re: [FFmpeg-devel] [PATCH V5 2/2] configure: replace 'pr' with > > printf

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-04 Thread Alexander Strasser
Hi! On 2019-05-04 06:28 +, avih wrote: > > On 2019-05-02 08:55 +, avih wrote: > > > > It seems awk is unconditionally required already. However I wanted to > > > > say that it's a very nice dep to have > > > > > > While it's possibly nicer than other deps to have, it's still better to > >

Re: [FFmpeg-devel] [PATCH 1/2] configure: print_in_columns: Replace pr with awk

2019-05-05 Thread Alexander Strasser
Am 5. Mai 2019 03:53:20 MESZ schrieb "Guo, Yejun" : > > >> -Original Message- >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >Of >> Alexander Strasser >> Sent: Thursday, May 02, 2019 12:08 AM >> To: ffmpeg-devel

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-06 Thread Alexander Strasser
g the execution of other programs. Sometimes when you do not have a particular program around to help you in your shell script, that void needs to be filled. For this purpose awk is often a good choice, because it was especially designed to be a fit for this purpose amongst other things. Using only aw

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-05-06 Thread Alexander Strasser
Am 7. Mai 2019 04:07:23 MESZ schrieb "Guo, Yejun" : > >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf >Of >> Alexander Strasser >> Sent: Tuesday, May 07, 2019 6:21 AM >> To: FFmpeg development discussions and patches > &

Re: [FFmpeg-devel] [PATCH V5 2/2] configure: replace 'pr' with printf since busybox does not support pr

2019-05-15 Thread Alexander Strasser
On 2019-05-05 01:38 +, Guo, Yejun wrote: > > > -Original Message- > > Alexander Strasser > > Sent: Sunday, May 05, 2019 3:42 AM > > > > On 2019-04-28 00:38 +, Guo, Yejun wrote: > > > > From: avih [mailto:avih...@yahoo.com] > > >

Re: [FFmpeg-devel] [PATCH 2/2] configure: log_file: Replace pr with awk invocation

2019-05-15 Thread Alexander Strasser
Dropped. Pushed Yejun's version instead: [PATCH V5 2/2] configure: replace 'pr' with printf since busybox does not support pr On 2019-05-01 18:08 +0200, Alexander Strasser wrote: > Fixes remaining part of ticket #5680 > > Signed-off-by: Alexander Strasser > -

Re: [FFmpeg-devel] [DECISION] Ban Nicolas George from project

2019-05-18 Thread Alexander Strasser
On 2019-05-18 11:57 -0300, James Almer wrote: > On 5/18/2019 9:24 AM, Paul B Mahol wrote: > > On 5/18/19, Reimar Döffinger wrote: > >> Voting is not yet another mobbing tool for when you get tired of other > >> ways. > >> If you wish for an end to that feud, you can ask other developers to try >

Re: [FFmpeg-devel] [PATCH] avfilter/f_loop: do not loop if loop size is 0

2019-05-21 Thread Alexander Strasser
Hi! On 2019-05-20 20:51 +0200, Marton Balint wrote: > > On Mon, 20 May 2019, Gyan wrote: > > > On 20-05-2019 02:18 AM, Marton Balint wrote: > > > > > > On Sun, 19 May 2019, Paul B Mahol wrote: > > > > > > > On 5/19/19, Marton Balint wrote: > > > > > > > > > > On Sun, 19 May 2019, Paul B Mahol wro

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-06-02 Thread Alexander Strasser
Hi all, hi Avi! Sorry for the late reply. On 2019-05-07 12:22 +, avih wrote: > > patch1 (awk) configure: print_in_columns: replace pr with awk version: > > http://ffmpeg.org/pipermail/ffmpeg-devel/2019-May/243380.html > > patch2 (shell) configure: sort decoder/encoder/filter/... names in alp

[FFmpeg-devel] [PATCH] lavd/v4l2: produce a 0 byte packet when a dequeued buffer's size is unexpected

2019-06-05 Thread Alexander Strasser
error message, you will find quite a few instances of this problem. Some of them experienced on other devices. Probably fixes ticket #4795 Signed-off-by: Alexander Strasser --- This is exactly Stephan's patch except for the commit message. @Stephan: I hope you are OK with my wording in th

Re: [FFmpeg-devel] [PATCH V5 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

2019-06-24 Thread Alexander Strasser
On 2019-06-03 06:24 +, Guo, Yejun wrote: > > > > -Original Message- > > From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of > > avih > > Sent: Monday, June 03, 2019 6:29 AM > > To: FFmpeg development discussions and patches > > Subject: Re: [FFmpeg-devel] [PATCH V5

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: produce a 0 byte packet when a dequeued buffer's size is unexpected

2019-06-24 Thread Alexander Strasser
On 2019-06-05 22:04 +0200, Alexander Strasser wrote: > From: Stephan Hilb > > Behave like we do for V4L2_BUF_FLAG_ERROR, implemented in commit 28f20d2ff4 . > > For some devices (probably also related to the V4L driver implementation) > it happens that when invoking the ioctl D

Re: [FFmpeg-devel] [PATCH 1/2] configure: print_in_columns: Replace pr with awk

2019-06-24 Thread Alexander Strasser
On 2019-05-05 11:36 +0200, Alexander Strasser wrote: > > > Am 5. Mai 2019 03:53:20 MESZ schrieb "Guo, Yejun" : > > > > > >> -Original Message- > >> From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf > >Of > >&g

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: produce a 0 byte packet when a dequeued buffer's size is unexpected

2019-07-30 Thread Alexander Strasser
On 2019-06-25 00:08 +0200, Alexander Strasser wrote: > On 2019-06-05 22:04 +0200, Alexander Strasser wrote: > > From: Stephan Hilb > > > > Behave like we do for V4L2_BUF_FLAG_ERROR, implemented in commit 28f20d2ff4 > > . > > > > For some devices (

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

2019-08-06 Thread Alexander Strasser
Hi Marton! Not really sure if we need the API, but it definitely looks handy. Just not sure how often it will used and really result in clearer or shorter code. Not opposed to it though; no strong opinion on this one. Some comments follow inline. No in depth review, just what came to my mind whe

Re: [FFmpeg-devel] [PATCH] doc/developper: always use braces for statements

2019-08-06 Thread Alexander Strasser
On 2019-08-06 22:51 +0200, Nicolas George wrote: > Tomas Härdin (12019-08-06): > > I think they should. See > > Documenting the coding standard and changing it are different matters. I guess a change was intended, because it should be fairly obvious that in the current code base the described styl

Re: [FFmpeg-devel] libavcodec: add timer bitstream filter [v2]

2019-08-06 Thread Alexander Strasser
Hi Andreas! On 2019-08-05 17:25 +, Andreas Håkon wrote: > On Monday, 5 de August de 2019 17:31, Moritz Barsnick > wrote: > [...] > > > +static const AVClass timer_class = { > > > + .class_name = "timer", > > > > In about half of the existing BSFs, this would be called "timer_bsf". I > > d

Re: [FFmpeg-devel] [PATCH 1/3] avformat/avio: add avio_print_n_strings and avio_print

2019-08-07 Thread Alexander Strasser
On 2019-08-07 21:28 +0200, Marton Balint wrote: > > On Wed, 7 Aug 2019, Alexander Strasser wrote: > > > Hi Marton! > > > > Not really sure if we need the API, but it definitely looks > > handy. Just not sure how often it will used and really result > > in cle

Re: [FFmpeg-devel] libavcodec: add timeshift bitstream filter [v3]

2019-08-07 Thread Alexander Strasser
On 2019-08-07 15:51 +, Andreas Håkon wrote: > Hi, > > This new version changes the name of the filter, and implements all > suggestions. Sorry, I couldn't reply earlier. Thanks for renaming; "timeshift" sounds better to me, compared to the previous "timer". Other suggestions which I came up

Re: [FFmpeg-devel] libavcodec: add timeshift bitstream filter [v3]

2019-08-08 Thread Alexander Strasser
Hi Nicolas! On 2019-08-08 11:43 +0200, Nicolas George wrote: > Andreas Håkon (12019-08-08): > > > setpts or setts, similar to lavfi. > > > Then I proposse these two alternatives: > > > > 1) "editpts_bsf" > > 2) "setpts_bsf" > > > > The "_bsf" is not real, only to differenciate (here) from other fi

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

2020-06-19 Thread Alexander Strasser
On 2020-06-19 20:41 +0200, Carl Eugen Hoyos wrote: > Am Fr., 19. Juni 2020 um 20:33 Uhr schrieb Cristian Bicheru > : > > > > On Fri, Jun 19, 2020 at 12:53 PM Carl Eugen Hoyos > > wrote: > > > > > > Am Fr., 19. Juni 2020 um 18:42 Uhr schrieb : > > > > > > > > From: Cristian Bicheru > > > > > > >

Re: [FFmpeg-devel] [Patch] vf_edgedetect: properly implement double_threshold()

2020-06-19 Thread Alexander Strasser
Hi Valery! On 2020-06-19 17:15 +0200, Valery Kot wrote: > vf_edgedetect video filter implements Canny algorithm > (https://en.wikipedia.org/wiki/Canny_edge_detector) > > Important part of this algo is the double threshold step: pixels above > "high" threshold being kept, pixels below "low" thresho

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

2020-06-19 Thread Alexander Strasser
Am 20. Juni 2020 00:23:53 MESZ schrieb Hendrik Leppkes : >On Fri, Jun 19, 2020 at 9:58 PM Alexander Strasser >wrote: >> >> How do others think about adding support for more pixel formats? >> > >A new pixel format should present a clear improvement, a use-case you

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/af_ladspa: check another directory for plugins

2020-06-21 Thread Alexander Strasser
Hi Paul! On 2020-06-21 12:49 +, Paul B Mahol wrote: > ffmpeg | branch: master | Paul B Mahol | Sun Jun 21 > 14:46:29 2020 +0200| [842bc312ade8fab82465423b22c4fbe3bee63383] | committer: > Paul B Mahol > > avfilter/af_ladspa: check another directory for plugins > > > http://git.videolan.org/g

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/af_ladspa: check another directory for plugins

2020-06-21 Thread Alexander Strasser
On 2020-06-21 21:19 +0200, Alexander Strasser wrote: > On 2020-06-21 12:49 +, Paul B Mahol wrote: > > ffmpeg | branch: master | Paul B Mahol | Sun Jun 21 > > 14:46:29 2020 +0200| [842bc312ade8fab82465423b22c4fbe3bee63383] | > > committer: Paul B Mahol > > &g

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/aac_ac3_parser: improve the raw AAC file bit rate calculation

2020-06-27 Thread Alexander Strasser
On 2020-06-26 01:56 +, Jun Zhao wrote: > ffmpeg | branch: master | Jun Zhao | Sun May 17 > 12:10:05 2020 +0800| [60d79b1df9d4c6030010ccb0c134ede9e33158c2] | committer: > Jun Zhao > > lavc/aac_ac3_parser: improve the raw AAC file bit rate calculation > > Now we just use one ADTS raw frame to

Re: [FFmpeg-devel] [FFmpeg-cvslog] lavc/aac_ac3_parser: improve the raw AAC file bit rate calculation

2020-06-28 Thread Alexander Strasser
On 2020-06-28 21:10 +0800, myp...@gmail.com wrote: > On Sun, Jun 28, 2020 at 5:30 AM Alexander Strasser wrote: > > > > On 2020-06-26 01:56 +, Jun Zhao wrote: > > > ffmpeg | branch: master | Jun Zhao | Sun May 17 > > > 12:10:05 2020 +0800| [60d79b1

Re: [FFmpeg-devel] [PATCH v3 1/2] libavcodec/pgxdec: Add PGX decoder

2020-06-28 Thread Alexander Strasser
On 2020-06-28 22:44 +0200, Michael Niedermayer wrote: > On Sun, Jun 28, 2020 at 08:13:28PM +0530, gautamr...@gmail.com wrote: > > From: Gautam Ramakrishnan > > > > This patch adds a pgx decoder. > > --- > > Changelog | 1 + > > doc/general.texi| 2 + > > libavcodec/Makef

Re: [FFmpeg-devel] [PATCH] ffprobe: Add option to allow unknown format private AVOptions

2020-06-28 Thread Alexander Strasser
On 2020-06-28 22:23 +0200, Marton Balint wrote: > > > On Sun, 28 Jun 2020, Michael Niedermayer wrote: > > > On Sun, Jun 28, 2020 at 01:22:58PM +0100, Derek Buitenhuis wrote: > > > On 26/06/2020 14:49, Nicolas George wrote: > > > > Probably a good idea, but these explanation should probably go in >

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-10 Thread Alexander Strasser
On 2021-05-10 10:22 +0200, Anton Khirnov wrote: > Export them in UTC, not the local timezone. This way the output is > the same everywhere. The timezone information stored in the file is > still ignored, since there seems to be no simple way to export it > correctly. > > Format them according to I

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-11 Thread Alexander Strasser
On 2021-05-11 17:51 +0200, Anton Khirnov wrote: > Quoting Alexander Strasser (2021-05-10 15:35:02) > > On 2021-05-10 10:22 +0200, Anton Khirnov wrote: > > > Export them in UTC, not the local timezone. This way the output is > > > the same everywhere. The timezone inform

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-15 Thread Alexander Strasser
Hi Anton! On 2021-05-14 10:09 +0200, Anton Khirnov wrote: > Quoting Alexander Strasser (2021-05-12 01:04:28) > > > > If the timezone data of an AMF 0 date type is non-zero, include that > > information as UTC offset in hours and minutes. > > --- >

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: normalize exporting date metadata

2021-05-19 Thread Alexander Strasser
On 2021-05-16 21:18 +0200, Anton Khirnov wrote: > Quoting Alexander Strasser (2021-05-15 20:20:30) [...] > > > > Returning to the code I quoted before now and stating my > > understanding of if now. > > > > def write__AMF_date(time) > > write__UI8

[FFmpeg-devel] [PATCH] libavdevice/v4l2: Return EAGAIN when receiving a frame of unexpected size

2019-03-17 Thread Alexander Strasser
I had found that when capturing video for some hours from USB Camera-B4.09.24.1 (Manufacturer: OmniVision Technologies, Inc.), sometimes when invoking the ioctl DQBUF, the returned buffer is not filled with the size we expect for the raw video frame. Here are two examples for such occurrences:

Re: [FFmpeg-devel] [PATCH] libavdevice/v4l2: Return EAGAIN when receiving a frame of unexpected size

2019-03-17 Thread Alexander Strasser
On 2019-03-17 18:25 +0100, Nicolas George wrote: > Alexander Strasser (12019-03-17): > > I had found that when capturing video for some hours from > > USB Camera-B4.09.24.1 (Manufacturer: OmniVision Technologies, Inc.), > > sometimes when invoking the ioctl DQBUF, the r

Re: [FFmpeg-devel] [PATCH] libavdevice/v4l2: Return EAGAIN when receiving a frame of unexpected size

2019-03-18 Thread Alexander Strasser
Hi Nicolas! On 2019-03-18 00:24 +0100, Nicolas George wrote: > Alexander Strasser (12019-03-18): > > I tested the EAGAIN version and it worked. I also tested returning a > > ffmpeg.c uses the non-blocking mode. That would explain it. I now tested, the FFERROR_REDO approach, and I

[FFmpeg-devel] [PATCH] lavd/v4l2: Return FFERROR_REDO when receiving a frame of unexpected size

2019-03-20 Thread Alexander Strasser
I had found that when capturing video for some hours from USB Camera-B4.09.24.1 (Manufacturer: OmniVision Technologies, Inc.), sometimes when invoking the ioctl DQBUF, the returned buffer is not filled with the size we expect for the raw video frame. Here are two examples for such occurrences:

Re: [FFmpeg-devel] [PATCH] libavdevice/v4l2: Return EAGAIN when receiving a frame of unexpected size

2019-03-20 Thread Alexander Strasser
On 2019-03-18 23:26 +0100, Alexander Strasser wrote: > On 2019-03-18 00:24 +0100, Nicolas George wrote: > > Alexander Strasser (12019-03-18): > > > I tested the EAGAIN version and it worked. I also tested returning a > > > > ffmpeg.c uses the non-blocking mode. &g

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: Return FFERROR_REDO when receiving a frame of unexpected size

2019-03-20 Thread Alexander Strasser
Hi all! On 2019-03-20 19:56 +0100, Alexander Strasser wrote: > I had found that when capturing video for some hours from > USB Camera-B4.09.24.1 (Manufacturer: OmniVision Technologies, Inc.), > sometimes when invoking the ioctl DQBUF, the returned buffer is not > filled with the si

Re: [FFmpeg-devel] [PATCH] lavd/v4l2: Return FFERROR_REDO when receiving a frame of unexpected size

2019-03-21 Thread Alexander Strasser
Hi Marton, hi Nicolas! On 2019-03-21 10:07 +0100, Nicolas George wrote: > Marton Balint (12019-03-21): > > Maybe just set the packet corrupt flag and log an error, but return the > > partial buffer? > > I think it may be the best solution. But it would be interesting to know > how the tools react

Re: [FFmpeg-devel] [PATCH] avformat/dashdec: add dash demuxer base version

2017-03-19 Thread Alexander Strasser
On 2017-03-19 11:01 +0100, wm4 wrote: > On Sun, 19 Mar 2017 10:57:21 +0100 > Clément Bœsch wrote: > > > > > If you want to have DASH enabled by default, make sure to have a native > > XML parser, or discuss the policy about detecting all the external > > libraries within the configure. > > > >

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/h264: Check weight values to be within the specs limits.

2017-03-23 Thread Alexander Strasser
On 2017-03-22 09:06 -0400, Ronald S. Bultje wrote: > On Tue, Mar 21, 2017 at 9:59 PM, Michael Niedermayer > wrote: > > > @@ -59,6 +59,9 @@ int ff_h264_pred_weight_table(GetBitContext *gb, const > > SPS *sps, > > if (luma_weight_flag) { > > pwt->luma_weight[i][list][0

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-27 Thread Alexander Strasser
Hi Clément, I think your idea of introducing this function and the convenience macro is good. Below are some comments, feel free to ignore. On 2017-03-27 09:51 +0200, Clément Bœsch wrote: > --- > doc/APIchanges | 4 > libavutil/avutil.h | 14 ++ > libavutil/utils.c |

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Alexander Strasser
On 2017-03-28 10:19 +0200, Clément Bœsch wrote: > On Tue, Mar 28, 2017 at 12:47:39AM +0200, Alexander Strasser wrote: > [...] > > > +#define AV_FOURCC_MAX_STRING_SIZE 32 > > > > Should be fine, though I don't know how you came up with this max size in > >

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Alexander Strasser
On 2017-03-28 18:16 +0200, Clément Bœsch wrote: > On Tue, Mar 28, 2017 at 06:12:14PM +0200, Michael Niedermayer wrote: > > On Tue, Mar 28, 2017 at 10:19:46AM +0200, Clément Bœsch wrote: > > [...] > > > diff --git a/libavutil/utils.c b/libavutil/utils.c > > > index 36e4dd5fdb..29f2746338 100644 > >

Re: [FFmpeg-devel] [PATCH] pthread_frame: minor simplification to error handling

2017-03-29 Thread Alexander Strasser
Hi, I already saw this on -cvslog ml, so apparently I am too late... On 2017-03-29 12:19 +0200, wm4 wrote: > On Mon, 27 Mar 2017 14:28:17 +0200 > wm4 wrote: > > > Get rid of the "ret" variable, and always use err. Report the packet as > > consumed if err is unset. This should be equivalent to t

Re: [FFmpeg-devel] [PATCH] pthread_frame: minor simplification to error handling

2017-03-29 Thread Alexander Strasser
On 2017-03-30 00:00 +0200, wm4 wrote: > On Wed, 29 Mar 2017 22:25:48 +0200 > Alexander Strasser wrote: > > > On Mon, 27 Mar 2017 14:28:17 +0200 > > > wm4 wrote: > > > > > > > Get rid of the "ret" variable, and always use err. Report the

Re: [FFmpeg-devel] [PATCH] Ignore expired cookies

2017-03-30 Thread Alexander Strasser
Hi! On 2017-03-29 19:42 -0400, Micah Galizia wrote: > I'm going to have to submit a v2 for this patch (hopefully soon) -- > this version only accomplishes half the job: not sending expired > cookies. The change should also prevent storing them in the first > place. > > Regardless, thanks for your

Re: [FFmpeg-devel] [PATCH] Ignore expired cookies

2017-03-31 Thread Alexander Strasser
Hi Nicolas! On 2017-03-30 22:12 +0200, Nicolas George wrote: > Le decadi 10 germinal, an CCXXV, Alexander Strasser a écrit : > > If expiry is zero terminated and you are going through it one byte at a > > time, > > you could omit the strlen call and just check if

Re: [FFmpeg-devel] [PATCH] avutil/avstring: add av_strreplace API into avstring

2017-04-01 Thread Alexander Strasser
On 2017-04-01 13:12 +0200, wm4 wrote: > On Sat, 1 Apr 2017 10:01:15 +0200 > Nicolas George wrote: [...] > > Therefore, it was not acceptable as is and should not have been pushed. > > > > Pushing while leaving only half a day to answer this was UNACCEPTABLE. > > Do not ever do it again please. >

Re: [FFmpeg-devel] [PATCH] avformat: Add max_streams option

2016-11-18 Thread Alexander Strasser
Hi Michael! On 2016-11-18 17:03 +0100, Michael Niedermayer wrote: > This allows user apps to stop OOM due to excessive number of streams > TODO: bump & docs > > Signed-off-by: Michael Niedermayer > --- > libavformat/avformat.h | 7 +++ > libavformat/options_table.h | 1 + > libavformat

Re: [FFmpeg-devel] [PATCH] Added the interface for the Turing codec

2016-11-19 Thread Alexander Strasser
On 2016-11-19 14:29 +0100, Moritz Barsnick wrote: > On Fri, Nov 18, 2016 at 20:15:30 +0100, wm4 wrote: > > This lib has a really weird API... > > I can't judge the algorithms or the development direction and stuff - > I'm sure it's all state of the art. But the API sure gives me > goosebumps, and

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

2017-03-04 Thread Alexander Strasser
age.de/2017/en/ Currently registered as booth staff: Thilo Borgmann Carl Eugen Hoyos Thomas Volkert Alexander Strasser If you want to drop by or help out, it would be nice if you let us know about it here. I apologize for the late announcement this year. Best Regards, Alexander P.S. Slight

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

2017-03-04 Thread Alexander Strasser
Hi Paul! On 2017-03-04 12:33 +0100, Paul B Mahol wrote: > On 3/4/17, Alexander Strasser wrote: > > > > like in the previous years we applied for a booth at 'Chemnitzer Linux-Tage' > > in Germany. Fortunately we were accepted again! > > > > The event wi

[FFmpeg-devel] [PATCH 1/2] lavf: Remove unnecessary escaping of ' in string literals

2017-03-11 Thread Alexander Strasser
Signed-off-by: Alexander Strasser --- libavformat/avio.c | 4 ++-- libavformat/utils.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/avio.c b/libavformat/avio.c index 62233a6..9020aa9 100644 --- a/libavformat/avio.c +++ b/libavformat/avio.c @@ -179,12

[FFmpeg-devel] [PATCH 2/2] lavf: Be more explicit in logging white/black list matches

2017-03-11 Thread Alexander Strasser
The current form of the messages indicating matches in the white or black lists seems to be a bit too much relying on context. Make the messages more explicit. This also matches the way codec white list errors are reported. Signed-off-by: Alexander Strasser --- libavformat/avio.c | 4

[FFmpeg-devel] [PATCH 0/2] lavf: Error reporting when matching in black/white lists

2017-03-11 Thread Alexander Strasser
. Alexander Strasser (2): lavf: Remove unnecessary escaping of ' in string literals lavf: Be more explicit in logging white/black list matches libavformat/avio.c | 4 ++-- libavformat/utils.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) --

Re: [FFmpeg-devel] [PATCH 0/2] lavf: Error reporting when matching in black/white lists

2017-03-14 Thread Alexander Strasser
On 2017-03-11 18:54 +0100, Alexander Strasser wrote: > while investigating this issue filed at Debian: > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=857369 > > I found that the error messages can easily be misunderstood. > > Though in that particular case the

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix duration wrong when no pkt duration

2017-03-15 Thread Alexander Strasser
Hi! On 2017-03-13 17:12 +0800, Steven Liu wrote: > when cannot get pkt duration, hlsenc segments duration will > be set to 0, this patch can fix it. > > Signed-off-by: Steven Liu > --- > libavformat/hlsenc.c | 7 ++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/libavfo

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/vp56: Require 1 undamaged frame for 5 damaged frames for concealment to be used

2017-03-15 Thread Alexander Strasser
Hi! This is not a review, just a comment you can address in case you push this. On 2017-03-15 04:12 +0100, Michael Niedermayer wrote: > Fixes timeout with 847/clusterfuzz-testcase-5291877358108672 > Fixes timeout with 850/clusterfuzz-testcase-5721296509861888 > > This likely will need to be twe

Re: [FFmpeg-devel] [PATCH 01/16] doc/filters: document the unstability of the shorthand options notation.

2017-08-11 Thread Alexander Strasser
Hi all, sorry for jumping into the discussion late. I think it is very important though. Please pardon me if I missed anything from the previous discussions on this topic. On 2017-08-11 23:34 +0200, Nicolas George wrote: > Le quartidi 24 thermidor, an CCXXV, Clement Boesch a écrit : [...] > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/pngdec: Fix () placement

2017-08-23 Thread Alexander Strasser
On 2017-08-22 17:23 +, Michael Niedermayer wrote: > ffmpeg | branch: master | Michael Niedermayer | Tue > Aug 22 18:36:26 2017 +0200| [a2e444d5bb2e3115d3afcc0cca9d1506c90436a2] | > committer: Michael Niedermayer > > avcodec/pngdec: Fix () placement > > Signed-off-by: Michael Niedermayer >

Re: [FFmpeg-devel] [PATCH] swresample: Add AVFrame based API

2014-08-11 Thread Alexander Strasser
On 2014-08-11 21:42 +0200, Michael Niedermayer wrote: > On Mon, Aug 11, 2014 at 09:25:20AM +0100, Kieran Kunhya wrote: > > On 11 August 2014 00:49, Michael Niedermayer wrote: > > > From: Luca Barbato > > > > Clearly this is Luca's patch but rewritten by you for swr so you > > should say "Based o

[FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-19 Thread Alexander Strasser
Should fix CID1231988 (RESOURCE_LEAK) Signed-off-by: Alexander Strasser --- WARNING: Sorry, I only compile-tested so far. There is one remaining thing, I am not sure of: It looks like the variable feet could be uninitialized at the point av_freep is called. I think it cannot, but I had to

Re: [FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-19 Thread Alexander Strasser
On 2014-08-20 01:25 +0200, Alexander Strasser wrote: > Should fix CID1231988 (RESOURCE_LEAK) > > Signed-off-by: Alexander Strasser > --- > > WARNING: Sorry, I only compile-tested so far. > > There is one remaining thing, I am not sure of: > It looks like

[FFmpeg-devel] [PATCH v2] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-21 Thread Alexander Strasser
) Signed-off-by: Alexander Strasser --- I tested with valgrind and the mem leak goes away. About the rest it would be good if someone could have a 2nd look. And maybe Lukasz can comment if he finds some time because he maintains this code. libavformat/ftp.c | 7 ++- 1 file changed, 6

Re: [FFmpeg-devel] [PATCH] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-21 Thread Alexander Strasser
On 2014-08-20 06:27 +0200, Alexander Strasser wrote: > On 2014-08-20 01:25 +0200, Alexander Strasser wrote: > > Should fix CID1231988 (RESOURCE_LEAK) > > > > Signed-off-by: Alexander Strasser > > --- > > > > WARNING: Sorry, I only compile-tested so far.

Re: [FFmpeg-devel] [PATCH v2] libavformat/ftp: Do not leak memory in routine ftp_features

2014-08-25 Thread Alexander Strasser
On 2014-08-21 23:02 +0200, Alexander Strasser wrote: > Setting the pointer to NULL inside both ftp_send_command > and ftp_features is redundant. Generally always setting to > NULL in ftp_send_command seems safer, but throughout the file > that parameter was always passed initialized

Re: [FFmpeg-devel] [PATCH] Include content of the news article in the website RSS

2014-08-28 Thread Alexander Strasser
Hi! On 2014-05-23 00:28 +0200, Alexander Strasser wrote: > On 2014-05-11 01:37 +0200, Gerion Entrup wrote: > > Am Donnerstag 01 Mai 2014, 23:48:29 schrieb Alexander Strasser: > > > alternative. What do you and/or others think of it? > > > > > > Besides val

Re: [FFmpeg-devel] [libav-devel] Common mailing-list for API evolutions

2014-08-28 Thread Alexander Strasser
On 2014-08-28 18:58 +0200, Anton Khirnov wrote: > On Sun, 24 Aug 2014 00:28:56 +0200, =?utf-8?B?Q2zDqW1lbnQgQsWTc2No?= > wrote: > > Kieran suggested tonight on #ffmpeg-devel to have a common mailing-list > > between the two projects to start communicating again in sane terms. > > > > The proposi

Re: [FFmpeg-devel] [PATCH] Include content of the news article in the website RSS

2014-08-30 Thread Alexander Strasser
On 2014-08-28 07:23 -0700, Timothy Gu wrote: > On Thu, Aug 28, 2014 at 7:26 AM, Alexander Strasser wrote: > > Hi! > > > > On 2014-05-23 00:28 +0200, Alexander Strasser wrote: > >> On 2014-05-11 01:37 +0200, Gerion Entrup wrote: > >> > Am Donnerstag 01 Mai

Re: [FFmpeg-devel] [PATCH] avformat/img2dec: fix glob pattern detection.

2014-10-02 Thread Alexander Strasser
Hi Benoit, thank you for investigating this issue and sending a patch. On 2014-09-22 12:30 +0200, Benoit Fouet wrote: > The is_glob() function was not working with unescaped glob patterns, > which is the way only glob_sequence (which is deprecated) works. > Fixes ticket #3948 > --- > libavform

Re: [FFmpeg-devel] [PATCH] avformat/img2dec: fix glob pattern detection.

2014-10-02 Thread Alexander Strasser
On 2014-10-02 22:51 +0200, Alexander Strasser wrote: [...] > Maybe something like this patch would be acceptable > (WARNING: only lightly tested): > > diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c > index 16bd699..aa7c2f6 100644 > --- a/libavformat/img2dec.c &

[FFmpeg-devel] [PATCH] avformat/img2dec: Attempt to detect non-escaped glob patterns too (-pattern_type glob)

2014-10-04 Thread Alexander Strasser
Fixes ticket #3948 Based-on-patch-by: Michael Niedermayer Signed-off-by: Alexander Strasser --- - fixes the \0 typo in the string literal - cosmetic change to the score comment - remove the -pattern_type comment because as the options matching system works now it would not be the "

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

2014-10-05 Thread Alexander Strasser
Try to shortly summarize where we are now. Signed-off-by: Alexander Strasser --- This is all RFC as marked in the subject. Please give me some feed back! It would also be nice if someone could test the patch, I do not have the tools at hand ATM. src/index | 16 1

  1   2   3   >