[vlc-devel] vlc: svn commit r22784 (funman)

Rémi Denis-Courmont rdenis at simphalempin.com
Mon Oct 22 18:24:38 CEST 2007

Le Monday 22 October 2007 05:01:59 Subversion daemon, vous avez écrit :
> r22784 | funman | 2007-10-22 04:01:55 +0200 (Mon, 22 Oct 2007) | 4 lines
> Changed paths:
>    M /trunk/include/vlc_configuration.h
>    M /trunk/modules/gui/qt4/components/simple_preferences.cpp
> configuration: add config_AddIntf() config_RemoveIntf() and
> config_ExistIntf() to handle extra interfaces in the preferences make use
> of it for last.fm simple preferences (ref #1262)
> by the way the duplication of "control" with "extraintf" is very boring

Please. Don't put such big functions into header files. Couldn't you see the 
warnings emitted by GCC (declared but not used) in just about every VLC file?

Rémi Denis-Courmont
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20071022/f9cd86c8/attachment.sig>

More information about the vlc-devel mailing list