bswap: Modify prototype of stb_p (avoid type conversions)
authorStefan Weil <sw@weilnetz.de>
Wed, 19 Feb 2014 05:11:44 +0000 (06:11 +0100)
committerMichael Tokarev <mjt@tls.msk.ru>
Sun, 2 Mar 2014 13:12:41 +0000 (17:12 +0400)
The function uses an uint8_t value, so show this in the function
prototype. Non-optimizing compilers will avoid unnecessary type
conversions from (u)int8_t to int and back to uint8_t when generating
calls of this inline function.

stw_p, stl_p and stq_p already use similar prototypes.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
include/qemu/bswap.h

index 437b8e0a9ed3785990037c061c189f64b1a2db17..0cc11a5e79cb7c9ce613f2fdaba1946aa66f024d 100644 (file)
@@ -228,7 +228,7 @@ static inline int ldsb_p(const void *ptr)
     return *(int8_t *)ptr;
 }
 
-static inline void stb_p(void *ptr, int v)
+static inline void stb_p(void *ptr, uint8_t v)
 {
     *(uint8_t *)ptr = v;
 }