[patch] KStandardDirs: one crash fix + code cleanups
Allen Winter
winter at kde.org
Tue Jul 8 21:08:29 BST 2008
On Tuesday 08 July 2008 15:52:49 Jaroslaw Staniek wrote:
>
> (for review)
>
> Important fix:
> - Windows: avoid crash in KStandardDirs::exists(fullPath) for empty fullPath
>
> Minor cleanups:
> - KStandardDirs::addResourceDir(): check for empty absdir or type
> - use more constness
> - lookupPrefix(): do not check for relpath.isEmpty() twice
> - replace checking for (bool)filter.length() with !filter.isEmpty()
> - use QString::isEmpty() instead of isNull()
> - use default value of 2nd arg for QString::mid()
>
Certainly the crash fix is good.
The other changes also look ok to me.
Did you run the kstandarddirstest unit test?
More information about the kde-core-devel
mailing list