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

Michael Niedermayer michaelni at gmx.at
Tue Jul 29 16:15:50 CEST 2014


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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The greatest way to live with honor in this world is to be what we pretend
to be. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140729/dcc42a01/attachment.asc>


More information about the ffmpeg-devel mailing list