From quanta-devel  Wed Dec 21 06:27:01 2005
From: Adriaan de Groot <groot () kde ! org>
Date: Wed, 21 Dec 2005 06:27:01 +0000
To: quanta-devel
Subject: Re: [quanta-devel] [kde-freebsd]
Message-Id: <20051221082206.W25237 () hex ! athame ! co ! uk>
X-MARC-Message: https://marc.info/?l=quanta-devel&m=113514999025796

On Tue, 20 Dec 2005, David Johnson wrote:
> On Tuesday 20 December 2005 01:06 pm, Adriaan de Groot wrote:
>> SVN commit 490122 by adridg:
>> -  item->setText(VariablesListViewColumns::Value, (newvar->isScalar() ? oldvar->value() : ""));
>> +  item->setText(VariablesListViewColumns::Value, (newvar->isScalar() ? oldvar->value() : QString()));
>
> QString() creates a null string, not an empty string. While you're probably
> safe with listview items, there are some QString methods which will segfault
> on null QStrings.

Good nitpick. CC'ing quanta-devel so they're aware of the issue I may have 
caused here. QString() vs. "" is one of those nasty subtleties in the 
interface. If relevant, this should become QString::fromLatin1("") for 
greatest portability also in the face of QT_NO_CAST_ASCII.

[ade]
_______________________________________________
quanta-devel mailing list
quanta-devel@kde.org
https://mail.kde.org/mailman/listinfo/quanta-devel