Merge branch 'resp-translator-refactor' into '47-proposal-feedback'

Refactor `RESPTranslator` to mutate the passed `ByteBuffer` directly.

See merge request Mordil/swift-redis-nio-client!63

Author: Mordil

Sources/RedisNIO/ChannelHandlers/RedisByteDecoder.swift

@@ -21,18 +21,19 @@ import NIO
 public final class RedisByteDecoder: ByteToMessageDecoder {
     /// `ByteToMessageDecoder.InboundOut`
     public typealias InboundOut = RESPValue
+    
+    private let parser: RESPTranslator
+    
+    public init() {
+        self.parser = RESPTranslator()
+    }
 
     /// See `ByteToMessageDecoder.decode(context:buffer:)`
     public func decode(context: ChannelHandlerContext, buffer: inout ByteBuffer) throws -> DecodingState {
-        var position = 0
-
-        switch try RESPTranslator.parseBytes(&buffer, fromIndex: &position) {
-        case .incomplete: return .needMoreData
-        case let .parsed(value):
-            context.fireChannelRead(wrapInboundOut(value))
-            buffer.moveReaderIndex(forwardBy: position)
-            return .continue
-        }
+        guard let value = try self.parser.parseBytes(from: &buffer) else { return .needMoreData }
+        
+        context.fireChannelRead(wrapInboundOut(value))
+        return .continue
     }
 
     /// See `ByteToMessageDecoder.decodeLast(context:buffer:seenEOF)`

Sources/RedisNIO/ChannelHandlers/RedisMessageEncoder.swift

@@ -33,7 +33,7 @@ public final class RedisMessageEncoder: MessageToByteEncoder {
     ///
     /// See [https://redis.io/topics/protocol](https://redis.io/topics/protocol) and `RESPEncoder.encode(data:out:)`
     public func encode(data: RESPValue, out: inout ByteBuffer) throws {
-        RESPTranslator.writeValue(data, into: &out)
+        out.writeRESPValue(data)
 
         logger.debug("Encoded \(data) to \(getPrintableString(for: &out))")
     }