On Wednesday 28 January 2009, Dan Meltzer wrote: > On Tue, Jan 27, 2009 at 7:12 PM, Christophe Giboudeaux > 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@kde.org, sponsored by Qt Software @ Nokia to work on KDE, Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org). _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel