[vlc-devel] [vlc-commits] configure.ac: remove redundant comments

Jean-Baptiste Kempf jb at videolan.org
Tue Nov 29 09:29:41 CET 2011


On Tue, Nov 29, 2011 at 07:32:04AM +0100, Rémi Denis-Courmont wrote :
> On Mon, 28 Nov 2011 16:22:42 -0500, Rafaël Carré <funman at videolan.org>
> wrote:
> > dnl
> > dnl  mpc demux plugin
> > dnl
> >  AC_ARG_ENABLE(mpc,
> > 
> > => the comments give no information at all
> 
> It *does*. Besides, it gives makes separation between parts of the code
> much easier to spot, especially with syntax highlighting.

The configure.ac is still too long, IMVHO and sometimes the comments are
longer than the actual code.
However, it does help readability and also helps to know that libsidsdq is
the package required for $module.

Best regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device



More information about the vlc-devel mailing list