Change all Bridge.call in library

Author: facchinm

README.md

@@ -41,7 +41,7 @@ void setup() {
 
 void loop() {
     float res;
-    if (!Bridge.call("multiply", res, 1.0, 2.0)) {
+    if (!Bridge.call("multiply", 1.0, 2.0).result(res)) {
         Serial.println("Error calling method: multiply");
         Serial.println(Bridge.get_error_code());
         Serial.println(Bridge.get_error_message());

examples/simple_bridge/simple_bridge.ino

@@ -47,7 +47,7 @@ void setup() {
 
 void loop() {
     float res;
-    if (!Bridge.call("multiply", res, 1.0, 2.0)) {
+    if (!Bridge.call("multiply", 1.0, 2.0).result(res)) {
         Serial.println("Error calling method: multiply");
         Serial.println(Bridge.get_error_code());
         Serial.println(Bridge.get_error_message());

src/bridge.h

@@ -110,7 +110,7 @@ class BridgeClass {
                                 UPDATE_THREAD_PRIORITY, 0, K_NO_WAIT);
 
         bool res;
-        call(RESET_METHOD, res);
+        call(RESET_METHOD).result(res);
         if (res) {
             started = true;
         }
@@ -120,7 +120,7 @@ class BridgeClass {
     template<typename F>
     bool provide(const MsgPack::str_t& name, F&& func) {
         bool res;
-        if (!call(BIND_METHOD, res, name)) {
+        if (!call(BIND_METHOD, name).result(res)) {
             return false;
         }
         return server->bind(name, func);
@@ -129,7 +129,7 @@ class BridgeClass {
     template<typename F>
     bool provide_safe(const MsgPack::str_t& name, F&& func) {
         bool res;
-        if (!call(BIND_METHOD, res, name)) {
+        if (!call(BIND_METHOD, name).result(res)) {
             return false;
         }
 

src/monitor.h

@@ -44,7 +44,7 @@ class BridgeMonitor: public Stream {
         if (!bridge_started) {
             bridge_started = bridge->begin();
         }
-        return bridge_started && bridge->call(MON_CONNECTED_METHOD, is_connected);
+        return bridge_started && bridge->call(MON_CONNECTED_METHOD).result(is_connected);
     }
 
     explicit operator bool() const {
@@ -99,7 +99,7 @@ class BridgeMonitor: public Stream {
         }
 
         size_t written;
-        const bool ret = bridge->call(MON_WRITE_METHOD, written, send_buffer);
+        const bool ret = bridge->call(MON_WRITE_METHOD, send_buffer).result(written);
         if (ret) {
             return written;
         }
@@ -109,7 +109,7 @@ class BridgeMonitor: public Stream {
 
     bool reset() {
         bool res;
-        bool ok = bridge->call(MON_RESET_METHOD, res);
+        bool ok = bridge->call(MON_RESET_METHOD).result(res);
         if (ok && res) {
             is_connected = false;
         }
@@ -124,7 +124,7 @@ class BridgeMonitor: public Stream {
         k_mutex_lock(&monitor_mutex, K_FOREVER);
 
         MsgPack::arr_t<uint8_t> message;
-        bool ret = bridge->call(MON_READ_METHOD, message, size);
+        bool ret = bridge->call(MON_READ_METHOD, size).result(message);
 
         if (ret) {
             for (size_t i = 0; i < message.size(); ++i) {

src/tcp_client.h

@@ -61,7 +61,7 @@ class BridgeTCPClient : public Client {
 
         k_mutex_lock(&client_mutex, K_FOREVER);
 
-        const bool resp = bridge->call(TCP_CONNECT_METHOD, connection_id, hostname, port);
+        const bool resp = bridge->call(TCP_CONNECT_METHOD, hostname, port).result(connection_id);
 
         if (!resp) {
             _connected = false;
@@ -84,7 +84,7 @@ class BridgeTCPClient : public Client {
 
         k_mutex_lock(&client_mutex, K_FOREVER);
 
-        const bool resp = bridge->call(TCP_CONNECT_SSL_METHOD, connection_id, hostname, port, ca_cert_str);
+        const bool resp = bridge->call(TCP_CONNECT_SSL_METHOD, hostname, port, ca_cert_str).result(connection_id);
 
         if (!resp) {
             _connected = false;
@@ -116,7 +116,7 @@ class BridgeTCPClient : public Client {
         }
 
         size_t written;
-        const bool ret = bridge->call(TCP_WRITE_METHOD, written, connection_id, payload);
+        const bool ret = bridge->call(TCP_WRITE_METHOD, connection_id, payload).result(written);
         if (ret) {
             return written;
         }
@@ -170,7 +170,7 @@ class BridgeTCPClient : public Client {
     void stop() override {
         k_mutex_lock(&client_mutex, K_FOREVER);
         String msg;
-        const bool resp = bridge->call(TCP_CLOSE_METHOD, msg, connection_id);
+        const bool resp = bridge->call(TCP_CLOSE_METHOD, connection_id).result(msg);
         if (resp) {
             _connected = false;
         }
@@ -198,7 +198,7 @@ class BridgeTCPClient : public Client {
         k_mutex_lock(&client_mutex, K_FOREVER);
 
         MsgPack::arr_t<uint8_t> message;
-        const bool ret = bridge->call(TCP_READ_METHOD, message, connection_id, size);
+        const bool ret = bridge->call(TCP_READ_METHOD, connection_id, size).result(message);
 
         if (ret) {
             for (size_t i = 0; i < message.size(); ++i) {

src/tcp_server.h

@@ -52,7 +52,7 @@ class BridgeTCPServer final: public Server {
         k_mutex_lock(&server_mutex, K_FOREVER);
 
         String hostname = _addr.toString();
-        _listening = bridge->call(TCP_LISTEN_METHOD, listener_id, hostname, _port);
+        _listening = bridge->call(TCP_LISTEN_METHOD, hostname, _port).result(listener_id);
 
         k_mutex_unlock(&server_mutex);
 
@@ -70,7 +70,7 @@ class BridgeTCPServer final: public Server {
 
         k_mutex_lock(&server_mutex, K_FOREVER);
 
-        const bool ret = bridge->call(TCP_ACCEPT_METHOD, connection_id, listener_id);
+        const bool ret = bridge->call(TCP_ACCEPT_METHOD, listener_id).result(connection_id);
 
         k_mutex_unlock(&server_mutex);
 
@@ -104,7 +104,7 @@ class BridgeTCPServer final: public Server {
         k_mutex_lock(&server_mutex, K_FOREVER);
 
         String msg;
-        const bool ret = bridge->call(TCP_CLOSE_LISTENER_METHOD, msg, listener_id);
+        const bool ret = bridge->call(TCP_CLOSE_LISTENER_METHOD, listener_id).result(msg);
 
         if (ret) {
             _listening = false;