[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