diff options
| author | Stefan Weil <sw@weilnetz.de> | 2014-02-19 06:11:45 +0100 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2014-03-02 17:12:41 +0400 |
| commit | 55e7c29e46f845f503fda3e8cac3f5e0e8b3d984 (patch) | |
| tree | 89f148b484c700b6ce99ba0976e6380a02ce4458 | |
| parent | 0064aceb29faea018dfb7cf0bca5d9644272a295 (diff) | |
bswap: Modify prototypes of st[wl]_{le, be}_p (avoid type conversions)
The functions use uint16_t or uint32_t values, so show this in the function
prototypes. Non-optimizing compilers will avoid unnecessary type
conversions when generating calls of these inline functions.
stq_le_p, stq_be_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>
| -rw-r--r-- | include/qemu/bswap.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/include/qemu/bswap.h b/include/qemu/bswap.h index 0cc11a5e79..0cb7c05554 100644 --- a/include/qemu/bswap.h +++ b/include/qemu/bswap.h @@ -300,12 +300,12 @@ static inline uint64_t ldq_le_p(const void *ptr) return le_bswap(ldq_p(ptr), 64); } -static inline void stw_le_p(void *ptr, int v) +static inline void stw_le_p(void *ptr, uint16_t v) { stw_p(ptr, le_bswap(v, 16)); } -static inline void stl_le_p(void *ptr, int v) +static inline void stl_le_p(void *ptr, uint32_t v) { stl_p(ptr, le_bswap(v, 32)); } @@ -365,12 +365,12 @@ static inline uint64_t ldq_be_p(const void *ptr) return be_bswap(ldq_p(ptr), 64); } -static inline void stw_be_p(void *ptr, int v) +static inline void stw_be_p(void *ptr, uint16_t v) { stw_p(ptr, be_bswap(v, 16)); } -static inline void stl_be_p(void *ptr, int v) +static inline void stl_be_p(void *ptr, uint32_t v) { stl_p(ptr, be_bswap(v, 32)); } |