skiplist: avoid (undefined) unsigned to signed conversion and save cycles in

the hot path get_b_rand().

git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@1672575 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 3 insertions, 4 deletions

diff --git a/tables/apr_skiplist.c b/tables/apr_skiplist.c
index b5027ecf7..5ea8643db 100644
--- a/tables/apr_skiplist.c
+++ b/tables/apr_skiplist.c
@@ -63,13 +63,12 @@ struct apr_skiplistnode {
 static int get_b_rand(void)
 {
     static int ph = 32;         /* More bits than we will ever use */
-    static apr_uint32_t randseq;
+    static int randseq;
     if (ph > 31) {              /* Num bits in return of rand() */
         ph = 0;
-        randseq = (apr_uint32_t) rand();
+        randseq = rand();
     }
-    ph++;
-    return ((randseq & (1 << (ph - 1))) >> (ph - 1));
+    return randseq & (1 << ph++);
 }
 
 typedef struct {