[FFmpeg-devel] [PATCH 1/4] ffserver: move configuration code to separate file
Stefano Sabatini
stefasab at gmail.com
Tue Oct 21 15:40:54 CEST 2014
On date Tuesday 2014-10-21 13:18:01 +0200, Lukasz Marek encoded:
> 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.
I'm fine with whatever the maintainer will decide.
--
FFmpeg = Faithful Frightening Majestic Ponderous Extreme Guru
More information about the ffmpeg-devel
mailing list