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

[Lukasz Marek]

On 21 October 2014 12:08, Stefano Sabatini <stefasab at 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 not opposed, but please tell why you think it's better this way. I
> still hate the ffmpeg.c/ffmpeg_opt.c split, since it makes debugging
> and browsing the code much harder for me, also it increases the
> complexity since you have to declare otherwise static functions.


ffserver.c has almost 5k lines now. The Commit moves 1k to separate file
and there are only 2 calls in ffserver.c to moved code now.
Is it a matter of personal preferences, but handling 5k lines file is not
easy in general I think.
Especially when there is still some work to be done in parsing code so it
will grow a bit probably.