Regarding commit 049175604 (fix compile with QT_USE_FAST_CONCATENATION....)

David Nolden zwabel at googlemail.com
Thu Aug 11 14:24:11 UTC 2011


2011/8/11 Aleix Pol <aleixpol at kde.org>:
> On Thu, Aug 11, 2011 at 4:05 PM, David Nolden <zwabel at googlemail.com> wrote:
>>
>> I think this makes the code uglier, without any gain. All of the
>> changed code-locations are not performance relevant.
>>
>> Basically all the changes are:
>>
>> myString + "myText"
>> ->
>> QString(myString + "myText")
>>
>> Which is the opposite of natural. So why do this?
>>
>> --
>> KDevelop-devel mailing list
>> KDevelop-devel at kdevelop.org
>> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>
> That happens because with this change + returns a QStringBuilder.
> It makes string concatenation to work much faster so I think it's a good
> trade off.
> Aleix

The trade-off is "no gain" vs. "uglier code". That's good for you?




More information about the KDevelop-devel mailing list