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