[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: KDE/kdelibs/plasma
From:       David Faure <faure () kde ! org>
Date:       2009-01-28 11:22:48
Message-ID: 200901281222.49523.faure () kde ! org
[Download RAW message or body]

On Wednesday 28 January 2009, Dan Meltzer wrote:
> On Tue, Jan 27, 2009 at 7:12 PM, Christophe Giboudeaux
> <cgiboudeaux@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@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
[prev in list] [next in list] [prev in thread] [next in thread]