Review Request 118547: Expose Formats as singleton
David Edmundson
david at davidedmundson.co.uk
Thu Jun 5 00:03:27 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118547/
-----------------------------------------------------------
Review request for KDE Frameworks and Plasma.
Repository: kdeclarative
Description
-------
Expose Formats as singleton
Formats is basically just a collection of invokable static methods.
This saves creating a few objects and makes for a a nicer API.
Without it you have to have a bit of code like:
- KCoreAddons.Formats {
- id: formats
- }
then do text:formats.formatData(...) in your labels
this becomes now
KCoreAddons.Format.formatData(...)
Diffs
-----
src/qmlcontrols/kcoreaddons/kcoreaddonsplugin.cpp 65dd75f
Diff: https://git.reviewboard.kde.org/r/118547/diff/
Testing
-------
Made the two changes needed in plasma-workspace. Everything seemed to be ok.
Thanks,
David Edmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140605/6d9fb168/attachment.html>
More information about the Plasma-devel
mailing list