[patch] KLocale: minor optimization + use safer qgetenv
Jarosław Staniek
js at iidea.pl
Thu Mar 6 11:50:06 GMT 2008
For review.
- Minor optimization: before we've typically iterated twice (in
initLanguageList() and setLanguage() over lost of (typically) empty strings.
- Use safer qgetenv, based on getenv_s under msvc>=2k5 and just ::getenv
elsewhere.
I shall also propose more changes to qgetenv (and qputenv), especially in
places where we cast to QByteArray or QString.
--
regards / pozdrawiam, Jaroslaw Staniek
Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi)
KDE Libraries for MS Windows (http://windows.kde.org)
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: klocale-getenv.patch
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080306/bcf3c9ae/attachment.ksh>
More information about the kde-core-devel
mailing list