|
 |
0a122b |
From e9f800ec6e87d1fee21c59dc83236456ee7f91f8 Mon Sep 17 00:00:00 2001
|
|
|
0a122b |
Message-Id: <e9f800ec6e87d1fee21c59dc83236456ee7f91f8.1387369730.git.minovotn@redhat.com>
|
|
|
0a122b |
In-Reply-To: <091eecc4fa42754760dfff393dabcc2b444e9693.1387369730.git.minovotn@redhat.com>
|
|
|
0a122b |
References: <091eecc4fa42754760dfff393dabcc2b444e9693.1387369730.git.minovotn@redhat.com>
|
|
|
0a122b |
From: Markus Armbruster <armbru@redhat.com>
|
|
|
0a122b |
Date: Tue, 10 Dec 2013 15:29:06 +0100
|
|
|
0a122b |
Subject: [PATCH 06/21] qapi: fix visitor serialization tests for
|
|
|
0a122b |
numbers/doubles
|
|
|
0a122b |
|
|
|
0a122b |
RH-Author: Markus Armbruster <armbru@redhat.com>
|
|
|
0a122b |
Message-id: <1386689361-30281-4-git-send-email-armbru@redhat.com>
|
|
|
0a122b |
Patchwork-id: 56121
|
|
|
0a122b |
O-Subject: [PATCH 7.0 qemu-kvm 03/18] qapi: fix visitor serialization tests for numbers/doubles
|
|
|
0a122b |
Bugzilla: 997915
|
|
|
0a122b |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
0a122b |
RH-Acked-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
0a122b |
RH-Acked-by: Luiz Capitulino <lcapitulino@redhat.com>
|
|
|
0a122b |
|
|
|
0a122b |
From: Michael Roth <mdroth@linux.vnet.ibm.com>
|
|
|
0a122b |
|
|
|
0a122b |
We never actually stored the stringified double values into the strings
|
|
|
0a122b |
before we did the comparisons. This left number/double values completely
|
|
|
0a122b |
uncovered in test-visitor-serialization tests.
|
|
|
0a122b |
|
|
|
0a122b |
Fixing this exposed a bug in our handling of large whole number values
|
|
|
0a122b |
in QEMU's JSON parser which is now fixed.
|
|
|
0a122b |
|
|
|
0a122b |
Simplify the code while we're at it by dropping the
|
|
|
0a122b |
calc_float_string_storage() craziness in favor of GStrings.
|
|
|
0a122b |
|
|
|
0a122b |
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
|
|
|
0a122b |
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
0a122b |
Reviewed-by: Amos Kong <akong@redhat.com>
|
|
|
0a122b |
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
|
|
|
0a122b |
(cherry picked from commit 089f26bb735fb414b79f5fa3753910d5339d2a1d)
|
|
|
0a122b |
Signed-off-by: Markus Armbruster <armbru@redhat.com>
|
|
|
0a122b |
---
|
|
|
0a122b |
tests/test-visitor-serialization.c | 25 ++++++++-----------------
|
|
|
0a122b |
1 file changed, 8 insertions(+), 17 deletions(-)
|
|
|
0a122b |
|
|
|
0a122b |
Signed-off-by: Michal Novotny <minovotn@redhat.com>
|
|
|
0a122b |
---
|
|
|
0a122b |
tests/test-visitor-serialization.c | 25 ++++++++-----------------
|
|
|
0a122b |
1 file changed, 8 insertions(+), 17 deletions(-)
|
|
|
0a122b |
|
|
|
0a122b |
diff --git a/tests/test-visitor-serialization.c b/tests/test-visitor-serialization.c
|
|
|
0a122b |
index 8c8adac..fed6810 100644
|
|
|
0a122b |
--- a/tests/test-visitor-serialization.c
|
|
|
0a122b |
+++ b/tests/test-visitor-serialization.c
|
|
|
0a122b |
@@ -229,17 +229,6 @@ typedef struct TestArgs {
|
|
|
0a122b |
void *test_data;
|
|
|
0a122b |
} TestArgs;
|
|
|
0a122b |
|
|
|
0a122b |
-#define FLOAT_STRING_PRECISION 6 /* corresponding to n in %.nf formatting */
|
|
|
0a122b |
-static gsize calc_float_string_storage(double value)
|
|
|
0a122b |
-{
|
|
|
0a122b |
- int whole_value = value;
|
|
|
0a122b |
- gsize i = 0;
|
|
|
0a122b |
- do {
|
|
|
0a122b |
- i++;
|
|
|
0a122b |
- } while (whole_value /= 10);
|
|
|
0a122b |
- return i + 2 + FLOAT_STRING_PRECISION;
|
|
|
0a122b |
-}
|
|
|
0a122b |
-
|
|
|
0a122b |
static void test_primitives(gconstpointer opaque)
|
|
|
0a122b |
{
|
|
|
0a122b |
TestArgs *args = (TestArgs *) opaque;
|
|
|
0a122b |
@@ -248,7 +237,6 @@ static void test_primitives(gconstpointer opaque)
|
|
|
0a122b |
PrimitiveType *pt_copy = g_malloc0(sizeof(*pt_copy));
|
|
|
0a122b |
Error *err = NULL;
|
|
|
0a122b |
void *serialize_data;
|
|
|
0a122b |
- char *double1, *double2;
|
|
|
0a122b |
|
|
|
0a122b |
pt_copy->type = pt->type;
|
|
|
0a122b |
ops->serialize(pt, &serialize_data, visit_primitive_type, &err;;
|
|
|
0a122b |
@@ -260,14 +248,17 @@ static void test_primitives(gconstpointer opaque)
|
|
|
0a122b |
g_assert_cmpstr(pt->value.string, ==, pt_copy->value.string);
|
|
|
0a122b |
g_free((char *)pt_copy->value.string);
|
|
|
0a122b |
} else if (pt->type == PTYPE_NUMBER) {
|
|
|
0a122b |
+ GString *double_expected = g_string_new("");
|
|
|
0a122b |
+ GString *double_actual = g_string_new("");
|
|
|
0a122b |
/* we serialize with %f for our reference visitors, so rather than fuzzy
|
|
|
0a122b |
* floating math to test "equality", just compare the formatted values
|
|
|
0a122b |
*/
|
|
|
0a122b |
- double1 = g_malloc0(calc_float_string_storage(pt->value.number));
|
|
|
0a122b |
- double2 = g_malloc0(calc_float_string_storage(pt_copy->value.number));
|
|
|
0a122b |
- g_assert_cmpstr(double1, ==, double2);
|
|
|
0a122b |
- g_free(double1);
|
|
|
0a122b |
- g_free(double2);
|
|
|
0a122b |
+ g_string_printf(double_expected, "%.6f", pt->value.number);
|
|
|
0a122b |
+ g_string_printf(double_actual, "%.6f", pt_copy->value.number);
|
|
|
0a122b |
+ g_assert_cmpstr(double_actual->str, ==, double_expected->str);
|
|
|
0a122b |
+
|
|
|
0a122b |
+ g_string_free(double_expected, true);
|
|
|
0a122b |
+ g_string_free(double_actual, true);
|
|
|
0a122b |
} else if (pt->type == PTYPE_BOOLEAN) {
|
|
|
0a122b |
g_assert_cmpint(!!pt->value.max, ==, !!pt->value.max);
|
|
|
0a122b |
} else {
|
|
|
0a122b |
--
|
|
|
0a122b |
1.7.11.7
|
|
|
0a122b |
|