[PATCH] Cast input argument for json_object_add_value_int to long long

[PATCH] Cast input argument for json_object_add_value_int to long long

[Puthikorn Voravootivat]

The latency related (and some other) variables type is uint32_t
but the JSON integer type is long long. So we need to cast
the numbers to long long type to make JSON output correctly.

Signed-off-by: Puthikorn Voravootivat <puthik@chromium.org>
---
Found this bug when test on arm. Fix may be a little hacky.
Feel free to change to better idea.
 json.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/json.h b/json.h
index 081afd6..962c11c 100644
--- a/json.h
+++ b/json.h
@@ -52,7 +52,7 @@ void json_free_object(struct json_object *obj);
 
 int json_object_add_value_type(struct json_object *obj, const char *name, int type, ...);
 #define json_object_add_value_int(obj, name, val) \
-	json_object_add_value_type((obj), name, JSON_TYPE_INTEGER, (val))
+	json_object_add_value_type((obj), name, JSON_TYPE_INTEGER, (long long) (val))
 #define json_object_add_value_float(obj, name, val) \
 	json_object_add_value_type((obj), name, JSON_TYPE_FLOAT, (val))
 #define json_object_add_value_string(obj, name, val) \
-- 
2.0.0.526.g5318336

Re: [PATCH] Cast input argument for json_object_add_value_int to long long

[Jens Axboe]

On 2014-06-24 16:59, Puthikorn Voravootivat wrote:
> The latency related (and some other) variables type is uint32_t
> but the JSON integer type is long long. So we need to cast
> the numbers to long long type to make JSON output correctly.

Thanks a lot, applied.

-- 
Jens Axboe