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

Rafaël Carré funman at videolan.org
Tue Nov 29 09:33:31 CET 2011


Le Tue, 29 Nov 2011 09:29:41 +0100,
Jean-Baptiste Kempf <jb at videolan.org> a écrit :

> 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.

Not when $module requires lib$module IMVHO , that's why I wrote
'redundant' in the log

-- 
Rafaël Carré



More information about the vlc-devel mailing list