He probably only thought about the extra QString() which Pino just
prove it's useless, I was with lamarque because of the Krazy
check but this one is indeed crazy :P
Maybe QLatin1String("") can be better than just "" in this case
2013/5/6 Eike Hein <eikeh...@gmail.com>:
>
>>
>> $ grep -A1 QString::clear /usr/include/qt4/QtCore/qstring.h
>> inline void QString::clear()
>> { if (!isNull()) *this = QString(); }
>
>
> Fun - the "and makes it empty" part in the QString docs sure makes it sound
> like it's going to make it non-null, I guess that's what Lamarque thought.
>
>
> OK, so back to revision 1 in that case.
>
>
> Cheers,
> Eike
>
--
Daniel Nicoletti
KDE Developer - http://dantti.wordpress.com
