[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]