diff options
author | Salvatore Sanfilippo <antirez@gmail.com> | 2020-05-05 10:26:02 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-05 10:26:02 +0200 |
commit | 154e735c46dd77d264eb469928cb3ff7bab1e091 (patch) | |
tree | f1657a4760969ec29d2fbbe447d2f39fe2ac8246 | |
parent | 8257a3c1f61ae05b9545446c0ee0a29cdc080e1a (diff) | |
parent | 4477ee33ac1354b3eecc581e48227d96d14f6c94 (diff) | |
download | redis-154e735c46dd77d264eb469928cb3ff7bab1e091.tar.gz |
Merge pull request #7194 from tryfinally/dev-reverse-bits
Fixed compiler warnings on rev(ulong) function
-rw-r--r-- | src/dict.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/dict.c b/src/dict.c index ac6f8cfde..57619c280 100644 --- a/src/dict.c +++ b/src/dict.c @@ -766,9 +766,9 @@ dictEntry *dictGetFairRandomKey(dict *d) { /* Function to reverse bits. Algorithm from: * http://graphics.stanford.edu/~seander/bithacks.html#ReverseParallel */ static unsigned long rev(unsigned long v) { - unsigned long s = 8 * sizeof(v); // bit size; must be power of 2 - unsigned long mask = ~0; - while ((s >>= 1) > 0) { + unsigned long s = CHAR_BIT * sizeof(v); // bit size; must be power of 2 + unsigned long mask = ~0UL; + while ((s >>= 1) > 0UL) { mask ^= (mask << s); v = ((v >> s) & mask) | ((v << s) & ~mask); } |