Re: [FFmpeg-devel] [PATCH] avformat/icoenc: Remove deprecated use of codec_name

2014-08-28 Thread Paul B Mahol
On 8/27/14, Michael Niedermayer michae...@gmx.at wrote: Signed-off-by: Michael Niedermayer michae...@gmx.at --- libavformat/icoenc.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/icoenc.c b/libavformat/icoenc.c index dcf9065..53d4420 100644 ---

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 02:15:45AM +0200, Michael Niedermayer wrote: On Wed, Aug 27, 2014 at 11:47:59PM +0200, Clément Bœsch wrote: On Wed, Aug 27, 2014 at 10:16:12PM +0200, Michael Niedermayer wrote: Hi all I was thinking about if we should put an automatic 50 USD bounty on every

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Derek Buitenhuis
On 8/28/2014 2:25 AM, James Almer wrote: wget is not available on a standard Msys installation. Probably the same with Cygwin. bc isn't standard either and FATE uses it. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Michael Niedermayer
On Thu, Aug 28, 2014 at 11:50:37AM +0200, Clément Bœsch wrote: On Thu, Aug 28, 2014 at 02:15:45AM +0200, Michael Niedermayer wrote: On Wed, Aug 27, 2014 at 11:47:59PM +0200, Clément Bœsch wrote: On Wed, Aug 27, 2014 at 10:16:12PM +0200, Michael Niedermayer wrote: Hi all I was

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Carl Eugen Hoyos
Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: But I'm fine with moving lena.pnm to the FATE samples, which are downloaded with 'make fate-rsync'. Attached patch removes tests/lena.pnm and adapts the tests, assuming lena.pnm would be added to the top-level of the SAMPLES

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at gmx.at writes: I was thinking about if we should put an automatic 50 USD bounty on every bug thats open since more than 12 month on trac, and limited to the first 20 claims so its limited to 1000 USD max expenses for us (for now) I believe this is a good

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Andreas Cadhalpun
On 28.08.2014 14:01, Carl Eugen Hoyos wrote: Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: But I'm fine with moving lena.pnm to the FATE samples, which are downloaded with 'make fate-rsync'. Attached patch removes tests/lena.pnm and adapts the tests, assuming lena.pnm would be

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Derek Buitenhuis
On 8/28/2014 12:28 PM, Andreas Cadhalpun wrote: ffmpeg -f data -i http://samples.ffmpeg.org/image-samples/lena.pnm -c copy -f data -map 0 -y lena.pnm From: https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-August/161881.html possible but would this make andreas / debian happy ? No.

Re: [FFmpeg-devel] [PATCH 2/3] build: rely on pkg-config for libx264 probing

2014-08-28 Thread Clément Bœsch
On Thu, Aug 21, 2014 at 02:34:42PM +0200, Clément Bœsch wrote: On Thu, Aug 21, 2014 at 02:24:40PM +0200, Clément Bœsch wrote: On Thu, Aug 21, 2014 at 12:16:35PM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: [...] Sorry, I seem to have lost track of what you

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Derek Buitenhuis
On 8/28/2014 1:42 PM, Andreas Cadhalpun wrote: That's clear. But it doesn't work without internet connection. So why not use fate-rsync, which is only run manually, when internet is available? Probably because it entails reworking the vsynth build system stuff quite a bit to placate a single

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/rtpdec_asf: fix compiler warning about const qualifier being discarded

2014-08-28 Thread Michael Niedermayer
On Wed, Aug 27, 2014 at 07:58:53PM -0700, Timothy Gu wrote: On Wed, Aug 27, 2014 at 7:28 PM, Michael Niedermayer michae...@gmx.at wrote: On Wed, Aug 27, 2014 at 06:17:36PM -0700, Timothy Gu wrote: Michael Niedermayer g...@videolan.org writes: ffmpeg | branch: master | Michael

Re: [FFmpeg-devel] [PATCH 2/3] build: rely on pkg-config for libx264 probing

2014-08-28 Thread Carl Eugen Hoyos
Clément Bœsch u at pkh.me writes: Did you try the patch with pkg-config only detection? And try to add --pkg-config=true to trick the detect function. So, I did try myself Sorry, I was away and still didn't test all new tickets. [~/src/ffmpeg]☭ dash ./configure --disable-everything

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Carl Eugen Hoyos
Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: But I'm not sure what copying it to tests/data would be good for? So people can still run the limited tests without downloading the whole fate suite. Carl Eugen ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] Adds support for constant quality mode in VP9.

2014-08-28 Thread Michael Niedermayer
On Wed, Aug 27, 2014 at 07:08:00PM -0700, James Zern wrote: On Wed, Aug 27, 2014 at 1:04 PM, Deb Mukherjee debar...@google.com wrote: Changes in the parameter mapping for libvpx to support the constant quality mode in VP9. The assumption in the patch is that if crf is provided but bitrate

Re: [FFmpeg-devel] [PATCH 2/3] build: rely on pkg-config for libx264 probing

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 01:18:55PM +, Carl Eugen Hoyos wrote: Clément Bœsch u at pkh.me writes: Did you try the patch with pkg-config only detection? And try to add --pkg-config=true to trick the detect function. So, I did try myself Sorry, I was away and still didn't test

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Andreas Cadhalpun
On 28.08.2014 15:16, Carl Eugen Hoyos wrote: Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: But I'm not sure what copying it to tests/data would be good for? So people can still run the limited tests without downloading the whole fate suite. With my patch that would still be

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

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

2014-08-28 Thread Timothy Gu
On Thu, Aug 28, 2014 at 7:26 AM, Alexander Strasser eclip...@gmx.net 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 2014, 23:48:29 schrieb Alexander Strasser: alternative. What do you and/or others think

[FFmpeg-devel] [PATCH] lavu/avstring: check for overlong encodings

2014-08-28 Thread Stefano Sabatini
Fix reopened trac ticket #1163. --- libavutil/avstring.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavutil/avstring.c b/libavutil/avstring.c index a63fb84..df27d5e 100644 --- a/libavutil/avstring.c +++ b/libavutil/avstring.c @@ -331,7 +331,10 @@ int

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 02:12:14PM +0200, Andreas Cadhalpun wrote: On 28.08.2014 14:01, Carl Eugen Hoyos wrote: Andreas Cadhalpun andreas.cadhalpun at googlemail.com writes: But I'm fine with moving lena.pnm to the FATE samples, which are downloaded with 'make fate-rsync'. Attached patch

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

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 06:58:57PM +0200, Anton Khirnov wrote: [...] Second, how do you propose this arrangement will actually function? As you probably know, I see many of the API additions done in your project as ugly hacks, and would be strongly opposed to having them in our tree in their

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Andreas Cadhalpun
On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm File size and dimensions are the same, and we can of course keep the file name since the person looks like Lena a bit. Would that one be OK? According to http://goo.gl/0K3qyP it should be OK as long as the

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Nicolas George
On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm According to http://goo.gl/0K3qyP it should be OK as long as the attribution is kept, which can be done in the commit message. Clément, I believe this URL shortener will please you very much:

[FFmpeg-devel] [PATCH] add silenceremove filter

2014-08-28 Thread Paul B Mahol
Signed-off-by: Paul B Mahol one...@gmail.com --- doc/filters.texi | 55 + libavfilter/Makefile | 1 + libavfilter/af_silenceremove.c | 461 + libavfilter/allfilters.c | 1 + 4 files changed, 518 insertions(+) create

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Michael Niedermayer
On Wed, Aug 27, 2014 at 04:27:15PM -0800, Lou Logan wrote: On Wed, 27 Aug 2014 22:16:12 +0200, Michael Niedermayer wrote: Hi all I was thinking about if we should put an automatic 50 USD bounty on every bug thats open since more than 12 month on trac, and limited to the first 20

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Andreas Cadhalpun
On 28.08.2014 19:37, Nicolas George wrote: On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm According to http://goo.gl/0K3qyP it should be OK as long as the attribution is kept, which can be done in the commit message. Clément, I believe this URL shortener will

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Michael Niedermayer
On Thu, Aug 28, 2014 at 07:47:11PM +0200, Andreas Cadhalpun wrote: On 28.08.2014 19:37, Nicolas George wrote: On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm According to http://goo.gl/0K3qyP it should be OK as long as the attribution is kept, which can be done

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Andreas Cadhalpun
On 28.08.2014 19:49, Reimar Döffinger wrote: I completely lost track of the discussion, but this all so shouts bikeshed and overkill. If I may suggest another colour for the shed: Put some random, freely licensed image where the Lena image was, put the Lena one in the sample repository and run

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Nicolas George
Le primidi 11 fructidor, an CCXXII, Andreas Cadhalpun a écrit : I like this colour of the shed. ;) * move lena.pnm to the FATE samples * add e.g. mona_lisa.pnm and duplicate the vsynth2 tests (as vsynth4?) Has someone a problem with this solution? I find it very good. Just a small

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 07:28:07PM +0200, Andreas Cadhalpun wrote: On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm File size and dimensions are the same, and we can of course keep the file name since the person looks like Lena a bit. Would that one be OK?

Re: [FFmpeg-devel] [PATCH] mark null encoder as ts-nonstrict , avoids error messages for null output

2014-08-28 Thread Rodger Combs
On Thu, Aug 21, 2014 at 02:00:22PM +0200, Hendrik Leppkes wrote: On Thu, Aug 21, 2014 at 1:38 PM, compn tempn at mi.rr.com http://ffmpeg.org/mailman/listinfo/ffmpeg-devel wrote: https://gist.github.com/anonymous/0e26f490ec13d67996fd

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?= 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] [RFC] Bug bounties

2014-08-28 Thread Lou Logan
On Thu, 28 Aug 2014 19:41:30 +0200, Michael Niedermayer wrote: this looks pretty cool i am not sure how successfull this would be for us but i definitly support the idea of trying this ... I was thinking it would be more useful as a tool for users and downstreams to offer bounties (but not

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Anshul
On 08/29/2014 01:34 AM, Lou Logan wrote: On Thu, 28 Aug 2014 19:41:30 +0200, Michael Niedermayer wrote: this looks pretty cool i am not sure how successfull this would be for us but i definitly support the idea of trying this ... I was thinking it would be more useful as a tool for users and

Re: [FFmpeg-devel] [RFC] Bug bounties

2014-08-28 Thread Michael Niedermayer
On Thu, Aug 28, 2014 at 12:04:21PM -0800, Lou Logan wrote: On Thu, 28 Aug 2014 19:41:30 +0200, Michael Niedermayer wrote: this looks pretty cool i am not sure how successfull this would be for us but i definitly support the idea of trying this ... I was thinking it would be more useful

[FFmpeg-devel] [PATCH 2/2] Add missing const all over the place.

2014-08-28 Thread Reimar Döffinger
There is a risk that one in the code that is not always compiled might break things. Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- ffserver.c | 2 +- libavcodec/aaccoder.c | 2 +- libavcodec/aic.c| 2 +- libavcodec/atrac3plus.c

[FFmpeg-devel] [PATCH 1/2] patcheck: check for pointer arrays that are not const.

2014-08-28 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- tools/patcheck | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/patcheck b/tools/patcheck index 3342c6a..cbdbf8d 100755 --- a/tools/patcheck +++ b/tools/patcheck @@ -42,6 +42,7 @@ hiegrep2(){ cat $TMP } +hiegrep

Re: [FFmpeg-devel] [PATCH 2/2] Add missing const all over the place.

2014-08-28 Thread Timothy Gu
On Aug 28, 2014 3:46 PM, Reimar Döffinger reimar.doeffin...@gmx.de wrote: There is a risk that one in the code that is not always compiled might break things. What do you mean by 'one in the code'? [...] Timothy ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [FFmpeg-cvslog] avformat/rtpdec_asf: fix compiler warning about const qualifier being discarded

2014-08-28 Thread Timothy Gu
On Aug 28, 2014 6:14 AM, Michael Niedermayer michae...@gmx.at wrote: buf is assigned to AVIOContext.buffer, which is written to by some functions using AVIOContext Doesn't const only apply to this particular function? In other words, doesn't const only guarantee that it is not changed inside

Re: [FFmpeg-devel] [PATCH 2/2] Add missing const all over the place.

2014-08-28 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 12:45:34AM +0200, Reimar Döffinger wrote: There is a risk that one in the code that is not always compiled might break things. builds fine here [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in

Re: [FFmpeg-devel] [PATCH 1/2] patcheck: check for pointer arrays that are not const.

2014-08-28 Thread Michael Niedermayer
On Fri, Aug 29, 2014 at 12:45:33AM +0200, Reimar Döffinger wrote: Signed-off-by: Reimar Döffinger reimar.doeffin...@gmx.de --- tools/patcheck | 1 + 1 file changed, 1 insertion(+) should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Awnsering

Re: [FFmpeg-devel] [PATCH] doc/demuxers: document gif demuxer

2014-08-28 Thread Timothy Gu
On Aug 28, 2014 5:27 PM, Lou Logan l...@lrcd.com wrote: Signed-off-by: Lou Logan l...@lrcd.com --- doc/demuxers.texi | 26 ++ 1 file changed, 26 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index d51b9d0..7b96d85 100644 --- a/doc/demuxers.texi

Re: [FFmpeg-devel] [PATCH] tests: Download lena.pnm if its missing from the source tree

2014-08-28 Thread Michael Niedermayer
On Thu, Aug 28, 2014 at 08:55:40PM +0200, Clément Bœsch wrote: On Thu, Aug 28, 2014 at 07:28:07PM +0200, Andreas Cadhalpun wrote: On 28.08.2014 17:53, Clément Bœsch wrote: ffplay http://lucy.pkh.me/lena.pnm File size and dimensions are the same, and we can of course keep the file name

Re: [FFmpeg-devel] [PATCH 2/2] fate: Add basic tests for WebM Dash Manifest

2014-08-28 Thread Michael Niedermayer
On Mon, Aug 25, 2014 at 03:03:45PM -0700, Vignesh Venkatasubramanian wrote: Add fate tests that test out the functionality of WebM DASH Manifest XML generation. This patch contains the vpx.mak file changes and the reference gold XML files. --- tests/fate/vpx.mak

Re: [FFmpeg-devel] [FFmpeg-cvslog] fate: Add basic tests for WebM Dash Manifest

2014-08-28 Thread James Almer
diff --git a/tests/fate/vpx.mak b/tests/fate/vpx.mak index 54be950..77c7c76 100644 --- a/tests/fate/vpx.mak +++ b/tests/fate/vpx.mak @@ -28,6 +28,15 @@ fate-vp6f: CMD = framecrc -flags +bitexact -i $(TARGET_SAMPLES)/flash-vp6/clip10 FATE_VP8-$(call DEMDEC, FLV, VP8) += fate-vp8-alpha

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

2014-08-28 Thread Clément Bœsch
On Fri, Aug 29, 2014 at 01:00:45AM +0200, Diego Biurrun wrote: On Sun, Aug 24, 2014 at 12:28:56AM +0200, Clément Bœsch 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 proposition

Re: [FFmpeg-devel] [PATCH] doc/demuxers: document gif demuxer

2014-08-28 Thread Clément Bœsch
On Thu, Aug 28, 2014 at 04:27:08PM -0800, Lou Logan wrote: Signed-off-by: Lou Logan l...@lrcd.com --- doc/demuxers.texi | 26 ++ 1 file changed, 26 insertions(+) diff --git a/doc/demuxers.texi b/doc/demuxers.texi index d51b9d0..7b96d85 100644 ---

Re: [FFmpeg-devel] [PATCH] doc/demuxers: document gif demuxer

2014-08-28 Thread Clément Bœsch
On Fri, Aug 29, 2014 at 07:47:41AM +0200, Clément Bœsch wrote: On Thu, Aug 28, 2014 at 04:27:08PM -0800, Lou Logan wrote: Signed-off-by: Lou Logan l...@lrcd.com --- doc/demuxers.texi | 26 ++ 1 file changed, 26 insertions(+) diff --git a/doc/demuxers.texi