On 21.10.2014, at 12:15, Stefano Sabatini <stefa...@gmail.com> wrote:
> On date Monday 2014-10-20 23:57:01 +0200, Lukasz Marek encoded:
>> 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..22560ce 100644
>> --- a/ffserver.c
>> +++ b/ffserver.c
>> @@ -208,6 +208,12 @@ static FFServerConfig config = {
>> .logfilename = {0},
>> .http_addr = {0},
>> .rtsp_addr = {0},
>> + .errors = 0,
>> + .warnings = 0,
>> + .audio_id = AV_CODEC_ID_NONE,
>> + .video_id = AV_CODEC_ID_NONE,
>> + .audio_enc = {0},
>> + .video_enc = {0},
>> };
>
> Probably good, also I like the idea to split the big parsing function.
I was going to ask if it wouldn't me better to use memset or just a single {0}
to ensure that the whole struct is initialized at least and then assign the
values for which you want something else.
Though since it is "static" variable all that is kind of pointless since it
defaults to 0 initialization.
I which case I'd prefer if all the explicit initialization 0 was removed, it
feels like a lot of clutter to do effectively nothing.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
