feat: decoder.advance optimization. fix: single byte transmitted hangs the client

Author: eigen-value

src/client.h

@@ -39,7 +39,7 @@ class RPCClient {
         // blocking call
         while (!decoder->get_response(msg_id, result, error)){
             decoder->decode();
-            delay(1);
+            //delay(1);
         }
 
         lastError.code = error.code;

src/decoder.h

@@ -178,25 +178,16 @@ class RpcDecoder {
     // Fill the raw buffer to its capacity
     void advance() {
 
-        uint8_t temp_buf[CHUNK_SIZE];
-    
-        if (_transport.available() && !buffer_full()){
-            int bytes_read = _transport.read(temp_buf, CHUNK_SIZE);
-    
-            for (int i = 0; i < bytes_read; ++i) {
-                _raw_buffer[_bytes_stored] = temp_buf[i];
-                _bytes_stored++;
-                while (buffer_full()){
-                    delay(1);
-                }
-            }
+        if (_transport.available() && !buffer_full()) {
+            size_t bytes_read = _transport.read(_raw_buffer + _bytes_stored, BufferSize - _bytes_stored);
+            _bytes_stored += bytes_read;
         }
 
     }
 
     void parse_packet(){
 
-        if (packet_incoming() || buffer_empty()){return;}
+        if (packet_incoming() || _bytes_stored < 2){return;}
 
         MsgPack::Unpacker unpacker;
         unpacker.clear();

src/server.h

@@ -34,7 +34,7 @@ class RPCServer {
     void run() {
         decoder->decode();
         decoder->process_requests(dispatcher);
-        delay(1);
+        //delay(1);
     }
 
 };