[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: [Panel-devel] wrapping up for 3.5
From: p0z3r () earthlink ! net (Ryan Nickell)
Date: 2005-06-09 0:26:43
Message-ID: 1118269770.6246.1.camel () linux ! site
[Download RAW message or body]
On Wed, 2005-06-08 at 15:44 -0600, Aaron J. Seigo wrote:
>
> if (data.subtext == QString()) data.subtext = data.message;
>
> should be
>
> if (data.subtext == QString())
> {
> data.subtext = data.message;
> }
>
> i know, i know, i'm an anal bastard. =P otherwise it looks good. if you've
> tested and it works well for you, please commit.
Not so much anal. It's more readable and blunt as to what is happening.
I like brackets; they are your friend! =c)
>
> _______________________________________________
> Panel-devel mailing list
> Panel-devel@kde.org
> https://mail.kde.org/mailman/listinfo/panel-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic