Review Request 129342: Fix randomization in color scheme
Martin Tobias Holmedahl Sandsmark
martin.sandsmark at kde.org
Sat Nov 5 16:43:00 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129342/
-----------------------------------------------------------
Review request for Konsole and Kurt Hindenburg.
Repository: konsole
Description
-------
Use the new std::random APIs in c++11 for generating random numbers which is much safer and doesn't abuse qsrand().
This also means we don't need to copy in the QUuid::createUuid() code anymore, which generated a warning in ubsan.
Diffs (updated)
-----
src/ColorScheme.cpp a85072b
src/Session.cpp 302bddc
Diff: https://git.reviewboard.kde.org/r/129342/diff/
Testing (updated)
-------
It still generates random colors; all three HSV values are randomized properly and deterministically according to the randomSeed. Doesn't complain about Session::createUuid() anymore.
Thanks,
Martin Tobias Holmedahl Sandsmark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20161105/d214d91e/attachment.html>
More information about the konsole-devel
mailing list