TESTING:-
Again from list discussed with Damien -
15) Priceimpl.c - lld? Looks wrong in openmama.
-
snprintf (buf, bufMaxLen, "%lld", integer);
Luckily, every version of VS since 2005 seems to support '%lld', but now changed to the PRIi64 Macro.
PATCH:- 0002-Correct-the-printing-of-64bit-integers
From fca1911f42012cd72628648895c24d05c4b8e268 Mon Sep 17 00:00:00 2001
From: Gary Molloy <gmolloy@...>
Date: Thu, 7 Aug 2014 15:58:18 +0100
Subject: [PATCH 2/4] Correct the printing of 64bit integers
[OMAMA-280]
Signed-off-by: Gary Molloy <gmolloy@...>
---
mama/c_cpp/src/c/priceimpl.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/mama/c_cpp/src/c/priceimpl.c b/mama/c_cpp/src/c/priceimpl.c
index 58c3095..86d2ebd 100644
--- a/mama/c_cpp/src/c/priceimpl.c
+++ b/mama/c_cpp/src/c/priceimpl.c
@@ -103,7 +103,7 @@ void mamaPriceImpl_getAsStringInteger (double value,
mama_size_t bufMaxLen)
{
int64_t integer = (int64_t)value;
- snprintf (buf, bufMaxLen, "%lld", integer);
+ snprintf (buf, bufMaxLen, "%"PRIi64, integer);
}
@@ -117,18 +117,18 @@ void mamaPriceImpl_getAsStringFraction (double value,
int64_t numer = (int64_t)((double)denom * fraction);
if (numer == 0)
{
- snprintf (buf, bufMaxLen, "%lld", integer);
+ snprintf (buf, bufMaxLen, "%"PRIi64, integer);
}
else if (integer == 0)
{
char formatSpecifier[20] = "";
- sprintf(formatSpecifier, "%s/%s", "%lld", "%ld");
+ sprintf(formatSpecifier, "%s/%s", "%"PRIi64, "%ld");
snprintf (buf, bufMaxLen, formatSpecifier, numer, denom);
}
else
{
char formatSpecifier[20] = "";
- sprintf(formatSpecifier, "%s %s/%s", "%lld", "%lld", "%ld");
+ sprintf(formatSpecifier, "%s %s/%s", "%"PRIi64, "%"PRIi64, "%ld");
if (integer < 0)
{
numer = -numer;
--
1.8.3.1
