nethack-gwaa-kiwi/nethack
1
0
Fork 0
Code Pull Requests Activity

another warning fix

Browse Source 
Comparison between signed and unsigned.  Compiler stupidity since
the range of possible values that signed 'i' can take is limited and
never negative.
This commit is contained in:
PatR
2019-01-29 15:18:17 -08:00
parent 1b49f60ded
commit 39b6a7e515
1 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/rnd.c
View File

@@ -17,16 +17,16 @@ struct rnglist_t {
isaac64_ctx rng_state;
};
enum {CORE = 0, DISP};
enum { CORE = 0, DISP = 1 };
static struct rnglist_t rnglist[] = {
{rn2, FALSE, {0}}, /* CORE */
{rn2_on_display_rng, FALSE, {0}}, /* DISP */
{ rn2, FALSE, { 0 } }, /* CORE */
{ rn2_on_display_rng, FALSE, { 0 } }, /* DISP */
};
int
whichrng(fn)
int (*fn)(int);
int FDECL((*fn), (int));
{
int i;
@@ -39,19 +39,21 @@ int (*fn)(int);
void
init_isaac64(seed, fn)
unsigned long seed;
int FDECL((*fn),(int));
int FDECL((*fn), (int));
{
unsigned char new_rng_state[sizeof(seed)];
int i, rngindx = whichrng(fn);
unsigned char new_rng_state[sizeof seed];
unsigned i;
int rngindx = whichrng(fn);
if (rngindx < 0)
panic("Bad rng function passed to init_isaac64().");
for (i=0; i<sizeof(seed); i++) {
new_rng_state[i]= (unsigned char)(seed & 0xFF);
for (i = 0; i < sizeof seed; i++) {
new_rng_state[i] = (unsigned char) (seed & 0xFF);
seed >>= 8;
}
isaac64_init(&rnglist[rngindx].rng_state, new_rng_state, sizeof(seed));
isaac64_init(&rnglist[rngindx].rng_state, new_rng_state,
(int) sizeof seed);
}
static int