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