Review Request: Molmasscalcuation for the kalzium-engine

Carsten Niehaus cniehaus at gmx.de
Wed Mar 10 17:20:41 CET 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3228/#review4454
-----------------------------------------------------------

Ship it!


Looks good to me. Thanks for taking care!

- Carsten


On 2010-03-09 19:11:33, eti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3228/
> -----------------------------------------------------------
> 
> (Updated 2010-03-09 19:11:33)
> 
> 
> Review request for KDE-Edu and Plasma.
> 
> 
> Summary
> -------
> 
> I was playing with plasma and made a little plasmoid on kde-look: http://kde-look.org/content/show.php?content=114429
> The dataengine used is derived from the kalzium dataengine, so i thought i could put the molmass-caculation-code in the "real" kalzium-engine.
>  
> new features:
> - Send a Molecule to the Engine and get 
>   - the total molecular mass
>   - a string with the summerized Elements. e.g CH3CH2OH > C2H6O
> - Added some Element informations, like electronegativity and electronconfiguration.
> 
> The new molecule query is "Molecule:Parser:<SomeMolecule>"
> 
> Thanks for feedbacks
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/kalzium/plasmoid/engine/kalzium_engine.h 1101250 
>   /trunk/KDE/kdeedu/kalzium/plasmoid/engine/kalzium_engine.cpp 1101250 
> 
> Diff: http://reviewboard.kde.org/r/3228/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> eti
> 
>



More information about the Plasma-devel mailing list