[PATCH] kdelibs: isNull() instead of QString::null
David Faure
faure at kde.org
Wed Oct 27 09:16:23 BST 2004
On Wednesday 27 October 2004 09:19, Craig Drummond wrote:
> > Another patch to kdelibs in response to the automated test script
> > report linked on dot.kde.org. This time it's to replace checks for str
> > != QString::null with !str.isNull().
>
> Should this not be str.isEmpty()?
In your code, if you know the semantics, yes. When touching someone else's
code which tests for == QString::null, then the exact equivalent is isNull().
--
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the kde-core-devel
mailing list