Review Request: Convenience KRandom methods

Sebastian Trueg trueg at kde.org
Thu Aug 12 14:40:52 BST 2010


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

(Updated 2010-08-12 13:40:52.150336)


Review request for kdelibs.


Changes
-------

New implementation for the int range method using % instead of /. Better?


Summary
-------

The KRandom namespace is quite useful. However, applications need to be full of code snippets that calculate a random number in a range. IMHO it makes perfect sense to do this once in the library. Thus, I am proposing this patch to introduce two methods providing random numbers from a range.
Feel free to improve the implementation. :)


Diffs (updated)
-----

  trunk/KDE/kdelibs/kdecore/util/krandom.h 1162164 
  trunk/KDE/kdelibs/kdecore/util/krandom.cpp 1162164 

Diff: http://reviewboard.kde.org/r/4992/diff


Testing
-------


Thanks,

Sebastian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100812/226a89d3/attachment.htm>


More information about the kde-core-devel mailing list