shared lib: conversion-lib in Plasma

Aaron J. Seigo aseigo at kde.org
Thu Jul 23 22:24:29 BST 2009


On Thursday 23 July 2009, Albert Astals Cid wrote:
> * Might be worth Conversion namespace to be named KConversion,
> KUnitConversion ?

+1 on KUnitConversion

>  * Complex class has empty constructor and destructor defined in a header,
> move it to a cpp

done

>  * Do you really need Unit being a QObject?

it's apparently used for memory management, but it's parented to the Converter 
singleton so they will live for the lifetime of the application -> not so 
good. will fix....

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Software
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090723/5f8c63fd/attachment.sig>


More information about the kde-core-devel mailing list