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

[Rémi Denis-Courmont]

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
http://www.remlab.net/
-------------- 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>