Fixed bugs from v0.2.0 release.

- Added inline to `internal::ClientInterface::roles` definition (fixes #47)
- Revised if conditions in `operator<<` for `Event`, `Result`, and `Invocation` (fixes #48)
- Changed Real to JSON serialization format string to `"%.17e"` (fixes #49)

Author: ecorm

cppwamp/include/cppwamp/internal/clientinterface.hpp

@@ -32,8 +32,7 @@ namespace internal
 {
 
 //------------------------------------------------------------------------------
-// Specifies the interface required for classes that implement the
-// wamp::Session class.
+// Specifies the interface required for classes that implement wamp::Session.
 //------------------------------------------------------------------------------
 class ClientInterface : public Callee, public Subscriber
 {
@@ -74,7 +73,7 @@ class ClientInterface : public Callee, public Subscriber
     virtual void postpone(std::function<void ()> functor) = 0;
 };
 
-const Object& ClientInterface::roles()
+inline const Object& ClientInterface::roles()
 {
     static const Object roles =
     {

cppwamp/include/cppwamp/internal/json.ipp

@@ -96,7 +96,7 @@ struct EncodeJson : public Visitor<>
     void operator()(Real x, TBuffer& buf) const
     {
         char str[32];
-        auto length = std::snprintf(str, sizeof(str), "%e.17", x);
+        auto length = std::snprintf(str, sizeof(str), "%.17e", x);
         assert(length < sizeof(str));
         buf.write(str, length);
     }

cppwamp/include/cppwamp/internal/sessiondata.ipp

@@ -283,7 +283,7 @@ CPPWAMP_INLINE std::ostream& operator<<(std::ostream& out, const Event& event)
         out << ", Details|dict = " << event.options();
     if (!event.args().empty())
         out << ", Arguments|list = " << event.args();
-    if (!event.args().empty())
+    if (!event.kwargs().empty())
         out << ", ArgumentsKw|dict = " << event.kwargs();
     return out << " ]";
 }
@@ -427,7 +427,7 @@ CPPWAMP_INLINE std::ostream& operator<<(std::ostream& out, const Result& result)
         out << ", Details|dict = " << result.options();
     if (!result.args().empty())
         out << ", Arguments|list = " << result.args();
-    if (!result.args().empty())
+    if (!result.kwargs().empty())
         out << ", ArgumentsKw|dict = " << result.kwargs();
     return out << " ]";
 }
@@ -524,7 +524,7 @@ CPPWAMP_INLINE std::ostream& operator<<(std::ostream& out,
         out << ", Details|dict = " << inv.options();
     if (!inv.args().empty())
         out << ", Arguments|list = " << inv.args();
-    if (!inv.args().empty())
+    if (!inv.kwargs().empty())
         out << ", ArgumentsKw|dict = " << inv.kwargs();
     return out << " ]";
 }

test/codectestjson.cpp

@@ -93,20 +93,20 @@ void checkInteger(const std::string& json, TInteger n)
 void checkReal(const std::string& json, double x)
 {
     INFO( "For JSON string \"" << json << "\"" );
-
+    auto epsilon = std::numeric_limits<Real>::epsilon()*10.0;
     {
         Variant v;
         CHECK_NOTHROW( Json::decode(json, v) );
         REQUIRE( v.is<Real>() );
-        CHECK( v.as<Real>() == Approx(x).epsilon(0.001) );
+        CHECK( v.as<Real>() == Approx(x).epsilon(epsilon) );
     }
 
     {
         Variant v;
         std::istringstream iss(json);
         CHECK_NOTHROW( Json::decode(iss, v) );
         REQUIRE( v.is<Real>() );
-        CHECK( v.as<Real>() == Approx(x).epsilon(0.001) );
+        CHECK( v.as<Real>() == Approx(x).epsilon(epsilon) );
     }
 }
 
@@ -145,7 +145,7 @@ GIVEN( "valid JSON numeric strings" )
     checkReal("0.0", 0.0);
     checkReal("1.0", 1.0);
     checkReal("-1.0", -1.0);
-    checkReal("3.14", 3.14);
+    checkReal("3.14159265358979324", 3.14159265358979324);
     checkReal("2.9979e8", 2.9979e8);
 }
 GIVEN( "valid JSON strings" )