Review Request 122246: Add addSample(qreal value) overload

Kai Uwe Broulik kde at privat.broulik.de
Sun Jan 25 15:11:52 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122246/
-----------------------------------------------------------

Review request for Plasma.


Repository: kdeclarative


Description
-------

Add an addSample(qreal value) overload making it easier to work with just a simple dataset. This way you can do plotter.addSample(50) instead of plotter.addSample([50]).

QML seems to accept single strings for QStringList but not other list types.


Diffs
-----

  src/qmlcontrols/kquickcontrolsaddons/plotter.h 3a10212 
  src/qmlcontrols/kquickcontrolsaddons/plotter.cpp 65dee7a 

Diff: https://git.reviewboard.kde.org/r/122246/diff/


Testing
-------

addSample(50) and addSample([40, 41, 42]) work


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150125/670e02ab/attachment.html>


More information about the Plasma-devel mailing list