[PATCH] KSpell2 improvement

Thiago Macieira thiago at kde.org
Mon Mar 27 20:29:47 BST 2006


Mashrab Kuvatov wrote:
>+    if (languagesNameCache.count() == languages().count() )
>+           return languagesNameCache;
>
>It is not constructed at every call, it is cached (if my definition of
> cache is the same as yours).

Ah, sorry. I somehow missed it.

My bad.

Code looks good this way. The non-static array can stay too, since the 
code will be run only once.

-- 
Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
  thiago.macieira (AT) trolltech.com     Trolltech AS
    GPG: 0x6EF45358                   |  Sandakerveien 116,
    E067 918B B660 DBD1 105C          |  NO-0402
    966C 33F5 F005 6EF4 5358          |  Oslo, Norway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060327/3c309b6b/attachment.sig>


More information about the kde-core-devel mailing list