aly/servatrice
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Remove useless bound checks.

Browse source
This commit is contained in:
Mark Morschhäuser 2014-05-30 11:48:07 +02:00
parent bf6ec478cb
commit 197dd0f3c8
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
common/rng_sfmt.cpp
View file

@ -62,7 +62,7 @@ RNG_SFMT::RNG_SFMT(QObject *parent)
unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max) unsigned int RNG_SFMT::getNumber(unsigned int min, unsigned int max)
{ {
// This all makes no sense if min > max, which should never happen. // This all makes no sense if min > max, which should never happen.
if(min > max || max >= UINT_MAX || min >= UINT_MAX) { if(min > max) {
throw std::invalid_argument( throw std::invalid_argument(
QString("Invalid bounds for RNG: min > max! Values were: min = " + QString("Invalid bounds for RNG: min > max! Values were: min = " +
QString::number(min) + ", max = " + QString::number(min) + ", max = " +