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] swresample: Add AVFrame based API

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 michae...@gmx.at wrote: From: Luca Barbato lu_z...@gentoo.org Clearly this is Luca's patch but rewritten by you for swr so

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

Should fix CID1231988 (RESOURCE_LEAK) Signed-off-by: Alexander Strasser eclip...@gmx.net --- 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

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

) Signed-off-by: Alexander Strasser eclip...@gmx.net --- 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

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

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 eclip...@gmx.net --- WARNING: Sorry, I only compile-tested so far. There is one remaining thing, I

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

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 validation I couldn't yet properly test what feed

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

On 2014-08-28 18:58 +0200, Anton Khirnov wrote: On Sun, 24 Aug 2014 00:28:56 +0200, =?utf-8?B?Q2zDqW1lbnQgQsWTc2No?= u...@pkh.me 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

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

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 +++ b/libavformat/img2dec.c

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

Fixes ticket #3948 Based-on-patch-by: Michael Niedermayer michae...@gmx.at Signed-off-by: Alexander Strasser eclip...@gmx.net --- - 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

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

Try to shortly summarize where we are now. Signed-off-by: Alexander Strasser eclip...@gmx.net --- 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

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

On 2014-10-06 01:35 +, Carl Eugen Hoyos wrote: Timothy Gu timothygu99 at 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

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 pr...@xvid.org 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] 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 eclip...@gmx.net wrote: On 2014-10-09 08:13 +0200, Reimar Döffinger wrote: On 7 October 2014 08:41:09 CEST, Peter Ross pr...@xvid.org wrote: [...] +if ((ret = init_get_bits8(gb, avpkt

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 si...@gmx.net 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 geo...@nsup.org wrote: Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : This fixes the issue that a not set

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_ssim *

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

and 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 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 eclip...@gmx.net --- 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

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

/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, please tell

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 @@ a href=main.rssstrongi class=fa

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 but useless.

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 user passed

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

2001 Message-Id: 0af78adac09fb6b93a4cc39c71d6e562f9689153.1427915596.git.eclip...@gmx.net From: Alexander Strasser eclip...@gmx.net Date: Wed, 1 Apr 2015 21:08:42 +0200 Subject: [PATCH] WIP: lavf/zip: Guess zip file or use full path and option to select the file inside the archive Signed-off

[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

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 spare

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. Just

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 eclip...@gmx.net wrote: Signed-off-by: Alexander Strasser eclip...@gmx.net --- libavutil/parseutils.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil

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

Signed-off-by: Alexander Strasser eclip...@gmx.net --- 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

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

Signed-off-by: Alexander Strasser eclip...@gmx.net --- 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

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

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

. Alexander Am 10. November 2015 10:01:23 MEZ, schrieb Alexander Strasser <eclip...@gmx.net>: > > >Am 6. September 2014 19:23:25 MESZ, schrieb "Clément Bœsch" <u...@pkh.me>: >>On Sat, Sep 06, 2014 at 09:12:47AM -0800, Lou Logan wrote: >>> On 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 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] [PATCH] avformat: Add max_streams option

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 +++ >

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

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

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

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) { > >

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

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

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

On 2017-03-30 00:00 +0200, wm4 wrote: > On Wed, 29 Mar 2017 22:25:48 +0200 > Alexander Strasser <eclip...@gmx.net> wrote: > > > On Mon, 27 Mar 2017 14:28:17 +0200 > > > wm4 <nfx...@googlemail.com> wrote: > > > > > > > Get rid of

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

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

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

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 > > part

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

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 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

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] Ignore expired cookies

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] avformat/dashdec: add dash demuxer base version

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

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

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

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

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 c

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

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(-) >

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

7/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. Slightl

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

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

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

. 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(-) -- 2.7.4

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

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 <eclip...@gmx.net> --- libav

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

Signed-off-by: Alexander Strasser <eclip...@gmx.net> --- 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/libav

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

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

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

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

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 > >

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.

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

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

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

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

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 > +++ >

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] lavd/v4l2: Return FFERROR_REDO when receiving a frame of unexpected size

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

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

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

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

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

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

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

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

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] libavdevice/v4l2: Return EAGAIN when receiving a frame of unexpected size

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

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

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 V5 2/2] configure: replace 'pr' with printf since busybox does not support pr

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] &g

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

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 > --- > config

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

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

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

for the 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 the new

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

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

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

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

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] [DECISION] Ban Nicolas George from project

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

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

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

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

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

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..81e3776060 100

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

. 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(+), 3 deletions

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

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

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

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 1/2] configure: sort decoder/encoder/filter/... names in alphabet order

u 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 awk to implement bigger systems or as a general purpose p

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

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

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

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

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

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 V5 2/2] configure: replace 'pr' with printf since busybox does not support pr

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 > >

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

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

ncols -t +sort | tr '\n' ' ' | awk -v width="$ncols" ' +{ +num_cols = width / 24; num_rows = (NF + num_cols-1) / num_cols; + y = x = 1; +for (i = 1; i <= NF; i++) { +if (y > num_rows) { +y = 1; x++; + } +d[x

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

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

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

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

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] avfilter/af_astats: count number of NaNs/Infs/denormals for floating-point audio too

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

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

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

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

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

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

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

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]

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

  1   2   >