[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 3/7] util/cutils: Turn FIXME comment into QEMU_BUILD_BUG_ON()
From: |
Laurent Vivier |
Subject: |
[PULL 3/7] util/cutils: Turn FIXME comment into QEMU_BUILD_BUG_ON() |
Date: |
Wed, 18 Dec 2019 14:01:01 +0100 |
From: Markus Armbruster <address@hidden>
qemu_strtoi64() assumes int64_t is long long. This is marked FIXME.
Replace by a QEMU_BUILD_BUG_ON() to avoid surprises.
Same for qemu_strtou64().
Fix a typo in qemu_strtoul()'s contract while there.
Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Message-Id: <address@hidden>
[lv: removed trailing whitespace]
Signed-off-by: Laurent Vivier <address@hidden>
---
util/cutils.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/util/cutils.c b/util/cutils.c
index 238016523039..36ce712271f1 100644
--- a/util/cutils.c
+++ b/util/cutils.c
@@ -542,7 +542,7 @@ int qemu_strtoul(const char *nptr, const char **endptr, int
base,
* Convert string @nptr to an int64_t.
*
* Works like qemu_strtol(), except it stores INT64_MAX on overflow,
- * and INT_MIN on underflow.
+ * and INT64_MIN on underflow.
*/
int qemu_strtoi64(const char *nptr, const char **endptr, int base,
int64_t *result)
@@ -557,8 +557,9 @@ int qemu_strtoi64(const char *nptr, const char **endptr,
int base,
return -EINVAL;
}
+ /* This assumes int64_t is long long TODO relax */
+ QEMU_BUILD_BUG_ON(sizeof(int64_t) != sizeof(long long));
errno = 0;
- /* FIXME This assumes int64_t is long long */
*result = strtoll(nptr, &ep, base);
return check_strtox_error(nptr, ep, endptr, errno);
}
@@ -581,8 +582,9 @@ int qemu_strtou64(const char *nptr, const char **endptr,
int base,
return -EINVAL;
}
+ /* This assumes uint64_t is unsigned long long TODO relax */
+ QEMU_BUILD_BUG_ON(sizeof(uint64_t) != sizeof(unsigned long long));
errno = 0;
- /* FIXME This assumes uint64_t is unsigned long long */
*result = strtoull(nptr, &ep, base);
/* Windows returns 1 for negative out-of-range values. */
if (errno == ERANGE) {
--
2.24.1
- [PULL 0/7] Trivial branch patches, Laurent Vivier, 2019/12/18
- [PULL 3/7] util/cutils: Turn FIXME comment into QEMU_BUILD_BUG_ON(),
Laurent Vivier <=
- [PULL 5/7] target/sparc: Remove old TODO file, Laurent Vivier, 2019/12/18
- [PULL 4/7] test-keyval: Tighten test of trailing crap after size, Laurent Vivier, 2019/12/18
- [PULL 7/7] qemu-doc: Remove the unused "Guest Agent" node, Laurent Vivier, 2019/12/18
- [PULL 2/7] monitor: Remove unused define, Laurent Vivier, 2019/12/18
- [PULL 1/7] MAINTAINERS: Add hw/sd/ssi-sd.c in the SD section, Laurent Vivier, 2019/12/18
- [PULL 6/7] Revert "qemu-options.hx: Update for reboot-timeout parameter", Laurent Vivier, 2019/12/18
- Re: [PULL 0/7] Trivial branch patches, Peter Maydell, 2019/12/20