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