[FFmpeg-devel] [PATCH]Fix dnxhd pix_fmt change

2015-03-28 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4400 for me, please comment. Carl Eugen diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c index cc011c4..053bf1c 100644 --- a/libavcodec/dnxhddec.c +++ b/libavcodec/dnxhddec.c @@ -119,6 +119,7 @@ static int dnxhd_decode_header(DNXHDContext *ctx, AVFrame

[FFmpeg-devel] [PATCH v2] af_channelmap: fix number of channels

2015-03-28 Thread Marton Balint
Fixes segfaults with the following command: ffmpeg -f lavfi -i aevalsrc=0:c=stereo:n=1920 -af 'channelmap=0' test.ac3 Signed-off-by: Marton Balint --- libavfilter/af_channelmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/af_channelmap.c b/libavfilter/af_channelmap.c index

Re: [FFmpeg-devel] Fwd: GSoC: APNG

2015-03-28 Thread Donny Yang
Sorry for the delay. I was a bit busy this week. On 25 March 2015 at 09:59, Michael Niedermayer wrote: > is there any advantage for multiple small IDATs ? > if not i suggest to make the IDAT change to png as well in a seperate > patch so that a single frame APNG and PNG produce identical data >

Re: [FFmpeg-devel] [PATCH v2] af_channelmap: fix number of channels

2015-03-28 Thread Nicolas George
L'octidi 8 germinal, an CCXXIII, Marton Balint a écrit : > Fixes segfaults with the following command: > > ffmpeg -f lavfi -i aevalsrc=0:c=stereo:n=1920 -af 'channelmap=0' test.ac3 I find strange that "=0" works at all, but the change itself seems absolutely right. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: Do not overwrite output if there is no input

2015-03-28 Thread Nicolas George
Le septidi 7 germinal, an CCXXIII, Michael Niedermayer a écrit : > yes, theres a test that does this in fate Oh, I realize I did not look at your patch closely enough. Sorry. No objection from me. Regards, -- Nicolas George signature.asc Description: Digital signature __

Re: [FFmpeg-devel] [GSoC] Proof-of-concept HTTP Server

2015-03-28 Thread Nicolas George
Le septidi 7 germinal, an CCXXIII, Stephan Holljes a écrit : > I just copied that from the line above, I was not aware of the meaning > of the letter at all. Changed to "E". D and E are short-cuts for AV_OPT_FLAG_DECODING_PARAM and AV_OPT_FLAG_ENCODING_PARAM, that allows front-ends to filter optio

Re: [FFmpeg-devel] [PATCH/TOY] zip files

2015-03-28 Thread Nicolas George
L'octidi 8 germinal, an CCXXIII, Peter Ross a écrit : > Did a cursory search before selecting libzip... had hoped to find a general > purpose > library for reading popular archive formats. Alas, none seem to exist. What about libarchive? http://www.libarchive.org/ It is packaged in Debian since

Re: [FFmpeg-devel] [PATCH v2] af_channelmap: fix number of channels

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 10:12:48AM +0100, Nicolas George wrote: > L'octidi 8 germinal, an CCXXIII, Marton Balint a écrit : > > Fixes segfaults with the following command: > > > > ffmpeg -f lavfi -i aevalsrc=0:c=stereo:n=1920 -af 'channelmap=0' test.ac3 > > I find strange that "=0" works at all, b

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

2015-03-28 Thread Nicolas George
Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : > But, this time I dont understand you comments, could you elaborate it? > What's wrong, what can I do? What I am saying is that there are a lot of different cases where we want to read archives (not only zip, see my previous mail, but that

Re: [FFmpeg-devel] [PATCH] Refactor repeated HLS option updates (Backport to 2.6)

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 04:16:49PM +1100, Micah Galizia wrote: > Hi, > > Backport code to refactor HLS option updates. > > TIA > -- > "The mark of an immature man is that he wants to die nobly for a > cause, while the mark of the mature man is that he wants to live > humbly for one." --W. Stek

Re: [FFmpeg-devel] [PATCH] Store cookies returned in HLS key (Backport to 2.6)

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 04:15:34PM +1100, Micah Galizia wrote: > Hi, > > Backport code to store cookies stored by HLS key. > > -- > "The mark of an immature man is that he wants to die nobly for a > cause, while the mark of the mature man is that he wants to live > humbly for one." --W. Stekel

Re: [FFmpeg-devel] [PATCH] replace cookies with updated values instead of appending forever (backport to 2.6)

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 04:18:39PM +1100, Micah Galizia wrote: > Hi, > > One more -- backport code to replace cookie values instead appending > to the header forever. > > -- > "The mark of an immature man is that he wants to die nobly for a > cause, while the mark of the mature man is that he wa

Re: [FFmpeg-devel] Fwd: GSoC: APNG

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 09:12:10AM +, Donny Yang wrote: > Sorry for the delay. I was a bit busy this week. > > On 25 March 2015 at 09:59, Michael Niedermayer wrote: > > > is there any advantage for multiple small IDATs ? > > if not i suggest to make the IDAT change to png as well in a sepera

[FFmpeg-devel] [PATCH 1/5] png: Use av_freep() instead of av_free()

2015-03-28 Thread Donny Yang
Signed-off-by: Donny Yang --- libavcodec/pngenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c index 9bdefc4..0231bf7 100644 --- a/libavcodec/pngenc.c +++ b/libavcodec/pngenc.c @@ -512,9 +512,9 @@ static int encode_frame(AVCod

[FFmpeg-devel] [PATCH 3/5] png: Clearly separate init, encoding headers and encoding frame

2015-03-28 Thread Donny Yang
Signed-off-by: Donny Yang --- libavcodec/pngenc.c | 286 ++-- 1 file changed, 163 insertions(+), 123 deletions(-) diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c index d6233d0..0264575 100644 --- a/libavcodec/pngenc.c +++ b/libavcodec/pngenc

[FFmpeg-devel] [PATCH 2/5] png: Minor whitespace change and added missing comment

2015-03-28 Thread Donny Yang
Signed-off-by: Donny Yang --- libavcodec/pngenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/pngenc.c b/libavcodec/pngenc.c index 0231bf7..d6233d0 100644 --- a/libavcodec/pngenc.c +++ b/libavcodec/pngenc.c @@ -399,9 +399,9 @@ static int encode_frame(AVCodecCont

Re: [FFmpeg-devel] Fwd: GSoC: APNG

2015-03-28 Thread Donny Yang
On 29 March 2015 at 00:08, Michael Niedermayer wrote: > git send-email should send one patch per mail Okay, here's my try with that > pts == 0 is not a reliable way to detect the first picture What would be the recommended way, then? > this patchset also breaks PAL8 Yes, I know. The only reason

[FFmpeg-devel] [PATCH 4/5] apng: Make PNG encoder only write headers once in APNG mode

2015-03-28 Thread Donny Yang
Signed-off-by: Donny Yang --- configure | 1 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 2 +- libavcodec/pngenc.c| 40 libavcodec/version.h | 2 +- 5 files changed, 40 insertions(+), 6 deletions(-) diff --git a/configu

[FFmpeg-devel] [PATCH 5/5] apng: Add a basic APNG muxer

2015-03-28 Thread Donny Yang
Additionally, update some documentation with support for APNG Signed-off-by: Donny Yang --- Changelog| 1 + doc/general.texi | 2 + libavformat/Makefile | 1 + libavformat/allformats.c | 2 +- libavformat/apngenc.c| 249

Re: [FFmpeg-devel] [PATCH]Fix dnxhd pix_fmt change

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 09:38:12AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4400 for me, please comment. > > Carl Eugen > dnxhddec.c |9 + > 1 file changed, 5 insertions(+), 4 deletions(-) > faf7809dc5b971e27c493e25e8f56a41e7743bd7 patchdnxhd.diff > dif

Re: [FFmpeg-devel] Fwd: GSoC: APNG

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 03:14:20PM +, Donny Yang wrote: > > On 29 March 2015 at 00:08, Michael Niedermayer wrote: > > git send-email should send one patch per mail > Okay, here's my try with that > > > pts == 0 is not a reliable way to detect the first picture > What would be the recommende

Re: [FFmpeg-devel] [PATCH 1/5] png: Use av_freep() instead of av_free()

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 03:14:21PM +, Donny Yang wrote: > Signed-off-by: Donny Yang > --- > libavcodec/pngenc.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No great genius ha

Re: [FFmpeg-devel] [PATCH 2/5] png: Minor whitespace change and added missing comment

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 03:14:22PM +, Donny Yang wrote: > Signed-off-by: Donny Yang > --- > libavcodec/pngenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No human being will ever

Re: [FFmpeg-devel] [PATCH]Add an ignore_delay option to the gif demuxer

2015-03-28 Thread Carl Eugen Hoyos
On Friday 27 March 2015 02:45:22 pm Michael Niedermayer wrote: > On Fri, Mar 27, 2015 at 08:07:23AM +, Carl Eugen Hoyos wrote: > > Michael Niedermayer gmx.at> writes: > > > On Fri, Mar 27, 2015 at 12:21:00AM +, Carl Eugen Hoyos wrote: > > > > Michael Niedermayer gmx.at> writes: > > > > >

Re: [FFmpeg-devel] [PATCH]Add an ignore_delay option to the gif demuxer

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 05:47:52PM +0100, Carl Eugen Hoyos wrote: > On Friday 27 March 2015 02:45:22 pm Michael Niedermayer wrote: > > On Fri, Mar 27, 2015 at 08:07:23AM +, Carl Eugen Hoyos wrote: > > > Michael Niedermayer gmx.at> writes: > > > > On Fri, Mar 27, 2015 at 12:21:00AM +, Carl

[FFmpeg-devel] [PATCH] tiff: Return more meaningful error codes

2015-03-28 Thread Himangi Saraogi
--- libavcodec/tiffenc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavcodec/tiffenc.c b/libavcodec/tiffenc.c index 44cd956..fc9b78e 100644 --- a/libavcodec/tiffenc.c +++ b/libavcodec/tiffenc.c @@ -164,7 +164,8 @@ static int add_entry1(TiffEncoderContext *s,

Re: [FFmpeg-devel] [PATCH]Add an ignore_delay option to the gif demuxer

2015-03-28 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > was this the intent of the creator of the sample file we have ? > i mean that the frames display more then 10 minutes There is no indication that anything else was intended. To the best of my knowledge, the file is not invalid (and there is no bug except t

Re: [FFmpeg-devel] [PATCH] tiff: Return more meaningful error codes

2015-03-28 Thread Carl Eugen Hoyos
Himangi Saraogi gmail.com> writes: > if (compress(dst, &zlen, src, n) != Z_OK) { > av_log(s->avctx, AV_LOG_ERROR, "Compressing failed\n"); > -return -1; > +return AVERROR_INVALIDDATA; -1 seems more correct to me. > case TIFF_LZW: > ret

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

2015-03-28 Thread Lukasz Marek
On 28.03.2015 11:53, Nicolas George wrote: Le septidi 7 germinal, an CCXXIII, Lukasz Marek a écrit : But, this time I dont understand you comments, could you elaborate it? What's wrong, what can I do? What I am saying is that there are a lot of different cases where we want to read archives (n

Re: [FFmpeg-devel] [PATCH] tiff: Return more meaningful error codes

2015-03-28 Thread Nicolas George
L'octidi 8 germinal, an CCXXIII, Carl Eugen Hoyos a écrit : > -1 seems more correct to me. -1 is never correct. On Linux (and apparently BSD too), -1 is AVERROR(EPERM), definitely not what you want. One libav guy suggested AVERROR_UNKNOWN; in FFmpeg we have AVERROR_EXTERNAL "Generic error in an e

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

2015-03-28 Thread Nicolas George
L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > I will try to use this libarchive first and do some tests. Your approach may > collapse in case compression libraries doesn't support parallel > compression/decompression (I mean that you write or read several files from > single archive fil

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

2015-03-28 Thread Lukasz Marek
On 28.03.2015 20:13, Nicolas George wrote: L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : I will try to use this libarchive first and do some tests. Your approach may collapse in case compression libraries doesn't support parallel compression/decompression (I mean that you write or read

Re: [FFmpeg-devel] [PATCH]Add an ignore_delay option to the gif demuxer

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 05:54:49PM +, Carl Eugen Hoyos wrote: > Michael Niedermayer gmx.at> writes: > > > was this the intent of the creator of the sample file we have ? > > i mean that the frames display more then 10 minutes > > There is no indication that anything else was intended. > To

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

2015-03-28 Thread Peter Ross
On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > On 28.03.2015 20:13, Nicolas George wrote: > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > >>I will try to use this libarchive first and do some tests. Your approach may > >>collapse in case compression libraries doesn't s

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

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 08:10:29 +1100 Peter Ross wrote: > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > On 28.03.2015 20:13, Nicolas George wrote: > > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > > >>I will try to use this libarchive first and do some tests. Your ap

[FFmpeg-devel] [PATCH 1/4] avfilter/tinterlace: Change enum to int, which is accessed via AVOption as int

2015-03-28 Thread Michael Niedermayer
This fixes depending on implementation defined behavior Signed-off-by: Michael Niedermayer --- libavfilter/tinterlace.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/tinterlace.h b/libavfilter/tinterlace.h index fa0a83a..d80a6e2 100644 --- a/libavfilter/tinte

[FFmpeg-devel] [PATCH 2/4] avfilter/vf_transpose: Change enums to int, which are accessed via AVOption as int

2015-03-28 Thread Michael Niedermayer
This fixes depending on implementation defined behavior Signed-off-by: Michael Niedermayer --- libavfilter/vf_transpose.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_transpose.c b/libavfilter/vf_transpose.c index d9b165c..dd570e6 100644 --- a/libavfi

[FFmpeg-devel] [PATCH 3/4] avformat/wavenc: Change enum to int, which is accessed via AVOption as int

2015-03-28 Thread Michael Niedermayer
This fixes depending on implementation defined behavior Signed-off-by: Michael Niedermayer --- libavformat/wavenc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/wavenc.c b/libavformat/wavenc.c index 2345fc5..0333622 100644 --- a/libavformat/wavenc.c +++ b/li

[FFmpeg-devel] [PATCH 4/4] avformat/img2: Change enum to int, which is accessed via AVOption as int

2015-03-28 Thread Michael Niedermayer
This fixes depending on implementation defined behavior Signed-off-by: Michael Niedermayer --- libavformat/img2.h |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavformat/img2.h b/libavformat/img2.h index 67bd7a3..e4bcb26 100644 --- a/libavformat/img2.h +++ b/lib

Re: [FFmpeg-devel] [PATCH 3/5] png: Clearly separate init, encoding headers and encoding frame

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 03:14:23PM +, Donny Yang wrote: > Signed-off-by: Donny Yang [...] > -enc_row_size= deflateBound(&s->zstream, row_size); > -max_packet_size = avctx->height * (int64_t)(enc_row_size + > - ((enc_row_size + IOBUF_SIZE - 1) /

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: Do not overwrite output if there is no input

2015-03-28 Thread Michael Niedermayer
On Sat, Mar 28, 2015 at 11:03:22AM +0100, Nicolas George wrote: > Le septidi 7 germinal, an CCXXIII, Michael Niedermayer a écrit : > > yes, theres a test that does this in fate > > Oh, I realize I did not look at your patch closely enough. Sorry. No > objection from me. ok patch applied thanks

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

2015-03-28 Thread Peter Ross
On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > On Sun, 29 Mar 2015 08:10:29 +1100 > Peter Ross wrote: > > > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > > On 28.03.2015 20:13, Nicolas George wrote: > > > >L'octidi 8 germinal, an CCXXIII, Lukasz Marek a écrit : > > > >

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

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 08:55:42 +1100 Peter Ross wrote: > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > > On Sun, 29 Mar 2015 08:10:29 +1100 > > Peter Ross wrote: > > > > > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > > > On 28.03.2015 20:13, Nicolas George wrote: > >

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

2015-03-28 Thread Lukasz Marek
W dniu sobota, 28 marca 2015 Peter Ross napisał(a): > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > > I assumed it is local file (no other option so far). So I stat full path > > (/tmp/outer.zip/tmp/inner.zip/tmp/data.bin) for being a file, if so then > I > > opened it as zip fi

[FFmpeg-devel] [PATCH] Add "split" mode to tinterlace filter.

2015-03-28 Thread Brian Matherly
From: Brian Matherly This mode is the opposite of the "merge" mode. --- This patch adds a new mode to tinterlace which performs the opposite operation as the "merge" mode. My primary motivation is that I have been working with Derek Buitenhuis to see about adding interlace support to the libx

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

2015-03-28 Thread Ivan Kalvachev
On 3/28/15, Peter Ross wrote: > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: >> On Sun, 29 Mar 2015 08:10:29 +1100 >> Peter Ross wrote: >> >> > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: >> > > On 28.03.2015 20:13, Nicolas George wrote: >> > > >L'octidi 8 germinal, an C

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

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 01:12:32 +0200 Ivan Kalvachev wrote: > On 3/28/15, Peter Ross wrote: > > On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: > >> On Sun, 29 Mar 2015 08:10:29 +1100 > >> Peter Ross wrote: > >> > >> > On Sat, Mar 28, 2015 at 08:38:40PM +0100, Lukasz Marek wrote: > >> > > On

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

2015-03-28 Thread Nicolas George
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. Regards, -- Nicolas George signature.asc Descript

Re: [FFmpeg-devel] [PATCH 2/4] libavdevice/avfoundation: add capture_screen_cursor option

2015-03-28 Thread Thilo Borgmann
Am 27.03.15 um 14:36 schrieb Matthieu Bouron: > On Sat, Mar 21, 2015 at 5:35 PM, Thilo Borgmann > wrote: > [...] > >> I suggest to rename the option to "capture_cursor"... _screen_ in it seems >> to be >> redundant. >> > > Updated in attached patch. Ok, please apply. Thanks, Thilo ___

Re: [FFmpeg-devel] [PATCH 3/4] libavdevice/avfoundation: add capture_screen_mouse_clicks option

2015-03-28 Thread Thilo Borgmann
Am 27.03.15 um 14:37 schrieb Matthieu Bouron: > On Sat, Mar 21, 2015 at 6:47 PM, Thilo Borgmann > wrote: > [...] > >> >> Please rename to "capture_mouse_clicks". >> > > Updated in attached patch. Ok, please apply. Thanks, Thilo ___ ffmpeg-devel mai

Re: [FFmpeg-devel] [PATCH 4/4] libavdevice/avfoundation: use pts/dts provided by the CMSampleBuffer API

2015-03-28 Thread Thilo Borgmann
Am 21.03.15 um 18:45 schrieb Thilo Borgmann: > Am 18.03.15 um 09:37 schrieb Matthieu Bouron: >> On Fri, Mar 13, 2015 at 8:16 PM, wrote: >> >>> From: Matthieu Bouron >>> >>> --- >>> libavdevice/avfoundation.m | 21 +++-- >>> 1 file changed, 15 insertions(+), 6 deletions(-) >>> >>>

Re: [FFmpeg-devel] [PATCH 1/4] libavdevice/avfoundation: add framerate and video size options

2015-03-28 Thread Thilo Borgmann
Am 27.03.15 um 14:34 schrieb Matthieu Bouron: > On Sat, Mar 21, 2015 at 4:51 PM, Thilo Borgmann > wrote: > [...] >> +[video_device setValue:selected_format forKey:@"activeFormat"]; >> +[video_device setValue:min_frame_duration >> forKey:@"activeVideoMinFrameDuration"]; >> +

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

2015-03-28 Thread Timo Rothenpieler
> 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. While it is useless, for example scripts freque

[FFmpeg-devel] [PATCH] avformat/http: Return an error in case of prematurely ending data

2015-03-28 Thread Michael Niedermayer
Fixes Ticket 4039 Signed-off-by: Michael Niedermayer --- libavformat/http.c |8 1 file changed, 8 insertions(+) diff --git a/libavformat/http.c b/libavformat/http.c index da3c9be..a1d3763 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -907,6 +907,14 @@ static int http_

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

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 00:20:53 +0100 Nicolas George 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

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

2015-03-28 Thread Alexander Strasser
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

Re: [FFmpeg-devel] [PATCH 1/4] lavf: add directory listing API

2015-03-28 Thread Mariusz Szczepańczyk
On Fri, Mar 27, 2015 at 6:52 PM, Michael Niedermayer wrote: > On Thu, Mar 26, 2015 at 03:31:27PM +0100, Mariusz Szczepańczyk wrote: > > On Thu, Mar 26, 2015 at 2:31 PM, Michael Niedermayer > > wrote: > > > > > On Thu, Mar 26, 2015 at 01:25:17AM +0100, Mariusz Szczepańczyk wrote: > > > > From: Lu

[FFmpeg-devel] [PATCH] libavformat/hls: add an option to start from a given segment in a live stream

2015-03-28 Thread Rodger Combs
--- libavformat/hls.c | 28 1 file changed, 24 insertions(+), 4 deletions(-) diff --git a/libavformat/hls.c b/libavformat/hls.c index af890bd..4a7d003 100644 --- a/libavformat/hls.c +++ b/libavformat/hls.c @@ -165,6 +165,7 @@ struct variant { }; typedef struct HLS

[FFmpeg-devel] [PATCH 1/4] libavformat/segment: don't leave the list pb open when not in use

2015-03-28 Thread Rodger Combs
--- libavformat/segment.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index b65a2eb..06bc459 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -357,17 +357,17 @@ static int segment_end(AVFormatContext *s

[FFmpeg-devel] [PATCH 2/4] lavf/segment: Mark output contexts as non-seekable

2015-03-28 Thread Rodger Combs
This prevents sub-muxers from trying to seek back to the beginning of the whole stream, only to find themselves overwriting some video data in the current (often last) segment. --- libavformat/segment.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/segment.c b/libavformat/segm

[FFmpeg-devel] [PATCH 3/4] libavformat/segment: add an option to write the header to a separate file

2015-03-28 Thread Rodger Combs
This permits some interesting segmenting techniques with formats like Matroska, where you can concatenate the header and segments [N, nb_segments) and get a working file that starts at segment N's start time. --- libavformat/segment.c | 18 +++--- 1 file changed, 15 insertions(+), 3 de

[FFmpeg-devel] [PATCH 4/4] lavf/matroskaenc: don't try to end the segment when the output isn't seekable

2015-03-28 Thread Rodger Combs
--- libavformat/matroskaenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 6b2e390..c242a0e 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -1948,9 +1948,9 @@ static int mkv_write_trailer(A

Re: [FFmpeg-devel] [PATCH 2/4] libavdevice/avfoundation: add capture_screen_cursor option

2015-03-28 Thread Michael Niedermayer
On Sun, Mar 29, 2015 at 12:25:13AM +0100, Thilo Borgmann wrote: > Am 27.03.15 um 14:36 schrieb Matthieu Bouron: > > On Sat, Mar 21, 2015 at 5:35 PM, Thilo Borgmann > > wrote: > > [...] > > > >> I suggest to rename the option to "capture_cursor"... _screen_ in it seems > >> to be > >> redundant. >

Re: [FFmpeg-devel] [PATCH 1/4] libavdevice/avfoundation: add framerate and video size options

2015-03-28 Thread Michael Niedermayer
On Sun, Mar 29, 2015 at 12:27:54AM +0100, Thilo Borgmann wrote: > Am 27.03.15 um 14:34 schrieb Matthieu Bouron: > > On Sat, Mar 21, 2015 at 4:51 PM, Thilo Borgmann > > wrote: > > [...] > >> +[video_device setValue:selected_format forKey:@"activeFormat"]; > >> +[video_device setValu

Re: [FFmpeg-devel] [PATCH 4/4] libavdevice/avfoundation: use pts/dts provided by the CMSampleBuffer API

2015-03-28 Thread Michael Niedermayer
On Sun, Mar 29, 2015 at 12:26:31AM +0100, Thilo Borgmann wrote: > Am 21.03.15 um 18:45 schrieb Thilo Borgmann: > > Am 18.03.15 um 09:37 schrieb Matthieu Bouron: > >> On Fri, Mar 13, 2015 at 8:16 PM, wrote: > >> > >>> From: Matthieu Bouron > >>> > >>> --- > >>> libavdevice/avfoundation.m | 21 +++

Re: [FFmpeg-devel] [PATCH 3/4] libavdevice/avfoundation: add capture_screen_mouse_clicks option

2015-03-28 Thread Michael Niedermayer
On Sun, Mar 29, 2015 at 12:25:27AM +0100, Thilo Borgmann wrote: > Am 27.03.15 um 14:37 schrieb Matthieu Bouron: > > On Sat, Mar 21, 2015 at 6:47 PM, Thilo Borgmann > > wrote: > > [...] > > > >> > >> Please rename to "capture_mouse_clicks". > >> > > > > Updated in attached patch. > > Ok, please

Re: [FFmpeg-devel] [GSoC] Proof-of-concept HTTP Server

2015-03-28 Thread Stephan Holljes
I hope this addresses the issues mentioned. I added a new label in case of failure in http_open() in favor of duplicated code (i.e. calling av_dict_free() multiple times). I copied the style from the other functions. Signed-off-by: Stephan Holljes --- doc/protocols.texi | 3 +++ libavformat/htt

Re: [FFmpeg-devel] [PATCH] Add "split" mode to tinterlace filter.

2015-03-28 Thread Paul B Mahol
Dana 28. 3. 2015. 23:35 osoba "Brian Matherly" napisala je: > > From: Brian Matherly > > This mode is the opposite of the "merge" mode. > --- > This patch adds a new mode to tinterlace which performs the opposite operation > as the "merge" mode. > > My primary motivation is that I have been worki

Re: [FFmpeg-devel] [PATCH] Add "split" mode to tinterlace filter.

2015-03-28 Thread Brian Matherly
>> This mode is the opposite of the "merge" mode. >> --- >> This patch adds a new mode to tinterlace which performs the opposite operation >> as the "merge" mode. >> >> My primary motivation is that I have been working with Derek Buitenhuis to see >> about adding interlace support to the libx265 en

Re: [FFmpeg-devel] [PATCH 5/5] apng: Add a basic APNG muxer

2015-03-28 Thread Carl Eugen Hoyos
Donny Yang kota.moe> writes: > +{ "final_delay_num", "Force delay numerator after the last frame", > OFFSET(last_delay_num), > + AV_OPT_TYPE_INT, { .i64 = 0 }, 0, USHRT_MAX, ENC }, > +{ "final_delay_den", "Force delay denominator after the last frame", > OFFSET(last_delay_den), > +

Re: [FFmpeg-devel] [PATCH]Fix dnxhd pix_fmt change

2015-03-28 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > > dnxhddec.c |9 + > > 1 file changed, 5 insertions(+), 4 deletions(-) > > faf7809dc5b971e27c493e25e8f56a41e7743bd7 patchdnxhd.diff > > diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c > > LGTM The change was merged. Thank you, Carl

[FFmpeg-devel] [PATCH] tiff: Return more meaningful error codes

2015-03-28 Thread Himangi Saraogi
--- libavcodec/tiffenc.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/libavcodec/tiffenc.c b/libavcodec/tiffenc.c index 44cd956..a94b700 100644 --- a/libavcodec/tiffenc.c +++ b/libavcodec/tiffenc.c @@ -164,7 +164,8 @@ static int add_entry1(TiffEncoderContext *s,

Re: [FFmpeg-devel] Fwd: GSoC: APNG

2015-03-28 Thread Donny Yang
On 29 March 2015 at 03:15, Michael Niedermayer wrote: > you can try AVCodecContext->frame_number, if that doesnt work > you need to keep track of if you already had received a frame > I've tried both, but it turns out that AVCodecContext->frame_number seems to be thread-specific. That is, it star

Re: [FFmpeg-devel] [PATCH] tiff: Return more meaningful error codes

2015-03-28 Thread James Almer
On 29/03/15 1:18 AM, Himangi Saraogi wrote: > --- > libavcodec/tiffenc.c | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/libavcodec/tiffenc.c b/libavcodec/tiffenc.c > index 44cd956..a94b700 100644 > --- a/libavcodec/tiffenc.c > +++ b/libavcodec/tiffenc.c > @@

Re: [FFmpeg-devel] [PATCH 5/5] apng: Add a basic APNG muxer

2015-03-28 Thread Donny Yang
On 29 March 2015 at 14:02, Carl Eugen Hoyos wrote: > Donny Yang kota.moe> writes: > > > +{ "final_delay_num", "Force delay numerator after the last frame", > > OFFSET(last_delay_num), > > + AV_OPT_TYPE_INT, { .i64 = 0 }, 0, USHRT_MAX, ENC }, > > +{ "final_delay_den", "Force delay de