From: Helmut Schaa <helmut.schaa@googlemail.com>
Date: Fri, 13 Dec 2013 13:17:14 +0000 (+0100)
Subject: lib/random: Make some functions in random.c inline
X-Git-Tag: sliver-openvswitch-2.1.90-1~10^2~187
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=2737b46d8703b0c100e5a7f1997545f647bb7c5d;p=sliver-openvswitch.git

lib/random: Make some functions in random.c inline

random_range, random_uint8 and random_uint16 are basically just wrappers
around random_uint32. Hence, don't export these functions so the compiler
can be smarter when it comes to optimizations.

Signed-off-by: Helmut Schaa <helmut.schaa@googlemail.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/lib/random.c b/lib/random.c
index d6f7d9d62..c0bc659d1 100644
--- a/lib/random.c
+++ b/lib/random.c
@@ -86,17 +86,6 @@ random_bytes(void *p_, size_t n)
     }
 }
 
-uint8_t
-random_uint8(void)
-{
-    return random_uint32();
-}
-
-uint16_t
-random_uint16(void)
-{
-    return random_uint32();
-}
 
 uint32_t
 random_uint32(void)
@@ -117,12 +106,6 @@ random_uint64(void)
     return x;
 }
 
-int
-random_range(int max)
-{
-    return random_uint32() % max;
-}
-
 static uint32_t
 random_next(void)
 {
diff --git a/lib/random.h b/lib/random.h
index eaac1437b..5fe50b725 100644
--- a/lib/random.h
+++ b/lib/random.h
@@ -24,10 +24,25 @@ void random_init(void);
 void random_set_seed(uint32_t);
 
 void random_bytes(void *, size_t);
-uint8_t random_uint8(void);
-uint16_t random_uint16(void);
 uint32_t random_uint32(void);
 uint64_t random_uint64(void);
-int random_range(int max);
+
+static inline int
+random_range(int max)
+{
+    return random_uint32() % max;
+}
+
+static inline uint8_t
+random_uint8(void)
+{
+    return random_uint32();
+}
+
+static inline uint16_t
+random_uint16(void)
+{
+    return random_uint32();
+}
 
 #endif /* random.h */