[FFmpeg-devel] [PATCH] avformat/http: support auto reconnect

2015-03-11 Thread Zhang Rui
--- libavformat/http.c | 32 +++- 1 file changed, 27 insertions(+), 5 deletions(-) diff --git a/libavformat/http.c b/libavformat/http.c index 55dcb6e..86380b2 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -93,6 +93,7 @@ typedef struct HTTPContext { A

Re: [FFmpeg-devel] [PATCH] avfilter/colormatrix:add slice threading

2015-03-11 Thread Michael Niedermayer
On Mon, Mar 09, 2015 at 10:01:47PM -0700, Yayoi wrote: > --- > separate a struct variable declaraion and assignment statements. > --- > > libavfilter/vf_colormatrix.c | 146 > --- > 1 file changed, 94 insertions(+), 52 deletions(-) applied thanks [...]

Re: [FFmpeg-devel] [PATCH 3/3] avutil/buffer: Avoid moving the AVBufferRef to a new place in memory in av_buffer_make_writable()

2015-03-11 Thread Michael Niedermayer
On Thu, Jan 15, 2015 at 01:04:56AM +0100, Michael Niedermayer wrote: > This allows making a AVBufferRef writable without the need to > update all pointers to it > > Signed-off-by: Michael Niedermayer > --- applied [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Re: [FFmpeg-devel] [PATCH 2/3] avutil/buffer: Avoid moving the AVBufferRef to a new place in memory in av_buffer_realloc()

2015-03-11 Thread Michael Niedermayer
On Thu, Jan 15, 2015 at 01:04:55AM +0100, Michael Niedermayer wrote: > This allows reallocating AVBufferRefs without the need to update > all pointers to it > > Signed-off-by: Michael Niedermayer > --- > libavutil/buffer.c |3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) applied [.

[FFmpeg-devel] [PATCH] avcodec/vp9: Fix undefined shifts in decode_frame_header()

2015-03-11 Thread Michael Niedermayer
Found-by: Clang -fsanitize=shift Reported-by: Thierry Foucu Signed-off-by: Michael Niedermayer --- libavcodec/vp9.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 265dc7e..0405c05 100644 --- a/libavcodec/vp9.c +++ b/libavcodec

[FFmpeg-devel] [PATCH] avcodec/h264_mb: Fix undefined shifts

2015-03-11 Thread Michael Niedermayer
Found-by: Clang -fsanitize=shift Reported-by: Thierry Foucu Signed-off-by: Michael Niedermayer --- libavcodec/h264_mb.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/libavcodec/h264_mb.c b/libavcodec/h264_mb.c index dd406c7..a4653aa 100644 --- a/libavcodec/

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Derek Buitenhuis
On 3/11/2015 2:42 PM, Marcus Johnson wrote: > I've been working on adding XLL for the last couple months, it's still not > quite complete, basically I have to combine the Core and XLL samples before > it's output, and I also have to finish the latter stages of decoding the > XLL like channel decorr

Re: [FFmpeg-devel] [PATCH 2/2] Moved templated c postprocessing routines into seperate file

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 01:51:13PM -0400, Tucker DiNapoli wrote: > Currently different versions of the postprocessing routines are > generated from a template. Ultimately I intend to remove this by > replacing the inline assembly with seperate yasm files. The c routines > will still be needed, so t

Re: [FFmpeg-devel] [PATCH 1/2] Moved postprocessing routines from postprocess.c to seperate file

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 01:51:12PM -0400, Tucker DiNapoli wrote: > This moves c functions to process blocks horozontally into a seperate > file, so that none of the postprocessing algorithms are in the main > postprecess.c file [...] > diff --git a/libpostproc/postprocess_c.c b/libpostproc/postproc

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Derek Buitenhuis
On 3/11/2015 9:36 PM, Michael Niedermayer wrote: > Thats analogous to saying "no its not important to put fuel in a > car, its important to drive the best car" No what I propose is to look at both and decide which is best. Simply being submitted to FFmpeg first does not make it better. > the 2nd

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 09:04:39PM +, Derek Buitenhuis wrote: > On 3/11/2015 8:25 PM, Michael Niedermayer wrote: > > whats important is a patchset based on and tested on FFmpeg, at least > > if people want a working decoder in FFmpeg > > No, what's important is that the best possible code is u

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Derek Buitenhuis
On 3/11/2015 8:25 PM, Michael Niedermayer wrote: > whats important is a patchset based on and tested on FFmpeg, at least > if people want a working decoder in FFmpeg No, what's important is that the best possible code is used. Origin is irrelevant. - Derek

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 04:07:00PM +0100, Hendrik Leppkes wrote: > On Wed, Mar 11, 2015 at 4:02 PM, Marcus Johnson > wrote: > > I thought the patch on LibAV was completely removed? > > http://thread.gmane.org/gmane.comp.video.libav.devel/66825/focus=66826 > > It'll probably be merged soon'ish. I

[FFmpeg-devel] (no subject)

2015-03-11 Thread Deepak Sharma
sir, i really want to contribute to your project on "SUBTITLES". i have good coding skill in c and i very well know how to use git. but i have no idea of fansubbing but i will learn it as soon as possible. please guide me here, i will be very thankful. __

[FFmpeg-devel] [PATCH] avcodec/ac3dec: Fix undefined shifts

2015-03-11 Thread Michael Niedermayer
Found-by: Clang -fsanitize=shift Reported-by: Thierry Foucu Signed-off-by: Michael Niedermayer --- libavcodec/ac3dec.c |7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/ac3dec.c b/libavcodec/ac3dec.c index 2f78d73..3200bcb 100644 --- a/libavcodec/ac3dec.c +

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Hendrik Leppkes
On Wed, Mar 11, 2015 at 4:02 PM, Marcus Johnson wrote: > I thought the patch on LibAV was completely removed? http://thread.gmane.org/gmane.comp.video.libav.devel/66825/focus=66826 It'll probably be merged soon'ish. It still has some open TODOs, but at this point its probably a much better idea

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Marcus Johnson
I thought the patch on LibAV was completely removed? it was purged from the codebase like 9 months ago or something, I stumbled on that while trying to fix some of the issues with the white paper I was having. I haven't bothered with the Core decoder, but everything I've extracted so far is fixed

[FFmpeg-devel] [PATCH v3] mips/asmdefs: use _ABI64 as defined by gcc

2015-03-11 Thread James Cowgill
Unfortunately android < api 21 (lollipop) doesn't have the sgidefs.h header, the easiest way around this is to just use the preprocessor definitions from gcc / clang. Signed-off-by: James Cowgill --- Hi, Sorry I forgot about this a little. I think that doing it this way is better than messing a

Re: [FFmpeg-devel] DCA Decoder

2015-03-11 Thread Derek Buitenhuis
On 3/11/2015 2:42 PM, Marcus Johnson wrote: > I've been working on adding XLL for the last couple months, it's still not > quite complete, basically I have to combine the Core and XLL samples before > it's output, and I also have to finish the latter stages of decoding the > XLL like channel decorr

[FFmpeg-devel] DCA Decoder

2015-03-11 Thread Marcus Johnson
I've been working on adding XLL for the last couple months, it's still not quite complete, basically I have to combine the Core and XLL samples before it's output, and I also have to finish the latter stages of decoding the XLL like channel decorreclation, and post processing. There are a few issu

Re: [FFmpeg-devel] [PATCH v2] mips/asmdefs: use asm/sgidefs.h header on linux

2015-03-11 Thread Michael Niedermayer
On Sat, Mar 07, 2015 at 12:42:30PM +, James Cowgill wrote: > On Sat, 2015-03-07 at 13:32 +0100, Michael Niedermayer wrote: > > On Sat, Mar 07, 2015 at 10:56:45AM +, James Cowgill wrote: > > > Unfortunately android < api 21 (lollipop) doesn't have the sgidefs.h > > > header, > > > but the l

Re: [FFmpeg-devel] [PATCH 2/4] x86: xvid_idct: port MMX IDCT to yasm

2015-03-11 Thread Christophe Gisquet
Hi, 2015-03-11 15:20 GMT+01:00 Michael Niedermayer : > personally iam in favor of more things being tested, but iam fine > with either I'll see what the opinions are in ~30H then. It looks to be around 4.5KB more data, which isn't huge, and doesn't cause a noticeable increase in compile time. --

Re: [FFmpeg-devel] [PATCH 2/4] x86: xvid_idct: port MMX IDCT to yasm

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 03:09:51PM +0100, Christophe Gisquet wrote: > 2015-03-11 2:26 GMT+01:00 Michael Niedermayer : > > this breaks > > make libavcodec/dct-test > > ... > > LD libavcodec/dct-test > > libavcodec/dct-test.o:(.rodata+0xe8): undefined reference to > > `ff_xvid_idct_mmx' > > lib

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 02:59:04PM +0100, Nicolas George wrote: > Le primidi 21 ventôse, an CCXXIII, Michael Niedermayer a écrit : > > the question or well rather my question is why do they need to put > > control characters in there in the first place. > > There should be no need for that > > Con

Re: [FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-11 Thread Ronald S. Bultje
Hi, On Wed, Mar 11, 2015 at 2:06 AM, James Almer wrote: > On 11/03/15 12:57 AM, Tucker DiNapoli wrote: > > From: Tucker DiNapoli > > A couple comments below. > > > > > The only changes were formating and moving code. > > This needs to be split into several different patches. Also, why does > "m

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Nicolas George
Le primidi 21 ventôse, an CCXXIII, Michael Niedermayer a écrit : > the question or well rather my question is why do they need to put > control characters in there in the first place. > There should be no need for that Control characters are the only characters that are not valid in a HTTP header

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Michael Niedermayer
On Wed, Mar 11, 2015 at 01:24:28PM +0100, Nicolas George wrote: > Le primidi 21 ventôse, an CCXXIII, Michael Niedermayer a écrit : > > what is the problem ? > > can you elaborate? > > The problem raised in this trac ticket is that windows users seem unable to > put control characters in their comm

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Nicolas George
Le primidi 21 ventôse, an CCXXIII, Reimar Döffinger a écrit : > Of course, it's just a pointless waste of server CPU time for already > compressed files. Possibly not just CPU time, user time too: on my box (rather fast for that kind of thing), the speed drops from 2500 Mo/s (based on used CPU tim

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Nicolas George
Le primidi 21 ventôse, an CCXXIII, Michael Niedermayer a écrit : > what is the problem ? > can you elaborate? The problem raised in this trac ticket is that windows users seem unable to put control characters in their command lines. I thought it was a shortcoming of the windows command line interp

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Reimar Döffinger
On 11.03.2015, at 13:05, Lukasz Marek wrote: > On 11.03.2015 11:15, Nicolas George wrote: >> >> >> I firmly believe in the following principles: >> >> 1. If the user made an explicit choice, the application must obey, even if >>the choice seems stupid. >> >>In the case of SSH: compr

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Lukasz Marek
On 11.03.2015 11:15, Nicolas George wrote: Le primidi 21 ventôse, an CCXXIII, Florian Jacob a écrit : Do we have a way to apply ffmpeg-specific options? We can force to use no compression in the code, by adding this line: ssh_options_set(libssh->session, SSH_OPTIONS_COMPRESSION, "no"); after

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Michael Niedermayer
On Tue, Mar 10, 2015 at 02:09:10PM +0100, Nicolas George wrote: > Le decadi 20 ventôse, an CCXXIII, Carl Eugen Hoyos a écrit : > > Please add a reference to ticket #3268 > > to the commit message. > > I was about to reply "locally added", but no, because it does not fix that > ticket. what is th

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-11 Thread Reimar Döffinger
On 09.03.2015, at 15:54, Hendrik Leppkes wrote: > > Of course avcodec shouldn't crash when you use it, but a patch to > disallow this combination would never be accepted by yourself and some > others that "like" this cheap (albeit wrong) method for fallback. I forgot to reply: of course I would

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-11 Thread Reimar Döffinger
On 11.03.2015, at 12:34, wm4 wrote: > On Tue, 10 Mar 2015 18:02:12 +0100 > Reimar Döffinger wrote: Also the buffering issue is the other way, when you try to go from multithreading to HW decode, how do you handle that? If it works well I'd be totally in favour, but I strongly s

Re: [FFmpeg-devel] [PATCH] pthread: Fix ff_thread_get_formatissues when called outside frame decode.

2015-03-11 Thread wm4
On Tue, 10 Mar 2015 18:02:12 +0100 Reimar Döffinger wrote: > On 10.03.2015, at 12:10, wm4 wrote: > > > On Mon, 09 Mar 2015 18:56:57 +0100 > > Reimar Döffinger wrote: > >>> > >>> What I do is simply restart decoding with the packet that failed the > >>> hardware decoder. Don't need to buffer a

Re: [FFmpeg-devel] [PATCH] [PATCH] lavfi: add inverse telecine filter

2015-03-11 Thread Himangi Saraogi
On 11 March 2015 at 15:10, Clément Bœsch wrote: > On Wed, Mar 11, 2015 at 07:12:16AM +, Carl Eugen Hoyos wrote: > > Timothy Gu gmail.com> writes: > > > > > > This is an exact inverse of the telecine filter > > > > unlike previously existing pullup and fieldmatch > > > > ones. > > > > > > Wha

Re: [FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

2015-03-11 Thread Nicolas George
Le decadi 20 ventôse, an CCXXIII, Nicolas George a écrit : > I will try to propose another patch for pure validation. See the attached patches. Regards, -- Nicolas George From cef95aaec223b6644db6e52f9c6452d7aa01648b Mon Sep 17 00:00:00 2001 From: Nicolas George Date: Tue, 10 Mar 2015 13:43:

Re: [FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-11 Thread Michael Niedermayer
Hi On Tue, Mar 10, 2015 at 11:57:56PM -0400, Tucker DiNapoli wrote: > From: Tucker DiNapoli > > The only changes were formating and moving code. > --- > libpostproc/postprocess.c | 436 ++-- > libpostproc/postprocess_c.c| 1328 > >

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Reimar Döffinger
On 11.03.2015, at 11:15, Nicolas George wrote: > I firmly believe in the following principles: > > 1. If the user made an explicit choice, the application must obey, even if > the choice seems stupid. > > In the case of SSH: compression is disabled by default, so enabling it is > an explic

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Reimar Döffinger
On 11.03.2015, at 11:00, Florian Jacob wrote: > Am Mittwoch, 11. März 2015, 10:27:55 schrieb Reimar Döffinger: >> On 11.03.2015, at 10:13, Florian Jacob > wrote: >> If so, should they default to disabling compression? > I only use ffmpeg to play videos in already-compressed formats, so it woul

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Nicolas George
Le primidi 21 ventôse, an CCXXIII, Florian Jacob a écrit : > > Do we have a way to apply > > ffmpeg-specific options? > We can force to use no compression in the code, by adding this line: > > ssh_options_set(libssh->session, SSH_OPTIONS_COMPRESSION, "no"); > > after parsing the config file. I t

Re: [FFmpeg-devel] [PATCH] Fixed memory leak in EvalContext::channel_values

2015-03-11 Thread Ole Andre Birkedal
I'm not familiar with writing fate tests but it's something I'll look into - Ole Andre Birkedal 2015-03-10 14:42 GMT+01:00 Michael Niedermayer : > On Tue, Mar 10, 2015 at 02:23:26PM +0100, Ole Andre Birkedal wrote: > > There are some doubles being allocated in aeval_config_output which is > > ca

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Florian Jacob
Am Mittwoch, 11. März 2015, 10:27:55 schrieb Reimar Döffinger: > On 11.03.2015, at 10:13, Florian Jacob wrote: > > 11.03.2015, 00:51:39 by Timothy Gu: > >> Does this patch change the default behavior in any way? > > > > No, at least it's not intended to change for users who don't have a > > ~/.s

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Nicolas George
Le primidi 21 ventôse, an CCXXIII, Reimar Döffinger a écrit : > Well, I can think of one case: users that enable compression in the config > file. > That makes sense for interactive session and when copying normal files, but > for multimedia it is mostly nonsense. Yes, but that applies to all SS

Re: [FFmpeg-devel] [PATCH] [PATCH] lavfi: add inverse telecine filter

2015-03-11 Thread Clément Bœsch
On Wed, Mar 11, 2015 at 07:12:16AM +, Carl Eugen Hoyos wrote: > Timothy Gu gmail.com> writes: > > > > This is an exact inverse of the telecine filter > > > unlike previously existing pullup and fieldmatch > > > ones. > > > > What's the difference between the three seemingly > > similar fi

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Reimar Döffinger
On 11.03.2015, at 10:13, Florian Jacob wrote: > 11.03.2015, 00:51:39 by Timothy Gu: >> Does this patch change the default behavior in any way? > > No, at least it's not intended to change for users who don't have a > ~/.ssh/config. > > If there are users who have a config lying around and expec

Re: [FFmpeg-devel] [PATCH] lavfi: Add support to process_command in vf_eq.c

2015-03-11 Thread Stefano Sabatini
On date Wednesday 2015-03-11 10:55:07 +0530, Arwa Arif encoded: > On Tue, Mar 10, 2015 at 2:41 PM, Stefano Sabatini > wrote: [...] > > > There are no parameters accepted by the expressions. I will add some > > > examples in the doc. > > > > Look how it is done in hue. In general an expression is u

Re: [FFmpeg-devel] [PATCH] sftp: support reading config from ~/.ssh/config

2015-03-11 Thread Florian Jacob
11.03.2015, 00:51:39 by Timothy Gu: > Does this patch change the default behavior in any way? No, at least it's not intended to change for users who don't have a ~/.ssh/config. If there are users who have a config lying around and expecting it to be ignored, let's say they don's specify a usern

Re: [FFmpeg-devel] [PATCH] [PATCH] lavfi: add inverse telecine filter

2015-03-11 Thread Carl Eugen Hoyos
Timothy Gu gmail.com> writes: > > This is an exact inverse of the telecine filter > > unlike previously existing pullup and fieldmatch > > ones. > > What's the difference between the three seemingly > similar filters? The other two are similar (speed and license are different): They try to