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 support

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

[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 michae...@gmx.at --- 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 ---

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

2015-03-28 Thread Brian Matherly
From: Brian Matherly pez4br...@yahoo.com 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

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 thilo.borgm...@mail.de 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 wm4
On Sun, 29 Mar 2015 08:10:29 +1100 Peter Ross pr...@xvid.org 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.

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

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

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

2015-03-28 Thread Lukasz Marek
W dniu sobota, 28 marca 2015 Peter Ross pr...@xvid.org 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

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

2015-03-28 Thread Ivan Kalvachev
On 3/28/15, Peter Ross pr...@xvid.org wrote: On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: On Sun, 29 Mar 2015 08:10:29 +1100 Peter Ross pr...@xvid.org 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

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, matthieu.bou...@gmail.com wrote: From: Matthieu Bouron matthieu.bou...@gmail.com --- libavdevice/avfoundation.m | 21 +++-- 1 file changed, 15

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 thilo.borgm...@mail.de 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 thilo.borgm...@mail.de wrote: [...] Please rename to capture_mouse_clicks. Updated in attached patch. Ok, please apply. Thanks, Thilo ___

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 michae...@gmx.at 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 michae...@gmx.at wrote: On Thu, Mar 26, 2015 at 01:25:17AM +0100, Mariusz Szczepańczyk

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 thilo.borgm...@mail.de wrote: [...] I suggest to rename the option to capture_cursor... _screen_ in it seems to be

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 michaelni at 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.

[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 michae...@gmx.at --- 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

[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 michae...@gmx.at --- 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 ---

[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 michae...@gmx.at --- 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 ---

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

2015-03-28 Thread wm4
On Sun, 29 Mar 2015 01:12:32 +0200 Ivan Kalvachev ikalvac...@gmail.com wrote: On 3/28/15, Peter Ross pr...@xvid.org wrote: On Sat, Mar 28, 2015 at 10:24:55PM +0100, wm4 wrote: On Sun, 29 Mar 2015 08:10:29 +1100 Peter Ross pr...@xvid.org wrote: On Sat, Mar 28, 2015 at 08:38:40PM +0100,

[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

[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

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 thilo.borgm...@mail.de wrote: [...] +[video_device setValue:selected_format forKey:@activeFormat]; +[video_device

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

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 pr...@xvid.org 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

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

[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 michae...@gmx.at --- 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 @@

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

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

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 frequently

[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

[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

[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

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, matthieu.bou...@gmail.com wrote: From: Matthieu Bouron matthieu.bou...@gmail.com ---

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

2015-03-28 Thread Donny Yang
On 29 March 2015 at 03:15, Michael Niedermayer michae...@gmx.at 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

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

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 klaxa1...@googlemail.com --- doc/protocols.texi

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 thilo.borgm...@mail.de wrote: [...] Please rename to capture_mouse_clicks. Updated in attached patch. Ok, please

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 @@ -164,7

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

2015-03-28 Thread Carl Eugen Hoyos
Donny Yang work at 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] Add split mode to tinterlace filter.

2015-03-28 Thread Paul B Mahol
Dana 28. 3. 2015. 23:35 osoba Brian Matherly c...@brianmatherly.com napisala je: From: Brian Matherly pez4br...@yahoo.com 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

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 encoder. It turns

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

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 w...@kota.moe [...] -enc_row_size= deflateBound(s-zstream, row_size); -max_packet_size = avctx-height * (int64_t)(enc_row_size + - ((enc_row_size + IOBUF_SIZE

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

2015-03-28 Thread Carl Eugen Hoyos
Michael Niedermayer michaelni at 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,

[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] [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. Stekel

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 hls.c

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

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 michae...@gmx.at wrote: is there any advantage for multiple small IDATs ? if not i suggest to make the IDAT change to png as well in

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 w...@kota.moe --- libavcodec/pngenc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No great

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 w...@kota.moe --- libavcodec/pngenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No human being

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 michaelni at gmx.at writes: On Fri, Mar 27, 2015 at 12:21:00AM +, Carl Eugen Hoyos wrote: Michael Niedermayer michaelni at gmx.at

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 michaelni at gmx.at writes: On Fri, Mar 27, 2015 at 12:21:00AM +,

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

2015-03-28 Thread Donny Yang
Signed-off-by: Donny Yang w...@kota.moe --- 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

[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 w...@kota.moe --- 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 +++

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

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 michae...@gmx.at 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

[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 w...@kota.moe --- Changelog| 1 + doc/general.texi | 2 + libavformat/Makefile | 1 + libavformat/allformats.c | 2 +- libavformat/apngenc.c| 249

[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 michaelni at 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

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

2015-03-28 Thread Carl Eugen Hoyos
Himangi Saraogi himangi774 at 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:

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

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

[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 c...@passwd.hu --- libavfilter/af_channelmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/af_channelmap.c

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 michae...@gmx.at 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

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 a

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 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, but the

[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

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