diff options
author | Solomon Peachy <pizza@shaftnet.org> | 2020-10-13 13:20:56 -0400 |
---|---|---|
committer | Solomon Peachy <pizza@shaftnet.org> | 2020-10-13 13:37:23 -0400 |
commit | ca326896d0dfd7693ab3b7097d0fe3ad6ab0a0c9 (patch) | |
tree | b2e5d3fb674aafecfef3562b2dda708b3c52db79 | |
parent | 4a3d04654580b1771fcda3492491843d5eac10ad (diff) | |
download | rockbox-ca326896d0.tar.gz rockbox-ca326896d0.zip |
Fix compile warnings (set-but-not-used) on big endian targets
Change-Id: Ia433122d6c0af68a47d2f4a531a0787a9d3d9f72
-rw-r--r-- | apps/plugins/lib/jhash.c | 32 | ||||
-rw-r--r-- | apps/talk.c | 6 |
2 files changed, 21 insertions, 17 deletions
diff --git a/apps/plugins/lib/jhash.c b/apps/plugins/lib/jhash.c index 780d220408..40fb861d71 100644 --- a/apps/plugins/lib/jhash.c +++ b/apps/plugins/lib/jhash.c @@ -23,8 +23,8 @@ lookup3.c, by Bob Jenkins, May 2006, Public Domain. These are functions for producing 32-bit hashes for hash table lookup. -hashword(), hashlittle(), hashlittle2(), hashbig(), mix(), and final() -are externally useful functions. Routines to test the hash are included +hashword(), hashlittle(), hashlittle2(), hashbig(), mix(), and final() +are externally useful functions. Routines to test the hash are included if SELF_TEST is defined. You can use this free for any purpose. It's in the public domain. It has no warranty. @@ -32,7 +32,7 @@ You probably want to use hashlittle(). hashlittle() and hashbig() hash byte arrays. hashlittle() is is faster than hashbig() on little-endian machines. Intel and AMD are little-endian machines. On second thought, you probably want hashlittle2(), which is identical to -hashlittle() except it returns two 32-bit hashes for the price of one. +hashlittle() except it returns two 32-bit hashes for the price of one. You could implement hashbig2() if you wanted but I haven't bothered here. If you want to find a hash of, say, exactly 7 integers, do @@ -45,9 +45,9 @@ final(a,b,c); then use c as the hash value. If you have a variable length array of 4-byte integers to hash, use hashword(). If you have a byte array (like a character string), use hashlittle(). If you have several byte arrays, or -a mix of things, see the comments above hashlittle(). +a mix of things, see the comments above hashlittle(). -Why is this so big? I read 12 bytes at a time into 3 4-byte integers, +Why is this so big? I read 12 bytes at a time into 3 4-byte integers, then mix those integers. This is fast (you can do a lot more thorough mixing with 12*3 instructions on 3 integers than you can with 3 instructions on 1 byte), but shoehorning those bytes into integers efficiently is messy. @@ -92,7 +92,7 @@ This was tested for: the output delta to a Gray code (a^(a>>1)) so a string of 1's (as is commonly produced by subtraction) look like a single 1-bit difference. -* the base values were pseudorandom, all zero but one bit set, or +* the base values were pseudorandom, all zero but one bit set, or all zero plus a counter that starts at zero. Some k values for my "a-=c; a^=rot(c,k); c+=b;" arrangement that @@ -102,7 +102,7 @@ satisfy this are 14 9 3 7 17 3 Well, "9 15 3 18 27 15" didn't quite get 32 bits diffing for "differ" defined as + with a one-bit base and a two-bit delta. I -used http://burtleburtle.net/bob/hash/avalanche.html to choose +used http://burtleburtle.net/bob/hash/avalanche.html to choose the operations, constants, and arrangements of the variables. This does not achieve avalanche. There are input bits of (a,b,c) @@ -139,7 +139,7 @@ produce values of c that look totally different. This was tested for the output delta to a Gray code (a^(a>>1)) so a string of 1's (as is commonly produced by subtraction) look like a single 1-bit difference. -* the base values were pseudorandom, all zero but one bit set, or +* the base values were pseudorandom, all zero but one bit set, or all zero plus a counter that starts at zero. These constants passed: @@ -187,7 +187,7 @@ uint32_t hashw(const uint32_t *k, size_t length, uint32_t initval) /* handle the last 3 uint32_t's */ switch(length) /* all the case statements fall through */ - { + { case 3: c+=k[2]; case 2: @@ -206,7 +206,7 @@ uint32_t hashw(const uint32_t *k, size_t length, uint32_t initval) /* hashw2() -- same as hashw(), but take two seeds and return two 32-bit values. pc and pb must both be nonnull, and *pc and *pb must -both be initialized with seeds. If you pass in (*pb)==0, the output +both be initialized with seeds. If you pass in (*pb)==0, the output (*pc) will be the same as the return value from hashword(). k: pointer to the key, an array of uint32_t length: number of elements in the key @@ -234,7 +234,7 @@ void hashw2 (const uint32_t *k, size_t length, uint32_t *pc, uint32_t *pb) /* handle the last 3 uint32_t's */ switch(length) /* all the case statements fall through */ - { + { case 3: c+=k[2]; case 2: @@ -279,13 +279,15 @@ acceptable. Do NOT use for cryptographic purposes. uint32_t hashs( const void *key, size_t length, uint32_t initval) { uint32_t a,b,c; /* internal state */ +#if HASH_LITTLE_ENDIAN union { const void *ptr; size_t i; } u;/* needed for Mac Powerbook G4 */ +#endif /* Set up the internal state */ a = b = c = 0xdeadbeef + ((uint32_t)length) + initval; - u.ptr = key; #if HASH_LITTLE_ENDIAN + u.ptr = key; if ((u.i & 0x3) == 0) { const uint32_t *k = (const uint32_t *)key; /* read 32-bit chunks */ @@ -351,7 +353,7 @@ uint32_t hashs( const void *key, size_t length, uint32_t initval) case 1: a += k[0] & 0xff; break; - case 0: + case 0: return c; /* zero length strings require no mixing */ } @@ -494,14 +496,16 @@ hashs2: return 2 32-bit hash values void hashs2(const void *key, size_t length, uint32_t *pc, uint32_t *pb) { uint32_t a, b, c; /* internal state */ +#if HASH_LITTLE_ENDIAN union { const void *ptr; size_t i; } u; /* needed for Mac Powerbook G4 */ +#endif /* Set up the internal state */ a = b = c = 0xdeadbeef + ((uint32_t)length) + *pc; c += *pb; - u.ptr = key; #if HASH_LITTLE_ENDIAN + u.ptr = key; if (((u.i & 0x3) == 0)) { const uint32_t *k = (const uint32_t *)key; /* read 32-bit chunks */ diff --git a/apps/talk.c b/apps/talk.c index a293e367a1..947f1665af 100644 --- a/apps/talk.c +++ b/apps/talk.c @@ -491,7 +491,6 @@ static int get_clip(long id, struct queue_entry *q) static bool load_index_table(int fd, const struct voicefile_header *hdr) { ssize_t ret; - struct clip_entry *buf; if (index_handle > 0) /* nothing to do? */ return true; @@ -505,14 +504,15 @@ static bool load_index_table(int fd, const struct voicefile_header *hdr) if (ret == alloc_size) { +#ifdef ROCKBOX_LITTLE_ENDIAN + struct clip_entry *buf; buf = core_get_data(index_handle); for (int i = 0; i < hdr->id1_max + hdr->id2_max; i++) { -#ifdef ROCKBOX_LITTLE_ENDIAN /* doesn't yield() */ structec_convert(&buf[i], "ll", 1, true); -#endif } +#endif } else index_handle = core_free(index_handle); |