KLibrary

Thiago Macieira thiago at kde.org
Fri Sep 14 12:33:35 BST 2007


On Friday 14 September 2007 10:56:47 Nhuh Put wrote:
> Hello
> I would like to apply this patch to KLibLoader next Monday. It moves the
> KLibrary class into its own file and makes it inherit QLibrary. With this
> change, the Library loading system is much smaller and thread-safe.
> I have also removed the unloading stuff from KLibLoader as this is not
> supported anymore.
> This shouldn't include any SC changes as KLibrary::name is unused afaics
>
> 	PutHuhn

You have some private functions in KLibrary. They mustn't be there.

Also, since QLibrary can unload and you're not overriding anything there, 
KLibrary can still be unloaded. And you've removed the clipboard-unset hook.

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20070914/1eadb2c7/attachment.sig>


More information about the kde-core-devel mailing list