[FFmpeg-devel] [PATCH 2/7] avformat/hlsenc: make the code easier to read

Nicolas Martyanoff khaelin at gmail.com
Wed Jul 30 10:25:45 CEST 2014


On 2014-07-29 16:15, Michael Niedermayer wrote:
> On Fri, Jul 18, 2014 at 10:57:42AM +0200, Nicolas Martyanoff wrote:
> > Before adding new features, I read the code and cleaned it. The main issue was
> > abstruse identifier names.
> > 
> > The behaviour of the muxer is *not* modified, by this patch, this is only
> > cosmetic. If this is not the case, it is a mistake.
> > ---
> >  libavformat/hlsenc.c | 360 ++++++++++++++++++++++++++++++---------------------
> >  1 file changed, 213 insertions(+), 147 deletions(-)
> 
> This should be ok if you add yourself to MAINTAINERs for hlsenc,
> that is you volunteer to maintain hlsenc.
> 
> Otherwise i think its better if we avoid large cosmetic changes so
> merges from any forks stay easy and non conflicting

Hi Michael,

I would be happy to help if bugs are found or if new features have to be
added, but I am not sure I have the time to commit to officially maintaining
the module.

Would that be satisfying ?

Regards,

-- 
Nicolas Martyanoff
http://wandrian.net
khaelin at gmail.com


More information about the ffmpeg-devel mailing list