Skip to content

Port away from KRandom

Sebastian Engel requested to merge work/blu-base/krandom into master

This commit replaces KRandom with QRandomGenerator as suggested by T12093

To use QRandomGenerator, the QT_REQUIRED_VERSION must be increased to 5.10 (from 5.8) I don't have an overview whether this restriction is currently undesired for this project.

There is one issue remaining: There is a KRandomSequence in src/core-impl/collections/support/MemoryQueryMaker.cpp, l.105. It is used in the Private struct of that class. As far as i understood, there isn't really a use of this member. In src/core-impl/collections/support/MemoryQueryMakerHelper.cpp there is also an include of KRandomSequence but without any call. Is there any reason to keep these occurrences of KRandomSequence?

Merge request reports