KDE/kdelibs/plasma

David Faure faure at kde.org
Wed Jan 28 12:22:48 CET 2009


On Wednesday 28 January 2009, Dan Meltzer wrote:
> On Tue, Jan 27, 2009 at 7:12 PM, Christophe Giboudeaux
> <cgiboudeaux at gmail.com> wrote:
> > SVN commit 917529 by cgiboudeaux:
> >
> > kaction.h is already included. Just remove the extra ones
> 
> Would it make sense, in terms of visual helpers--to not use camelcase
> for qt includes in kdelibs/plasma as well? I have a feeling that this
> would provide a style hint, as well as a reminder, to people about
> using "standard" includes.  Looking quickly at CamelCase qt includes
> provides a hint that they are acceptable, and might carry over into
> kde includes.. which arn't.

Well we do it in all the rest of kdelibs (CamelCase for Qt, lowercase for kde).
Not a big problem imho.

-- 
David Faure, faure at kde.org, sponsored by Qt Software @ Nokia to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).


More information about the Plasma-devel mailing list