[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