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

List:       kde-devel
Subject:    Re: Patch for KTeaTime
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2007-08-22 17:28:08
Message-ID: 200708221128.08692.aseigo () kde ! org
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Wednesday 22 August 2007, David Jarvie wrote:
> On Wednesday 22 August 2007 8:37, Daniel Teske wrote:
> > Hi Stefan,
> >
> >>+SettingsDialog::SettingsDialog(TopLevel *toplevel, QList<Tea> &teas):
> >> KDialog()
> >>+{
> >>+    this->toplevel=toplevel;
> >
> > In genetal name your member variables with an m_ prefix. And use a
> > initialization list whereever possible.
>
> Use (or not) of a prefix for member variables is a matter of personal
> preference. It's useful when reading the code, but not obligatory.

outside of libs and apps with this standard (plasma and krunner being two such 
examples =) this is true. which is, i'm sure, what David meant by "in 
general" ...

> > We have a coding standard, but otherwise the formating is pretty much
> > already as it should be.
>
> The coding standard applies to KDE libraries only. Application authors are
> free to choose their own coding standard.

David is a kteatime developer, so i guess he qualifies ;)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech

[Attachment #5 (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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