janwas
c0ed950657
this snowballed into a massive search+destroy of the hodgepodge of mostly equivalent types we had in use (int, uint, unsigned, unsigned int, i32, u32, ulong, uintN). it is more efficient to use 64-bit types in 64-bit mode, so the preferred default is size_t (for anything remotely resembling a size or index). tile coordinates are ssize_t to allow more efficient conversion to/from floating point. flags are int because we almost never need more than 15 distinct bits, bit test/set is not slower and int is fastest to type. finally, some data that is pretty much directly passed to OpenGL is now typed accordingly. after several hours, the code now requires fewer casts and less guesswork. other changes: - unit and player IDs now have an "invalid id" constant in the respective class to avoid casting and -1 - fix some endian/64-bit bugs in the map (un)packing. added a convenience function to write/read a size_t. - ia32: change CPUID interface to allow passing in ecx (required for cache topology detection, which I need at work). remove some unneeded functions from asm, replace with intrinsics where possible. This was SVN commit r5942.
65 lines
1.8 KiB
C++
65 lines
1.8 KiB
C++
/**
|
|
* =========================================================================
|
|
* File : rand.cpp
|
|
* Project : 0 A.D.
|
|
* Description : pseudorandom number generator
|
|
* =========================================================================
|
|
*/
|
|
|
|
// license: GPL; see lib/license.txt
|
|
|
|
#include "precompiled.h"
|
|
#include "rand.h"
|
|
|
|
// avoids several common pitfalls; see discussion at
|
|
// http://www.azillionmonkeys.com/qed/random.html
|
|
|
|
// rand() is poorly implemented (e.g. in VC7) and only returns < 16 bits;
|
|
// double that amount by concatenating 2 random numbers.
|
|
// this is not to fix poor rand() randomness - the number returned will be
|
|
// folded down to a much smaller interval anyway. instead, a larger XRAND_MAX
|
|
// decreases the probability of having to repeat the loop.
|
|
#if RAND_MAX < 65536
|
|
static const size_t XRAND_MAX = (RAND_MAX+1)*(RAND_MAX+1) - 1;
|
|
static size_t xrand()
|
|
{
|
|
return rand()*(RAND_MAX+1) + rand();
|
|
}
|
|
// rand() is already ok; no need to do anything.
|
|
#else
|
|
static const size_t XRAND_MAX = RAND_MAX;
|
|
static size_t xrand()
|
|
{
|
|
return rand();
|
|
}
|
|
#endif
|
|
|
|
size_t rand(size_t min_inclusive, size_t max_exclusive)
|
|
{
|
|
const size_t range = (max_exclusive-min_inclusive);
|
|
// huge interval or min >= max
|
|
if(range == 0 || range > XRAND_MAX)
|
|
{
|
|
WARN_ERR(ERR::INVALID_PARAM);
|
|
return 0;
|
|
}
|
|
|
|
const size_t inv_range = XRAND_MAX / range;
|
|
|
|
// generate random number in [0, range)
|
|
// idea: avoid skewed distributions when <range> doesn't evenly divide
|
|
// XRAND_MAX by simply discarding values in the "remainder".
|
|
// not expected to run often since XRAND_MAX is large.
|
|
size_t x;
|
|
do
|
|
{
|
|
x = xrand();
|
|
}
|
|
while(x >= range * inv_range);
|
|
x /= inv_range;
|
|
|
|
x += min_inclusive;
|
|
debug_assert(x < max_exclusive);
|
|
return x;
|
|
}
|