[FFmpeg-devel] [PATCH] avcodec/libdcadec: honor AVCodecContext bitexact flag

2015-04-08 Thread James Almer
Signed-off-by: James Almer --- libavcodec/libdcadec.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/libdcadec.c b/libavcodec/libdcadec.c index 3ac4bcb..9ea61fa 100644 --- a/libavcodec/libdcadec.c +++ b/libavcodec/libdcadec.c @@ -167,8 +167,13 @@ static av_co

[FFmpeg-devel] Does the H.263 in FFmpeg support two pass encoding

2015-04-08 Thread Ramprasad N
Hi, The second pass shows error (IO error: Error number 0 occurred) but the file gets generated and looks fine, but I don't know whether it actually used the 1st pass. C:\Users\ramprasad>ffmpeg.exe -i foreman_qcif.y4m -b:v 128k -pass 1 -vcodec h263 -f 3gp -y NUL ffmpeg version N-69422-gf5722ba Cop

Re: [FFmpeg-devel] [PATCH] vda: Cast to not discard constant qualifier and silence compiler warning

2015-04-08 Thread Michael Niedermayer
On Thu, Apr 09, 2015 at 03:40:31AM +0530, Himangi Saraogi wrote: > --- > ffmpeg_vda.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have often repented speaking, but never of holding my

[FFmpeg-devel] [PATCH] ffmdec: Check return value of ffm_append_recommended_configuration

2015-04-08 Thread Himangi Saraogi
--- libavformat/ffmdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/ffmdec.c b/libavformat/ffmdec.c index 33bbde0..7c0f451 100644 --- a/libavformat/ffmdec.c +++ b/libavformat/ffmdec.c @@ -438,7 +438,8 @@ static int ffm2_read_header(AVFormatContext *s)

[FFmpeg-devel] [PATCH] vda: Cast to not discard constant qualifier and silence compiler warning

2015-04-08 Thread Himangi Saraogi
--- ffmpeg_vda.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffmpeg_vda.c b/ffmpeg_vda.c index b9f0975..d15648d 100644 --- a/ffmpeg_vda.c +++ b/ffmpeg_vda.c @@ -73,7 +73,7 @@ static int vda_retrieve_data(AVCodecContext *s, AVFrame *frame) } av_image_copy(vda->t

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Lukasz Marek
W dniu środa, 8 kwietnia 2015 Gilles Chanteperdrix < gilles.chanteperd...@xenomai.org> And forgot to add my apologies for these bad patches and the noise > that followed. > No need for that. Your demuxer seems to use public API only so you can try to move this implementation to your project and

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 10:37:13PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 04:31:34PM -0400, compn wrote: > > On Wed, 8 Apr 2015 22:26:23 +0200 > > Gilles Chanteperdrix wrote: > > > > > ready to forget all about these patches. > > > > thanks for the patches, even if some peo

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 04:31:34PM -0400, compn wrote: > On Wed, 8 Apr 2015 22:26:23 +0200 > Gilles Chanteperdrix wrote: > > > ready to forget all about these patches. > > thanks for the patches, even if some people do not like them. > > please dont be discouraged by strongly worded mails. > >

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread compn
On Wed, 8 Apr 2015 22:26:23 +0200 Gilles Chanteperdrix wrote: > ready to forget all about these patches. thanks for the patches, even if some people do not like them. please dont be discouraged by strongly worded mails. its a long term bikeshed that you have stepped in. the developers are most

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 10:08:25PM +0200, wm4 wrote: > On Wed, 08 Apr 2015 20:52:12 +0100 > Derek Buitenhuis wrote: > > > On 4/8/2015 6:16 PM, Gilles Chanteperdrix wrote: > > > Signed-off-by: Gilles Chanteperdrix > > > --- > > > libavformat/Makefile | 1 + > > > libavformat/allformats.c |

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 22:16:22 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 09:59:17PM +0200, Nicolas George wrote: > > Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > > > As I explained, I did it this way for portability. system is ANSI > > > not POSIX, redirectio

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Nicolas George
Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > So, when you just need to run exec, it is stupid Exactly what I wrote: not your problem / premature optimization. > I do not follow you here. youtube-dl -J has a uselessly verbose > output, much harder to parse than youtube-dl -

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 08:52:12PM +0100, Derek Buitenhuis wrote: > On 4/8/2015 6:16 PM, Gilles Chanteperdrix wrote: > > Signed-off-by: Gilles Chanteperdrix > > --- > > libavformat/Makefile | 1 + > > libavformat/allformats.c | 1 + > > libavformat/youtubedl.c | 221 > >

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 09:59:17PM +0200, Nicolas George wrote: > Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > > As I explained, I did it this way for portability. system is ANSI > > not POSIX, redirections work with windows shell too, and av_tempfile > > is implemented by ff

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread wm4
On Wed, 08 Apr 2015 20:52:12 +0100 Derek Buitenhuis wrote: > On 4/8/2015 6:16 PM, Gilles Chanteperdrix wrote: > > Signed-off-by: Gilles Chanteperdrix > > --- > > libavformat/Makefile | 1 + > > libavformat/allformats.c | 1 + > > libavformat/youtubedl.c | 221 > > +

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/x86/Makefile: Make building and linking of emms.c conditional

2015-04-08 Thread James Almer
On 08/04/15 4:29 PM, Ronald S. Bultje wrote: > Yeah I didn't test it. Anyway, the goal is indeed to remove 200 bytes. I > don't care about the 200 bytes. I care that a vp9-decoder-only build has > about 100 source files, most of which are obviously not needed. I care > about the sum of the parts, n

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Nicolas George
Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > As I explained, I did it this way for portability. system is ANSI > not POSIX, redirections work with windows shell too, and av_tempfile > is implemented by ffmpeg, so I expect it to be portable. Implementing pipe reading for othe

Re: [FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 09:46:30PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 09:13:11PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 08, 2015 at 07:16:36PM +0200, Gilles Chanteperdrix wrote: > > > avoid calling cleanup functions on uninitialized variables > > > > > > Signed

Re: [FFmpeg-devel] [PATCH] lavf/segment: provide a virtual AVIOContext representing all the segments

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 10:52:50AM -0500, Rodger Combs wrote: > > > On Mar 30, 2015, at 21:23, Rodger Combs wrote: > > > > This needs a fair bit of testing and review before merge. [...] > > Bump. maybe stefano can test and review? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BAD

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Derek Buitenhuis
On 4/8/2015 6:16 PM, Gilles Chanteperdrix wrote: > Signed-off-by: Gilles Chanteperdrix > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/youtubedl.c | 221 > +++ > 3 files changed, 223 insertions(+) > create

Re: [FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 09:13:11PM +0200, Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 07:16:36PM +0200, Gilles Chanteperdrix wrote: > > avoid calling cleanup functions on uninitialized variables > > > > Signed-off-by: Gilles Chanteperdrix > > --- > > libavformat/libquvi.c | 41 +

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 09:07:06PM +0200, Nicolas George wrote: > Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > > Signed-off-by: Gilles Chanteperdrix > > --- > > libavformat/Makefile | 1 + > > libavformat/allformats.c | 1 + > > libavformat/youtubedl.c | 221 > > +

Re: [FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 09:22:06PM +0200, Nicolas George wrote: > Le nonidi 19 germinal, an CCXXIII, Michael Niedermayer a écrit : > > > + av_log(s, AV_LOG_ERROR, "%s\n", quvi_strerror(q, rc)); > > is quvi_strerror() thread safe ? > > Looking at the code, it is thread-safe as long as the q argumen

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/x86/Makefile: Make building and linking of emms.c conditional

2015-04-08 Thread Ronald S. Bultje
Hi, On Wed, Apr 8, 2015 at 1:36 PM, James Almer wrote: > > ffmpeg | branch: master | Ronald S. Bultje | > Wed Apr 8 17:17:24 2015 +0200| [b926f02e81165378e615ca619f1283358dac26e4] > | committer: Michael Niedermayer > > > > avutil/x86/Makefile: Make building and linking of emms.c conditional >

Re: [FFmpeg-devel] [PATCH 2/2] lavf/assenc: handle extra sections after Events

2015-04-08 Thread Michael Niedermayer
On Sat, Apr 04, 2015 at 05:31:22AM -0500, Rodger Combs wrote: > --- > libavformat/assenc.c | 28 ++-- > 1 file changed, 26 insertions(+), 2 deletions(-) do you have a testcase for this which you can share ? [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF13

Re: [FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Nicolas George
Le nonidi 19 germinal, an CCXXIII, Michael Niedermayer a écrit : > > + av_log(s, AV_LOG_ERROR, "%s\n", quvi_strerror(q, rc)); > is quvi_strerror() thread safe ? Looking at the code, it is thread-safe as long as the q argument is not used concurrently. Regards, -- Nicolas George signature.

Re: [FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 07:16:36PM +0200, Gilles Chanteperdrix wrote: > avoid calling cleanup functions on uninitialized variables > > Signed-off-by: Gilles Chanteperdrix > --- > libavformat/libquvi.c | 41 + > 1 file changed, 25 insertions(+), 16 deletion

Re: [FFmpeg-devel] [PATCH] microdvd: do not export framerate hint as subtitle packet

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 07:55:04PM +0200, wm4 wrote: > MicroDVD has a "hack" for specifying the video framerate the subtitle > was authored against. The demuxer reads this hint correctly, but didn't > skip it correctly. > > This was not noticed, because the exported packet has its duration set > t

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Nicolas George
Le nonidi 19 germinal, an CCXXIII, Gilles Chanteperdrix a écrit : > Signed-off-by: Gilles Chanteperdrix > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/youtubedl.c | 221 > +++ > 3 files changed, 223 inserti

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/x86/Makefile: Make building and linking of emms.c conditional

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 02:36:48PM -0300, James Almer wrote: > > ffmpeg | branch: master | Ronald S. Bultje | Wed > > Apr 8 17:17:24 2015 +0200| [b926f02e81165378e615ca619f1283358dac26e4] | > > committer: Michael Niedermayer > > > > avutil/x86/Makefile: Make building and linking of emms.c condi

Re: [FFmpeg-devel] Usecases of the AVIODir* API

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 07:58:19PM +0200, Mariusz Szczepańczyk wrote: > On Wed, Apr 8, 2015 at 6:33 PM, Michael Niedermayer > wrote: > > > Hi > > > > Some people on IRC asked about the uses of the AVIODir* API and it > > seems several developers do not know what the API is usefull for. > > > > I

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 02:08:21PM -0400, compn wrote: > On Wed, 8 Apr 2015 19:51:12 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 07:44:13PM +0200, wm4 wrote: > > > On Wed, 8 Apr 2015 19:39:00 +0200 > > > Gilles Chanteperdrix wrote: > > > > > > > On Wed, Apr 08, 2015 at 07:

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread compn
On Wed, 8 Apr 2015 19:51:12 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 07:44:13PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 19:39:00 +0200 > > Gilles Chanteperdrix wrote: > > > > > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > > > > +snprintf(buffer, sizeof(buffer

Re: [FFmpeg-devel] Usecases of the AVIODir* API

2015-04-08 Thread Mariusz Szczepańczyk
On Wed, Apr 8, 2015 at 6:33 PM, Michael Niedermayer wrote: > Hi > > Some people on IRC asked about the uses of the AVIODir* API and it > seems several developers do not know what the API is usefull for. > > I think this should be documented better > > also what can it be usefull for, random examp

[FFmpeg-devel] [PATCH] microdvd: do not export framerate hint as subtitle packet

2015-04-08 Thread wm4
MicroDVD has a "hack" for specifying the video framerate the subtitle was authored against. The demuxer reads this hint correctly, but didn't skip it correctly. This was not noticed, because the exported packet has its duration set to 0, making it invisible (depending on the API user's rendering l

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:44:13PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 19:39:00 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > > > +snprintf(buffer, sizeof(buffer), "youtube-dl -f %s -g '%s'", > > > > + yc->format, s->filename)

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:30:48PM +0200, Hendrik Leppkes wrote: > On Wed, Apr 8, 2015 at 7:27 PM, Gilles Chanteperdrix > wrote: > > > >> Nice security hole. > > > > how is that ? I do not see any buffer overflow possible. > > > > Executing a command with system() is very unsafe. Indeed, never t

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 19:39:00 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > > +snprintf(buffer, sizeof(buffer), "youtube-dl -f %s -g '%s'", > > > + yc->format, s->filename); > > Ok, missing single quotes here around the format. > Doesn't he

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:30:12PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 19:27:55 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > > On Wed, 8 Apr 2015 19:16:38 +0200 > > > Gilles Chanteperdrix wrote: > > > > > > > Signed-off-by: Gilles Chante

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > +snprintf(buffer, sizeof(buffer), "youtube-dl -f %s -g '%s'", > > + yc->format, s->filename); Ok, missing single quotes here around the format. -- Gilles. _

Re: [FFmpeg-devel] [FFmpeg-cvslog] avutil/x86/Makefile: Make building and linking of emms.c conditional

2015-04-08 Thread James Almer
> ffmpeg | branch: master | Ronald S. Bultje | Wed Apr > 8 17:17:24 2015 +0200| [b926f02e81165378e615ca619f1283358dac26e4] | > committer: Michael Niedermayer > > avutil/x86/Makefile: Make building and linking of emms.c conditional > > Signed-off-by: Michael Niedermayer > > > http://git.videola

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:30:12PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 19:27:55 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > > On Wed, 8 Apr 2015 19:16:38 +0200 > > > Gilles Chanteperdrix wrote: > > > > > > > Signed-off-by: Gilles Chante

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Hendrik Leppkes
On Wed, Apr 8, 2015 at 7:27 PM, Gilles Chanteperdrix wrote: > >> Nice security hole. > > how is that ? I do not see any buffer overflow possible. > Executing a command with system() is very unsafe. - Hendrik ___ ffmpeg-devel mailing list ffmpeg-devel@f

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 19:27:55 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 19:16:38 +0200 > > Gilles Chanteperdrix wrote: > > > > > Signed-off-by: Gilles Chanteperdrix > > > --- > > > libavformat/Makefile | 1 + > > > lib

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:24:27PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 19:16:38 +0200 > Gilles Chanteperdrix wrote: > > > Signed-off-by: Gilles Chanteperdrix > > --- > > libavformat/Makefile | 1 + > > libavformat/allformats.c | 1 + > > libavformat/youtubedl.c | 221 > > +

Re: [FFmpeg-devel] youtube patches

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 07:15:51PM +0200, Gilles Chanteperdrix wrote: > Enough talking, here are some patches, and I will let the > maintainers decide. > > On the libquvi side, the first patch cleans up the error handling in > the libquvi_read_header function to avoid calling libquvi cleanup > fu

Re: [FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 19:16:38 +0200 Gilles Chanteperdrix wrote: > Signed-off-by: Gilles Chanteperdrix > --- > libavformat/Makefile | 1 + > libavformat/allformats.c | 1 + > libavformat/youtubedl.c | 221 > +++ > 3 files changed, 223 insertio

Re: [FFmpeg-devel] Areas needing work

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 10:33:56PM +0530, AB wrote: > > > On Monday 06 April 2015 09:13 AM, Michael Niedermayer wrote: > > Hi all > > > > There are various areas in the project that are not maintained as well as > > they > > should be, heres a certainly not comlpete list, dont hesitate to reply

[FFmpeg-devel] [PATCH 3/3] avformat: add youtube-dl based demuxer

2015-04-08 Thread Gilles Chanteperdrix
Signed-off-by: Gilles Chanteperdrix --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/youtubedl.c | 221 +++ 3 files changed, 223 insertions(+) create mode 100644 libavformat/youtubedl.c diff --git a/libavformat/Make

[FFmpeg-devel] [PATCH 1/3] avformat/libquvi: fix error handling

2015-04-08 Thread Gilles Chanteperdrix
avoid calling cleanup functions on uninitialized variables Signed-off-by: Gilles Chanteperdrix --- libavformat/libquvi.c | 41 + 1 file changed, 25 insertions(+), 16 deletions(-) diff --git a/libavformat/libquvi.c b/libavformat/libquvi.c index 7c5f7a2..e8

[FFmpeg-devel] [PATCH 2/3] avformat/libquvi: copy inner format context fields to outer format context

2015-04-08 Thread Gilles Chanteperdrix
Signed-off-by: Gilles Chanteperdrix --- libavformat/libquvi.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavformat/libquvi.c b/libavformat/libquvi.c index e838b34..8673cb4 100644 --- a/libavformat/libquvi.c +++ b/libavformat/libquvi.c @@ -96,6 +96,10 @@ static int libquvi_read_h

[FFmpeg-devel] youtube patches

2015-04-08 Thread Gilles Chanteperdrix
Enough talking, here are some patches, and I will let the maintainers decide. On the libquvi side, the first patch cleans up the error handling in the libquvi_read_header function to avoid calling libquvi cleanup functions on uninitialized variables. The second patch adds a call to avformat_find_

Re: [FFmpeg-devel] Areas needing work

2015-04-08 Thread Timothy Gu
On Wed, Apr 8, 2015 at 10:04 AM AB wrote: > > * fate clients > > we lack a armcc test, a real android test > > someone should take over haiku and sunos/solaris fate clients, my > haiku VM just > > keeps crashing > > > > I have some spare capacity, I'll try setting up a Haiku VM this weekend

Re: [FFmpeg-devel] Areas needing work

2015-04-08 Thread AB
On Monday 06 April 2015 09:13 AM, Michael Niedermayer wrote: > Hi all > > There are various areas in the project that are not maintained as well as they > should be, heres a certainly not comlpete list, dont hesitate to reply and > add to it. > If someone wants to pick something up and contribu

Re: [FFmpeg-devel] [PATCH] microdvd: do not export framerate hint as subtitle packet

2015-04-08 Thread Michael Niedermayer
On Tue, Apr 07, 2015 at 08:53:12PM +0200, wm4 wrote: > MicroDVD has a "hack" for specifying the video framerate the subtitle > was authored against. The demuxer reads this hint correctly, but didn't > skip it correctly. > > This was not noticed, because the exported packet has its duration set > t

[FFmpeg-devel] Usecases of the AVIODir* API

2015-04-08 Thread Michael Niedermayer
Hi Some people on IRC asked about the uses of the AVIODir* API and it seems several developers do not know what the API is usefull for. I think this should be documented better also what can it be usefull for, random example: A "file open" dialog in a player. Without such API in a ffmpeg lib pla

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Lukasz Marek
On 8 April 2015 at 18:07, Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 06:02:16PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 17:52:18 +0200 > > Michael Niedermayer wrote: > > > > > On Wed, Apr 08, 2015 at 05:40:17PM +0200, Gilles Chanteperdrix wrote: > > > > On Wed, Apr 08, 2015 at 05:29:1

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 06:02:16PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 17:52:18 +0200 > Michael Niedermayer wrote: > > > On Wed, Apr 08, 2015 at 05:40:17PM +0200, Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 05:29:13PM +0200, wm4 wrote: > > > > On Wed, 8 Apr 2015 17:22:58 +0200 >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 17:52:18 +0200 Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 05:40:17PM +0200, Gilles Chanteperdrix wrote: > > On Wed, Apr 08, 2015 at 05:29:13PM +0200, wm4 wrote: > > > On Wed, 8 Apr 2015 17:22:58 +0200 > > > Gilles Chanteperdrix wrote: > > > > > > > On Wed, Apr 08, 2

[FFmpeg-devel] [PATCH] lavf/segment: provide a virtual AVIOContext representing all the segments

2015-04-08 Thread Rodger Combs
> On Mar 30, 2015, at 21:23, Rodger Combs wrote: > > This needs a fair bit of testing and review before merge. > --- > libavformat/segment.c | 259 ++ > 1 file changed, 198 insertions(+), 61 deletions(-) > > diff --git a/libavformat/segment.c b/lib

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 05:29:40PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 04:26:20PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 08, 2015 at 04:01:38PM +0200, Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 03:51:36PM +0200, Lukasz Marek wrote: > > > > On 8 Apr

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 05:40:17PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 05:29:13PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 17:22:58 +0200 > > Gilles Chanteperdrix wrote: > > > > > On Wed, Apr 08, 2015 at 04:13:58PM +0200, wm4 wrote: > > > > On Wed, 8 Apr 2015 15:37:03 +02

Re: [FFmpeg-devel] [PATCH 2/2] lavf/assenc: handle extra sections after Events

2015-04-08 Thread Rodger Combs
> On Apr 4, 2015, at 05:31, Rodger Combs wrote: > > --- > libavformat/assenc.c | 28 ++-- > 1 file changed, 26 insertions(+), 2 deletions(-) > > diff --git a/libavformat/assenc.c b/libavformat/assenc.c > index 3fb9384..5222616 100644 > --- a/libavformat/assenc.c > +++ b/l

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 17:40:17 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 05:29:13PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 17:22:58 +0200 > > Gilles Chanteperdrix wrote: > > > > > On Wed, Apr 08, 2015 at 04:13:58PM +0200, wm4 wrote: > > > > On Wed, 8 Apr 2015 15:37:03 +0200 > >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 05:29:13PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 17:22:58 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 04:13:58PM +0200, wm4 wrote: > > > On Wed, 8 Apr 2015 15:37:03 +0200 > > > Gilles Chanteperdrix wrote: > > > > > > > On Wed, Apr 08, 2015 at 02:40

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 04:26:20PM +0200, Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 04:01:38PM +0200, Gilles Chanteperdrix wrote: > > On Wed, Apr 08, 2015 at 03:51:36PM +0200, Lukasz Marek wrote: > > > On 8 April 2015 at 15:37, Gilles Chanteperdrix < > > > gilles.chanteperd...@xenomai.or

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 17:22:58 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 04:13:58PM +0200, wm4 wrote: > > On Wed, 8 Apr 2015 15:37:03 +0200 > > Gilles Chanteperdrix wrote: > > > > > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > > > On Wed, Apr 08, 2015

[FFmpeg-devel] [RFC][PATCH] avformat/flvdec: avoid reseting eof_reached to 0 silently

2015-04-08 Thread Zhang Rui
avio_feof() and avio_seek() could reset eof_reached to 0, after some silent IO error in avio_r*(). And the reconnection caused by seek makes demuxer to continue read at wrong position. --- libavformat/flvdec.c | 66 1 file changed, 41 insertions

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 04:13:58PM +0200, wm4 wrote: > On Wed, 8 Apr 2015 15:37:03 +0200 > Gilles Chanteperdrix wrote: > > > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > > > Hi, > > > > > > >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 16:18:29 +0200 Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 03:37:03PM +0200, Gilles Chanteperdrix wrote: > > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > > > Hi, >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 04:01:38PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 03:51:36PM +0200, Lukasz Marek wrote: > > On 8 April 2015 at 15:37, Gilles Chanteperdrix < > > gilles.chanteperd...@xenomai.org> wrote: > > > > > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Nieder

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 03:37:03PM +0200, Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > > Hi, > > > > > > I just triend libquvi, and get a segmentation fault in the >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 15:37:03 +0200 Gilles Chanteperdrix wrote: > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > > Hi, > > > > > > I just triend libquvi, and get a segmentation fault in the > > > l

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 03:51:36PM +0200, Lukasz Marek wrote: > On 8 April 2015 at 15:37, Gilles Chanteperdrix < > gilles.chanteperd...@xenomai.org> wrote: > > > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdri

Re: [FFmpeg-devel] [PATCH] chorus filter

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 01:06:19PM +, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol > --- > doc/filters.texi | 54 +++ > libavfilter/Makefile | 1 + > libavfilter/af_chorus.c | 379 > +++ > libavfilter/allfilters.c | 1 + >

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Lukasz Marek
On 8 April 2015 at 15:37, Gilles Chanteperdrix < gilles.chanteperd...@xenomai.org> wrote: > On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > > Hi, > > > > > > I just triend libquvi, and get a segmenta

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 02:40:52PM +0200, Michael Niedermayer wrote: > On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > > Hi, > > > > I just triend libquvi, and get a segmentation fault in the > > libquvi_read_header function, because ff_copy_whitelists is called > > before

Re: [FFmpeg-devel] Areas needing work

2015-04-08 Thread Michael Niedermayer
On Tue, Apr 07, 2015 at 09:08:10PM +0530, Mayank Agarwal wrote: > Hi > > I am interested in contributing to any of the projects.Please enroll > me and tell me the procedure how can > i contribute to ffmpeg You can work on anything you like to work on, anything thats fun to you. Add support for a

Re: [FFmpeg-devel] HQX codec path: faster decode with multi threaded decoding of slices; fixed clipping bug

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 09:26:05AM +0200, Ferdinand Oeinck wrote: [...] > -put_blocks(pic, 0, x, y, flag, ctx->block[0], ctx->block[2], > hqx_quant_luma); > -put_blocks(pic, 0, x + 8, y, flag, ctx->block[1], ctx->block[3], > hqx_quant_luma); > -put_blocks(pic, 2, x >> 1, y, fla

[FFmpeg-devel] [PATCH] chorus filter

2015-04-08 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- doc/filters.texi | 54 +++ libavfilter/Makefile | 1 + libavfilter/af_chorus.c | 379 +++ libavfilter/allfilters.c | 1 + 4 files changed, 435 insertions(+) create mode 100644 libavfilter/af_chorus.

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 10:55:45AM +0200, Gilles Chanteperdrix wrote: > Hi, > > I just triend libquvi, and get a segmentation fault in the > libquvi_read_header function, because ff_copy_whitelists is called > before qc->fmtctx is allocated by avformat_open_input. I added a > call to avformat_all

[FFmpeg-devel] [PATCH]Add disposition default when creating ffprobe-test.nut

2015-04-08 Thread Carl Eugen Hoyos
Hi! Attached patch changes ffprobe-test.nut and the ffprobe test output. It will allow to fix ticket #4455 easily by removing the avdevice dependency from the call to ffmpeg. I don't know how to add the avdevice dependency to tests/Makefile. Please comment, Carl Eugen diff --git a/tests/Makefi

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
On Wed, Apr 08, 2015 at 11:02:47AM +0200, wm4 wrote: > On Wed, 8 Apr 2015 10:55:45 +0200 > Gilles Chanteperdrix wrote: > > > Hi, > > > > I just triend libquvi, and get a segmentation fault in the > > libquvi_read_header function, because ff_copy_whitelists is called > > before qc->fmtctx is all

Re: [FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread wm4
On Wed, 8 Apr 2015 10:55:45 +0200 Gilles Chanteperdrix wrote: > Hi, > > I just triend libquvi, and get a segmentation fault in the > libquvi_read_header function, because ff_copy_whitelists is called > before qc->fmtctx is allocated by avformat_open_input. I added a > call to avformat_alloc_con

[FFmpeg-devel] Segmentation fault in libquvi.c

2015-04-08 Thread Gilles Chanteperdrix
Hi, I just triend libquvi, and get a segmentation fault in the libquvi_read_header function, because ff_copy_whitelists is called before qc->fmtctx is allocated by avformat_open_input. I added a call to avformat_alloc_context() before ff_copy_whitelists and the libquvi demuxer works. However, I

Re: [FFmpeg-devel] [PATCH] [BACKPORT] tests: Fix test name for pixfmts tests

2015-04-08 Thread Michael Niedermayer
On Wed, Apr 08, 2015 at 05:33:59AM +, Timothy Gu wrote: > Hi, > > Please if possible backport the following commit to all release branches > currently being tested: locally done thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Avoid a single point of

[FFmpeg-devel] HQX codec path: faster decode with multi threaded decoding of slices; fixed clipping bug

2015-04-08 Thread Ferdinand Oeinck
hqx.1.patch Description: Binary data hqx.2.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel