[FFmpeg-devel] [PATCH 07/11] lavu/dict: add av_dict_serialize

2014-11-16 Thread Lukasz Marek
TODO: bump minor, update doc/APIchanges Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/dict.c | 27 +++ libavutil/dict.h | 16 2 files changed, 43 insertions(+) diff --git a/libavutil/dict.c b/libavutil/dict.c index 475e906..a41d61e

[FFmpeg-devel] [PATCH 09/11] lavf/ffmdec: add common options to recommended encoder configuration

2014-11-16 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmdec.c | 36 +++- 1 file changed, 31 insertions(+), 5 deletions(-) diff --git a/libavformat/ffmdec.c b/libavformat/ffmdec.c index 7ae906b..987f419 100644 --- a/libavformat/ffmdec.c +++ b

[FFmpeg-devel] [PATCH 10/11] lavf/ffmenc: store recommended encoder configuration

2014-11-16 Thread Lukasz Marek
ffmenc will store recommended encoder configuration if present. This will allow the user to base on local defaults and apply only explicitly set options. If recommended encoder configuration is not present, then non-default context's options are stored. Signed-off-by: Lukasz Marek lukasz.m.lu

[FFmpeg-devel] [PATCH 08/11] ffserver: export recommented encoder configuration

2014-11-16 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver.c| 6 +++-- ffserver_config.c | 77 ++- 2 files changed, 63 insertions(+), 20 deletions(-) diff --git a/ffserver.c b/ffserver.c index e24243d..3702fd6 100644

[FFmpeg-devel] [PATCH 11/11] ffmpeg_opt: make use of recommended encoder configuration

2014-11-16 Thread Lukasz Marek
So far ffmpeg used recommended configuration only for codec priv options. ffmpeg will use now codec defaults and then apply recommended configuration for all options. Recommended configuration possibly contains minimal set of options to filful user configuration. Signed-off-by: Lukasz Marek

Re: [FFmpeg-devel] [PATCH 03/11] ffserver_config: map ffserver options to AVOptions

2014-11-16 Thread Lukasz Marek
On 17.11.2014 02:46, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 237 -- ffserver_config.h | 2 - 2 files changed, 69 insertions(+), 170 deletions(-) @Reynaldo, you may check the test

Re: [FFmpeg-devel] [PATCH 11/11] ffmpeg_opt: make use of recommended encoder configuration

2014-11-16 Thread Lukasz Marek
On 17.11.2014 02:46, Lukasz Marek wrote: So far ffmpeg used recommended configuration only for codec priv options. ffmpeg will use now codec defaults and then apply recommended configuration for all options. Recommended configuration possibly contains minimal set of options to filful user

[FFmpeg-devel] [PATCH 1/2] lavc/options: set timecode_frame_start to -1 as option default

2014-11-15 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options.c | 1 - libavcodec/options_table.h | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/options.c b/libavcodec/options.c index 461e4a1..ae5e5d5 100644 --- a/libavcodec/options.c +++ b

[FFmpeg-devel] [PATCH 2/2] lavc/options: initialize pkt_timebase

2014-11-15 Thread Lukasz Marek
It's default in option_table.h is 0, but without this fix it is represented as 0/0. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/options.c b/libavcodec/options.c index ae5e5d5..5437770 100644

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-15 Thread Lukasz Marek
On 15.11.2014 19:59, Michael Niedermayer wrote: On Sat, Nov 15, 2014 at 02:32:26AM +0100, Lukasz Marek wrote: On 15.11.2014 01:24, Michael Niedermayer wrote: +/** + * String containing paris of key and values describing recommended encoder configuration. + * Paris are separated

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-15 Thread Lukasz Marek
On 15.11.2014 19:59, Michael Niedermayer wrote: On Sat, Nov 15, 2014 at 02:32:26AM +0100, Lukasz Marek wrote: On 15.11.2014 01:24, Michael Niedermayer wrote: +/** + * String containing paris of key and values describing recommended encoder configuration. + * Paris are separated

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-14 Thread Lukasz Marek
On 12.11.2014 02:12, Michael Niedermayer wrote: On Tue, Nov 11, 2014 at 11:59:32PM +0100, Lukasz Marek wrote: On 11.11.2014 14:46, Michael Niedermayer wrote: the muxer might run in a seperate thread from the encoder, priv_data could change while its being accessed, iam not sure if that would

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-14 Thread Lukasz Marek
On 15.11.2014 01:24, Michael Niedermayer wrote: +/** + * String containing paris of key and values describing recommended encoder configuration. + * Paris are separated by ','. + * Keys are separated from values by '='. + */ +char *recommended_encoder_configuration;

Re: [FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-13 Thread Lukasz Marek
as string now, not in binary form. I updated that and added that change to last commit which cause it. From 71c510818674890ea2957f6a896100094b86ab09 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Tue, 11 Nov 2014 08:20:02 +0100 Subject: [PATCH] lavf/ffm: use AVOption API

Re: [FFmpeg-devel] [PATCH 1/9] lavu/opt: introduce av_opt_is_set_to_default()

2014-11-12 Thread Lukasz Marek
On 12.11.2014 09:03, Nicolas George wrote: Le primidi 21 brumaire, an CCXXIII, Lukasz Marek a écrit : TODO: bump minor version, update doc/APIchanges New function allows to check if option is set to its default I am a bit uncomfortable with that API: this function does not test whether

Re: [FFmpeg-devel] [PATCH 1/9] lavu/opt: introduce av_opt_is_set_to_default()

2014-11-12 Thread Lukasz Marek
On 12 November 2014 09:16, Nicolas George geo...@nsup.org wrote: Le primidi 21 brumaire, an CCXXIII, Lukasz Marek a écrit : I dont agree. Assert cannot be placed. Return value is not checked is set_defaults so errors are ignored. It can just leave invalid value. Assert here would terminate

Re: [FFmpeg-devel] [PATCH] lavc/options: set all codec options to defaults

2014-11-12 Thread Lukasz Marek
On 11.11.2014 18:30, Lukasz Marek wrote: avcodec_get_context_defaults3() sets only specific options to default the values. This commit force all options to be set to the defaults. This improves effectiveness of new ffm format. In particular it fixes bt option being out of range when context

Re: [FFmpeg-devel] [PATCH] ffserver_config: report not closed last tag

2014-11-12 Thread Lukasz Marek
On 09.11.2014 04:52, Reynaldo H. Verdejo Pinochet wrote: Looks good. Thanks. Feel free to push. Pushed ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavu/opt: fix av_opt_get function

2014-11-12 Thread Lukasz Marek
On 12.11.2014 00:48, Michael Niedermayer wrote: On Tue, Nov 11, 2014 at 09:18:44PM +0100, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) LGTM Pushed this one. You may also

Re: [FFmpeg-devel] [PATCH] lavc/options: set all codec options to defaults

2014-11-12 Thread Lukasz Marek
On 12.11.2014 23:05, Michael Niedermayer wrote: On Tue, Nov 11, 2014 at 06:30:39PM +0100, Lukasz Marek wrote: avcodec_get_context_defaults3() sets only specific options to default the values. This commit force all options to be set to the defaults. This improves effectiveness of new ffm format

Re: [FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-12 Thread Lukasz Marek
On 12.11.2014 03:19, Michael Niedermayer wrote: On Wed, Nov 12, 2014 at 12:02:07AM +0100, Lukasz Marek wrote: On 11.11.2014 08:31, Lukasz Marek wrote: This is a generic solution that will not reqiore modifications when new options are added. This also fixes problem with current implementation

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-12 Thread Lukasz Marek
On 12.11.2014 20:32, Michael Niedermayer wrote: On Wed, Nov 12, 2014 at 06:31:52PM +0100, Lukasz Marek wrote: On 12 November 2014 17:53, Michael Niedermayer michae...@gmx.at wrote: On Wed, Nov 12, 2014 at 05:47:01PM +0100, Stefano Sabatini wrote: On date Wednesday 2014-11-12 09:09:43 +0100

Re: [FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-12 Thread Lukasz Marek
W dniu czwartek, 13 listopada 2014 Michael Niedermayer michae...@gmx.at napisał(a): On Thu, Nov 13, 2014 at 12:50:38AM +0100, Lukasz Marek wrote: On 12.11.2014 03:19, Michael Niedermayer wrote: On Wed, Nov 12, 2014 at 12:02:07AM +0100, Lukasz Marek wrote: On 11.11.2014 08:31, Lukasz Marek

[FFmpeg-devel] [PATCH] lavc/options: set all codec options to defaults

2014-11-11 Thread Lukasz Marek
, when min=1) Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/options.c b/libavcodec/options.c index 461e4a1..bb472a9 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -104,6 +104,9 @@ int

Re: [FFmpeg-devel] [PATCH] lavc/options: set all codec options to defaults

2014-11-11 Thread Lukasz Marek
On 11.11.2014 18:30, Lukasz Marek wrote: avcodec_get_context_defaults3() sets only specific options to default the values. This commit force all options to be set to the defaults. This improves effectiveness of new ffm format. In particular it fixes bt option being out of range when context

Re: [FFmpeg-devel] [PATCH 1/9] lavu/opt: introduce av_opt_is_set_to_default()

2014-11-11 Thread Lukasz Marek
82677357d1033b384ded053ab0e55c4f02183c7f Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Mon, 10 Nov 2014 22:25:30 +0100 Subject: [PATCH] lavu/opt: introduce av_opt_is_set_to_default() TODO: bump minor version, update doc/APIchanges New function allows to check if option is set to its default value Signed

Re: [FFmpeg-devel] [PATCH 1/4] lavu/opt: check for NULL before parsing

2014-11-11 Thread Lukasz Marek
On 09.11.2014 23:22, Lukasz Marek wrote: set_string_binary crashes with called with val=NULL Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) Pushed whole patchset

Re: [FFmpeg-devel] [PATCH 5/9] ffmpeg_opt: use codec private context in ffserver streams

2014-11-11 Thread Lukasz Marek
On 11.11.2014 18:03, Michael Niedermayer wrote: On Tue, Nov 11, 2014 at 04:41:17PM +0100, Lukasz Marek wrote: On 11.11.2014 08:31, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 48 1 file changed

Re: [FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-11 Thread Lukasz Marek
that priv_data starts with a AVClass Fixed patch attached From af7c72d8d62e26c4b8fe9d8a758707532e727830 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Mon, 10 Nov 2014 23:22:59 +0100 Subject: [PATCH 1/2] lavf/ffm: store/restore private codec context TODO: bump minor

Re: [FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-11 Thread Lukasz Marek
On 11.11.2014 08:31, Lukasz Marek wrote: This is a generic solution that will not reqiore modifications when new options are added. This also fixes problem with current implementation when qmin or qmax=-1. Only 8 bits was sent and read back as 255. Fixes #1275 Fixes #1461 Signed-off

Re: [FFmpeg-devel] [PATCH 1/4] ffserver_config: drop requirement video size being multiple of 16

2014-11-10 Thread Lukasz Marek
On 01.11.2014 22:20, Reynaldo H. Verdejo Pinochet wrote: Hi On 10/31/2014 11:00 PM, Lukasz Marek wrote: [..] diff --git a/ffserver_config.c b/ffserver_config.c index d589ff2..f11db03 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -819,8 +819,6 @@ static int

[FFmpeg-devel] [PATCH 3/9] lavf/ffm: store/restore private codec context

2014-11-10 Thread Lukasz Marek
TODO: bump micro Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmdec.c | 41 - libavformat/ffmenc.c | 27 +-- 2 files changed, 61 insertions(+), 7 deletions(-) diff --git a/libavformat/ffmdec.c b/libavformat

[FFmpeg-devel] [PATCH 4/9] ffmpeg_opt: allow to force codec in new_output_stream

2014-11-10 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 23 --- 1 file changed, 12 insertions(+), 11 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 77ef0c4..1b3f73a 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -1036,11 +1036,12 @@ static int

[FFmpeg-devel] [PATCH 5/9] ffmpeg_opt: use codec private context in ffserver streams

2014-11-10 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 48 1 file changed, 32 insertions(+), 16 deletions(-) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index 1b3f73a..fdbab58 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -1618,6

[FFmpeg-devel] [PATCH 6/9] ffserver_config: handle codec private options

2014-11-10 Thread Lukasz Marek
This commit allows to set codec's private option. As side effect this commit also improves preset support. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- Changelog | 1 + doc/ffserver.texi | 10 ++- ffserver_config.c | 219

[FFmpeg-devel] [PATCH 8/9] bavcodec/options_table: add pixel_format and video_size options

2014-11-10 Thread Lukasz Marek
Adding these options simplify ffm format implementation based on AVOption API. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options_table.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h index beb767c..5dd0359

[FFmpeg-devel] [PATCH 7/9] libavcodec/options_table: set min to -1 for timecode_frame_start

2014-11-10 Thread Lukasz Marek
timecode_frame_start is set to -1 in avcodec_get_context_defaults3() AVOptions API complains about it. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavcodec/options_table.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/options_table.h b/libavcodec

[FFmpeg-devel] [PATCH 9/9] lavf/ffm: use AVOption API to store/restore stream properties

2014-11-10 Thread Lukasz Marek
This is a generic solution that will not reqiore modifications when new options are added. This also fixes problem with current implementation when qmin or qmax=-1. Only 8 bits was sent and read back as 255. Fixes #1275 Fixes #1461 Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com

Re: [FFmpeg-devel] [PATCH] Fixes for the Icecast protocol

2014-11-10 Thread Lukasz Marek
W dniu wtorek, 11 listopada 2014 Marvin Scholz epira...@gmail.com napisał(a): Please ignore the patches I sent before ([PATCH 1/2] ffserver_config: drop requirement videosize being multiple of 16), git send-email did something unexpected and somehow picked the wrong ones… This mailing list

Re: [FFmpeg-devel] [PATCH 2/4] lavu/opt: document av_opt_copy function

2014-11-09 Thread Lukasz Marek
On 09.11.2014 07:15, wm4 wrote: On Sat, 8 Nov 2014 18:13:51 +0100 Lukasz Marek lukasz.m.lu...@gmail.com wrote: --- libavutil/opt.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavutil/opt.h b/libavutil/opt.h index b9ac932..b78810c 100644 --- a/libavutil/opt.h +++ b

[FFmpeg-devel] [PATCH 3/4] lavu/opt: update tests

2014-11-09 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 81 - 1 file changed, 68 insertions(+), 13 deletions(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index f0798a1..58f745a 100644 --- a/libavutil/opt.c +++ b

[FFmpeg-devel] [PATCH 4/4] fate: add opt-test

2014-11-09 Thread Lukasz Marek
--- tests/fate/libavutil.mak | 4 +++ tests/ref/fate/opt | 90 2 files changed, 94 insertions(+) diff --git a/tests/fate/libavutil.mak b/tests/fate/libavutil.mak index 63a4dd6..eadebdb 100644 --- a/tests/fate/libavutil.mak +++

[FFmpeg-devel] [PATCH 1/4] lavu/opt: check for NULL before parsing

2014-11-09 Thread Lukasz Marek
set_string_binary crashes with called with val=NULL Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavutil/opt.c b/libavutil/opt.c index fca5354..bc62044 100644 --- a/libavutil/opt.c +++ b

[FFmpeg-devel] [PATCH 2/4] lavu/opt: add support for binary defaults

2014-11-09 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c| 2 ++ tests/ref/fate/opt | 0 2 files changed, 2 insertions(+) create mode 100644 tests/ref/fate/opt diff --git a/libavutil/opt.c b/libavutil/opt.c index bc62044..f0798a1 100644 --- a/libavutil/opt.c +++ b/libavutil

Re: [FFmpeg-devel] [PATCH 4/4] fate: add opt-test

2014-11-09 Thread Lukasz Marek
On 09.11.2014 23:22, Lukasz Marek wrote: --- tests/fate/libavutil.mak | 4 +++ tests/ref/fate/opt | 90 2 files changed, 94 insertions(+) diff --git a/tests/fate/libavutil.mak b/tests/fate/libavutil.mak index 63a4dd6..eadebdb 100644

[FFmpeg-devel] [PATCH 2/4] lavu/opt: document av_opt_copy function

2014-11-08 Thread Lukasz Marek
--- libavutil/opt.h | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavutil/opt.h b/libavutil/opt.h index b9ac932..b78810c 100644 --- a/libavutil/opt.h +++ b/libavutil/opt.h @@ -815,6 +815,16 @@ void av_opt_freep_ranges(AVOptionRanges **ranges); */ int

[FFmpeg-devel] [PATCH 4/4] lavu/opt: copy dict in av_opt_copy

2014-11-08 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavutil/opt.c b/libavutil/opt.c index 0692393..86d3ddf 100644 --- a/libavutil/opt.c +++ b/libavutil/opt.c @@ -1596,6 +1596,13 @@ int av_opt_copy(void *dst, void

[FFmpeg-devel] [PATCH 3/4] lavu/opt: explicity set NULL as dict/binary defaults

2014-11-08 Thread Lukasz Marek
Unless user clears context manually during creation, binary and dict pointer may contain random values. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/opt.c b/libavutil/opt.c index 4116f96..0692393 100644

[FFmpeg-devel] [PATCH] ffserver_config: report not closed last tag

2014-11-08 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ffserver_config.c b/ffserver_config.c index c05c942..2ca484f 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -1303,6 +1303,8 @@ int ffserver_parse_ffconfig

Re: [FFmpeg-devel] [PATCH 4/4] lavu/opt: copy dict in av_opt_copy

2014-11-08 Thread Lukasz Marek
On 08.11.2014 19:36, Michael Niedermayer wrote: On Sat, Nov 08, 2014 at 06:13:53PM +0100, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavutil/opt.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavutil/opt.c b/libavutil/opt.c index 0692393

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: set correct priv_data for feed stream

2014-11-07 Thread Lukasz Marek
On 6 November 2014 22:55, Michael Niedermayer michae...@gmx.at wrote: On Thu, Nov 06, 2014 at 09:27:45PM +0100, Lukasz Marek wrote: On 06.11.2014 01:32, Michael Niedermayer wrote: On Thu, Nov 06, 2014 at 12:59:45AM +0100, Lukasz Marek wrote: new_output_stream creates a codec context

[FFmpeg-devel] [PATCH] ffmpeg_opt: set correct priv_data for feed stream

2014-11-05 Thread Lukasz Marek
new_output_stream creates a codec context with arbitraty picked codec. Later data is updated, but priv_data are left alone. There is a bit chance there is a mismatch between codecs. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffmpeg_opt.c | 16 libavformat

Re: [FFmpeg-devel] FFserver bug?

2014-11-05 Thread Lukasz Marek
On 05.11.2014 02:15, Lukasz Marek wrote: On 31.10.2014 02:14, Sevan Gelici wrote: Hi, Two days ago i updated my linux system and also ffmpeg was updated. When i started to stream it was not working anymore. I am using ffserver. i determined the problem which causes the problem thats

Re: [FFmpeg-devel] FFserver bug?

2014-11-04 Thread Lukasz Marek
On 31.10.2014 03:19, Michael Niedermayer wrote: On Fri, Oct 31, 2014 at 02:14:29AM +0100, Sevan Gelici wrote: Hi, Two days ago i updated my linux system and also ffmpeg was updated. When i started to stream it was not working anymore. I am using ffserver. i determined the problem which causes

Re: [FFmpeg-devel] FFserver bug?

2014-11-04 Thread Lukasz Marek
On 05.11.2014 00:07, Lukasz Marek wrote: Maybe worth to mention I recently created function to dump AvCodecContext, to test if ffm is transmitting context properly. And strange thing I noticed ffm demuxer's read_header callback is called only once, even though there are 2 streams. I guess

Re: [FFmpeg-devel] FFserver bug?

2014-11-04 Thread Lukasz Marek
releases so cherry-pick would not be enough tho, to fix it on previous release Fix patches attached. From 9c1455bd9e830aedba645451b7c0cb44831658e2 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Wed, 5 Nov 2014 02:10:24 +0100 Subject: [PATCH 1/2] lavc/options: don't free

Re: [FFmpeg-devel] FFserver bug?

2014-11-04 Thread Lukasz Marek
W dniu środa, 5 listopada 2014 Michael Niedermayer michae...@gmx.at napisał(a): On Wed, Nov 05, 2014 at 12:07:14AM +0100, Lukasz Marek wrote: On 31.10.2014 03:19, Michael Niedermayer wrote: On Fri, Oct 31, 2014 at 02:14:29AM +0100, Sevan Gelici wrote: Hi, Two days ago i updated my

Re: [FFmpeg-devel] [PATCH 2/2] [RFC]lavf/ffm: update with more options.

2014-11-03 Thread Lukasz Marek
On 3 November 2014 01:31, Michael Niedermayer michae...@gmx.at wrote: On Sun, Nov 02, 2014 at 07:19:14PM +0100, Lukasz Marek wrote: shouldnt all these values be checked to be within the allowed ranges for the corresponding AVOptions? being able to set so many field so arbitrarily could

[FFmpeg-devel] [PATCH 1/2] lavf/ffm: silent deprecated warnings

2014-11-02 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmdec.c | 11 +++ libavformat/ffmenc.c | 5 + 2 files changed, 16 insertions(+) diff --git a/libavformat/ffmdec.c b/libavformat/ffmdec.c index 448762b..e95c139 100644 --- a/libavformat/ffmdec.c +++ b/libavformat

[FFmpeg-devel] [PATCH 2/2] [RFC]lavf/ffm: update with more options.

2014-11-02 Thread Lukasz Marek
TODO: bump micro Many common codec options are not via ffm protocol. This commit adds common A/V encoding options to protocol. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavformat/ffmdec.c | 78 libavformat/ffmenc.c | 60

Re: [FFmpeg-devel] FFserver bug?

2014-11-02 Thread Lukasz Marek
On 02.11.2014 23:34, Sevan Gelici wrote: I still don't have tested old gits. But i will try to expain. A half year ago i installed ffmpeg and i made a configuration file. When i start this bitrate was going up to 2000kbs max. I updated and now it will go higher like its sum up or something, in

Re: [FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-02 Thread Lukasz Marek
On 02.11.2014 22:37, Reynaldo H. Verdejo Pinochet wrote: If you are always returning 0 then this func should probably be declared static void instead? I left it as int for future, but probably no reason for that, so changed. [..] diff --git a/ffserver_config.h b/ffserver_config.h index

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-11-02 Thread Lukasz Marek
On 2 November 2014 00:02, Reynaldo H. Verdejo Pinochet reyna...@osg.samsung.com wrote: On 11/01/2014 07:59 PM, Lukasz Marek wrote: [..] I decreased it by 1 automatically rather than for any reason. I didn't want to change logic where it was not needed, and it was not needed here. I

Re: [FFmpeg-devel] [PATCH 2/2] [RFC]lavf/ffm: update with more options.

2014-11-02 Thread Lukasz Marek
On 03.11.2014 00:40, Stefano Sabatini wrote: On date Sunday 2014-11-02 19:19:14 +0100, Lukasz Marek encoded: TODO: bump micro Many common codec options are not via ffm protocol. This commit adds common A/V encoding options to protocol. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com

Re: [FFmpeg-devel] [PATCH] configure: add -Werror=declaration-after-statement for gcc

2014-11-01 Thread Lukasz Marek
On 01.11.2014 09:48, Nicolas George wrote: Le primidi 11 brumaire, an CCXXIII, Lukasz Marek a écrit : Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- configure | 1 + 1 file changed, 1 insertion(+) Did something change since 377dfa3d? Hmm, it can be annoying during debugging

Re: [FFmpeg-devel] OPW Qualification Task: Enable daemon mode for FFserver

2014-11-01 Thread Lukasz Marek
On 01.11.2014 17:20, Binathi Bingi wrote: +if (config.ffserver_daemon) { +int ffserver_id = 0; You may change int to pid_t too, which is actually returned by fork() +pid_t sid = 0; + +ffserver_id = fork(); ___

[FFmpeg-devel] [PATCH] ffserver_config: improve AVOption handing

2014-11-01 Thread Lukasz Marek
AVOption are checked right after found in config file. It allows to report exact line in config file. Options provided more than once are threated as errors. It also fixes flag options handing. Flags may occur more than once in config file. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-11-01 Thread Lukasz Marek
On 01.11.2014 22:27, Reynaldo H. Verdejo Pinochet wrote: Patch seems OK but I'm missing on the rationalle behind dropping the preset support. Can you elaborate? Patch 1 and 2 are just suggestion from previous reviews by Stefano. Regarding presents itself I have no pressure to remove it, but I

Re: [FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-11-01 Thread Lukasz Marek
On 01.11.2014 23:06, Reynaldo H. Verdejo Pinochet wrote: Hi On 10/31/2014 11:00 PM, Lukasz Marek wrote: [..] @@ -356,9 +356,7 @@ static int ffserver_parse_config_global(FFServerConfig *config, const char *cmd, if (!av_strcasecmp(cmd, Port)) WARNING(Port option

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: dont pass pointer to enum as pointer to int

2014-11-01 Thread Lukasz Marek
On 1 November 2014 23:07, Reynaldo H. Verdejo Pinochet reyna...@osg.samsung.com wrote: Looks OK. Feel free to push. Pushed this one ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-11-01 Thread Lukasz Marek
On 2 November 2014 00:15, Reynaldo H. Verdejo Pinochet reyna...@osg.samsung.com wrote: Hi On 11/01/2014 07:42 PM, Lukasz Marek wrote: [..] Patch 1 and 2 are just suggestion from previous reviews by Stefano. Regarding presents itself I have no pressure to remove it, but I agree

Re: [FFmpeg-devel] FFserver bug?

2014-10-31 Thread Lukasz Marek
On 31 October 2014 03:19, Michael Niedermayer michae...@gmx.at wrote: On Fri, Oct 31, 2014 at 02:14:29AM +0100, Sevan Gelici wrote: Hi, Two days ago i updated my linux system and also ffmpeg was updated. When i started to stream it was not working anymore. I am using ffserver. i

Re: [FFmpeg-devel] [PATCH] lavd/alsa: implement get_device_list callbacks

2014-10-31 Thread Lukasz Marek
On 27.10.2014 23:12, Lukasz Marek wrote: On 26.10.2014 23:27, Michael Niedermayer wrote: +/* ported from alsa-utils/aplay.c */ +int ff_alsa_get_device_list(AVDeviceInfoList *device_list, snd_pcm_stream_t stream_type) +{ +int ret = 0; +void **hints, **n; +char *name = NULL, *descr

[FFmpeg-devel] [PATCH] configure: add -Werror=declaration-after-statement for gcc

2014-10-31 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 40d1f07..3322779 100755 --- a/configure +++ b/configure @@ -5227,6 +5227,7 @@ elif enabled gcc; then check_cflags -Werror=missing-prototypes

[FFmpeg-devel] [PATCH 1/4] ffserver_config: drop requirement video size being multiple of 16

2014-10-31 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index d589ff2..f11db03 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -819,8 +819,6 @@ static int

[FFmpeg-devel] [PATCH 2/4] ffserver_config: drop presets support

2014-10-31 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 80 +-- ffserver_config.h | 2 -- 2 files changed, 1 insertion(+), 81 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index f11db03..6989bd8 100644

[FFmpeg-devel] [PATCH 3/4] ffserver_config: improve error handling

2014-10-31 Thread Lukasz Marek
Replace atoi with more advanced parsing routine. Set maximum port value to 65535 (not 65536). Other checks. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 48 +++- 1 file changed, 27 insertions(+), 21 deletions(-) diff

[FFmpeg-devel] [PATCH 4/4] ffserver_config: dont pass pointer to enum as pointer to int

2014-10-31 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index c694934..045b791 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -541,8 +541,11 @@ static int

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: postpone codec context creation

2014-10-25 Thread Lukasz Marek
*audio_opts; /* Contains AVOptions for audio encoder */ +AVDictionary *audio_conf; /* Contains values stored in audio AVCodecContext.fields */ Would drop the repeated Contains. Dropped. From ab5395a62b60cedd47d9e6894b685c29a8c87f3d Mon Sep 17 00:00:00 2001 From: Lukasz Marek

Re: [FFmpeg-devel] [PATCH] lavd/alsa: implement get_device_list callbacks

2014-10-24 Thread Lukasz Marek
On 23.10.2014 14:28, Nicolas George wrote: Le primidi 1er brumaire, an CCXXIII, Lukasz Marek a écrit : It gives the same list as aplay -L (upper case). For example (ffmpeg will display it in different way, but the content is the same) I thought for a time that dmix:CARD=PCH,DEV=0 had

[FFmpeg-devel] [PATCH] doc/fftools-common-opts: document -devices option

2014-10-24 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- doc/fftools-common-opts.texi | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/doc/fftools-common-opts.texi b/doc/fftools-common-opts.texi index 18027d2..c0bfb7e 100644 --- a/doc/fftools-common-opts.texi +++ b/doc

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: postpone codec context creation

2014-10-22 Thread Lukasz Marek
ca5e53e39254ce8ed205598b6ffc3ef0a3a41138 Mon Sep 17 00:00:00 2001 From: Lukasz Marek lukasz.m.lu...@gmail.com Date: Sun, 19 Oct 2014 21:29:40 +0200 Subject: [PATCH] ffserver_config: postpone codec context creation So far AVCodecContext was created without codec specified. This causes internal data

Re: [FFmpeg-devel] [PATCH 1/4] ffserver: move configuration code to separate file

2014-10-21 Thread Lukasz Marek
On 21 October 2014 12:08, Stefano Sabatini stefa...@gmail.com wrote: On date Monday 2014-10-20 23:56:59 +0200, Lukasz Marek encoded: This commit doesn't change any existing logic. It moves ffserver configuration related code to separate file. It intends to make maintaining easier. I'm

Re: [FFmpeg-devel] [PATCH] configure: add build tree's directories to rpath

2014-10-21 Thread Lukasz Marek
On 21 October 2014 12:01, Derek Buitenhuis derek.buitenh...@gmail.com wrote: On 10/21/2014 8:21 AM, Carl Eugen Hoyos wrote: This sounds like a very bad idea to me I agree. It's not nice to have remnants of the build process in the installed binaries. If you need to run them during

Re: [FFmpeg-devel] [PATCH] lavd/alsa: implement get_device_list callbacks

2014-10-21 Thread Lukasz Marek
On 19.10.2014 16:27, wm4 wrote: On Sat, 18 Oct 2014 20:36:22 +0200 Lukasz Marek lukasz.m.lu...@gmail.com wrote: --- libavdevice/alsa-audio-common.c | 60 + libavdevice/alsa-audio-dec.c| 6 + libavdevice/alsa-audio-enc.c| 6

Re: [FFmpeg-devel] [PATCH 1/2] lavd/fbdev_common: report error during probing fbdev device

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:28, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 08:25:29PM +0200, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavdevice/fbdev_common.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) LGTM Pushed

Re: [FFmpeg-devel] [PATCH] lavd/fbdev_dec: use default device when not provided

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:27, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 07:38:16PM +0200, Lukasz Marek wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavdevice/fbdev_dec.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) LGTM Pushed

Re: [FFmpeg-devel] [PATCH 2/2] opts: add list device sources/sinks options

2014-10-21 Thread Lukasz Marek
On 19.10.2014 00:20, Michael Niedermayer wrote: On Sat, Oct 18, 2014 at 08:25:30PM +0200, Lukasz Marek wrote: Allows to list sources/sinks of the devices that implement that functionality. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com [...] +int show_sinks(void *optctx, const char

Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-21 Thread Lukasz Marek
On 21.10.2014 08:59, James Almer wrote: On 20/10/14 5:03 PM, Lukasz Marek wrote: On 20.10.2014 14:55, Michael Niedermayer wrote: On Mon, Oct 20, 2014 at 12:14:14PM +0200, Lukasz Marek wrote: On 20 October 2014 08:47, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 19.10.2014, at 13:39

Re: [FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-20 Thread Lukasz Marek
On 20 October 2014 08:47, Reimar Döffinger reimar.doeffin...@gmx.de wrote: On 19.10.2014, at 13:39, Lukasz Marek lukasz.m.lu...@gmail.com wrote: Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavdevice/libavdevice.v | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[FFmpeg-devel] [PATCH 4/4] ffserver_config: postpone codec context creation

2014-10-20 Thread Lukasz Marek
So far AVCodecContext was created without codec specified. This causes internal data to not be initialized to defaults. This commit postpone context creation until all information are gathered. Partially fixes #1275 --- ffserver.c| 8 +- ffserver_config.c | 286

[FFmpeg-devel] [PATCH 2/4] ffserver_config: clean up variable initialization

2014-10-20 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver_config.c | 16 +--- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/ffserver_config.c b/ffserver_config.c index 927b6fe..351c4e9 100644 --- a/ffserver_config.c +++ b/ffserver_config.c @@ -341,11 +341,11

[FFmpeg-devel] [PATCH 3/4] ffserver_conf: factorize parse function per config tag

2014-10-20 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- ffserver.c|6 + ffserver_config.c | 1138 ++--- ffserver_config.h |7 + 3 files changed, 566 insertions(+), 585 deletions(-) diff --git a/ffserver.c b/ffserver.c index af6e8e1

Re: [FFmpeg-devel] [PATCH 4/4] ffserver_config: postpone codec context creation

2014-10-20 Thread Lukasz Marek
On 20.10.2014 23:57, Lukasz Marek wrote: So far AVCodecContext was created without codec specified. This causes internal data to not be initialized to defaults. This commit postpone context creation until all information are gathered. Partially fixes #1275 --- ffserver.c| 8

Re: [FFmpeg-devel] [PATCH] configure: add build tree's directories to rpath

2014-10-20 Thread Lukasz Marek
W dniu wtorek, 21 października 2014 Carl Eugen Hoyos ceho...@ag.or.at napisał(a): Lukasz Marek lukasz.m.luki2 at gmail.com writes: This commit allows to run programs linked dynamically in build dir without installing. Sorry if I misunderstand: Doesn't this patch change the properties

[FFmpeg-devel] [PATCH] lavd: export all symbols with av prefix

2014-10-19 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavdevice/libavdevice.v | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavdevice/libavdevice.v b/libavdevice/libavdevice.v index 663af85..d7c86ba 100644 --- a/libavdevice/libavdevice.v +++ b/libavdevice

[FFmpeg-devel] [PATCH] lavd/fbdev_dec: use default device when not provided

2014-10-18 Thread Lukasz Marek
Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- libavdevice/fbdev_dec.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavdevice/fbdev_dec.c b/libavdevice/fbdev_dec.c index 1593b10..7b13a85 100644 --- a/libavdevice/fbdev_dec.c +++ b/libavdevice

Re: [FFmpeg-devel] [PATCH 3/3] opts: add list device sources/sinks options

2014-10-18 Thread Lukasz Marek
On 13.08.2014 16:00, Michael Niedermayer wrote: On Thu, Aug 07, 2014 at 01:58:57AM +0200, Lukasz Marek wrote: Allows to list sources/sinks of the devices that implement that functionality. [...] +int show_sinks(void *optctx, const char *opt, const char *arg) +{ +#if CONFIG_AVDEVICE

[FFmpeg-devel] [PATCH 2/2] opts: add list device sources/sinks options

2014-10-18 Thread Lukasz Marek
Allows to list sources/sinks of the devices that implement that functionality. Signed-off-by: Lukasz Marek lukasz.m.lu...@gmail.com --- cmdutils.c | 191 + cmdutils.h | 12 cmdutils_common_opts.h | 6 ++ 3 files

[FFmpeg-devel] [PATCH] lavd/alsa: implement get_device_list callbacks

2014-10-18 Thread Lukasz Marek
--- libavdevice/alsa-audio-common.c | 60 + libavdevice/alsa-audio-dec.c| 6 + libavdevice/alsa-audio-enc.c| 6 + libavdevice/alsa-audio.h| 2 ++ 4 files changed, 74 insertions(+) diff --git a/libavdevice/alsa-audio-common.c

<    1   2   3   4   >