From kde-devel  Mon May 07 17:21:19 2012
From: Andreas Pakulat <apaku () gmx ! de>
Date: Mon, 07 May 2012 17:21:19 +0000
To: kde-devel
Subject: Re: Several new utilities for KDE: KLook and StackFolder
Message-Id: <CAExHGmQrmg=y7irUNLbtqfvoajb1iT_YFqqH-dhNKja1ebNeGw () mail ! gmail ! com>
X-MARC-Message: https://marc.info/?l=kde-devel&m=133641143923336
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--===============0619915389680437461=="

--===============0619915389680437461==
Content-Type: multipart/alternative; boundary=20cf303b42399a31e004bf757f0e

--20cf303b42399a31e004bf757f0e
Content-Type: text/plain; charset=ISO-8859-1

Hi,

On Mon, May 7, 2012 at 7:15 PM, David Edmundson
<david@davidedmundson.co.uk>wrote:

> Good to see these merged into playground!
>
> Some things we would need to proposing for KDE extragear/core
> - Watch your coding style
>  http://techbase.kde.org/Policies/Kdelibs_Coding_Style
>  In particular:
>    Always use braces
>    Don't call classes "MySomething". As the "My" isn't very helpful
>    Make sure the files actually match the names of the class.
> audio.h, video.h, text.h
>    MyAudio::~MyAudio() use m_mediaObject->deleteLater(); instead of
> delete (it's safer). Alternatively set a parent on it in the
> constructor and don't manually delete it.
>   Technically your whitespace is off in many ways too, but that's
> being pedantic.
>

Unlike what many people think the mentioned web-page is the coding-style
for kdelibs only. Other kde projects/modules may choose to adopt that style
or parts of the style, but its not mandatory for any module to follow it.
You're always free to choose whatever style suits you.

Andreass

--20cf303b42399a31e004bf757f0e
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Hi,<br><br><div class=3D"gmail_quote">On Mon, May 7, 2012 at 7:15 PM, David=
 Edmundson <span dir=3D"ltr">&lt;<a href=3D"mailto:david@davidedmundson.co.=
uk" target=3D"_blank">david@davidedmundson.co.uk</a>&gt;</span> wrote:<br><=
blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px=
 #ccc solid;padding-left:1ex">
Good to see these merged into playground!<br>
<br>
Some things we would need to proposing for KDE extragear/core<br>
- Watch your coding style<br>
 =A0<a href=3D"http://techbase.kde.org/Policies/Kdelibs_Coding_Style" targe=
t=3D"_blank">http://techbase.kde.org/Policies/Kdelibs_Coding_Style</a><br>
 =A0In particular:<br>
 =A0 =A0Always use braces<br>
 =A0 =A0Don&#39;t call classes &quot;MySomething&quot;. As the &quot;My&quo=
t; isn&#39;t very helpful<br>
 =A0 =A0Make sure the files actually match the names of the class.<br>
audio.h, video.h, text.h<br>
 =A0 =A0MyAudio::~MyAudio() use m_mediaObject-&gt;deleteLater(); instead of=
<br>
delete (it&#39;s safer). Alternatively set a parent on it in the<br>
constructor and don&#39;t manually delete it.<br>
 =A0 Technically your whitespace is off in many ways too, but that&#39;s<br=
>
being pedantic.<br></blockquote><div><br></div><div>Unlike what many people=
 think the mentioned web-page is the coding-style for kdelibs only. Other k=
de projects/modules may choose to adopt that style or parts of the style, b=
ut its not mandatory for any module to follow it. You&#39;re always free to=
 choose whatever style suits you.</div>
<div><br></div><div>Andreass</div></div>

--20cf303b42399a31e004bf757f0e--

--===============0619915389680437461==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


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

--===============0619915389680437461==--