Skip to content

Draft: Realign RESP Swift representation in preparation for RESP 3 support

Nathan Harris requested to merge respvalue-refactor into master

Motivation

As is being discussed in #118 for how to represent and streamline representation of RESP in Swift, the names are out of alignment with current Redis terminology of types and we want to start being more explicit as to the purpose of the enum's use in user and library code.

More work will be necessary to finalize the alignment of this type's purpose, especially in regards to RESPValueConvertible (now RESPDataTypeConvertible)

Changes

  • Change: RESPValue.simpleString payload type from NIO.ByteBuffer to String
  • Rename: RESPValue to RESPDataType
  • Rename: RESPValue.bulkString case to RESPDataType.blobString
  • Rename: RESPValueConvertible to RESPDataTypeConvertible

Merge request reports