- Reimplemented BytesToString more efficiently.

- Updated comments on Int96 struct.

Author: avalerio-tkd

src/common/bytes_utils.h

@@ -403,10 +403,5 @@ inline std::vector<uint8_t> StringToBytes(const std::string& str) {
 }
 
 inline std::string BytesToString(tcb::span<const uint8_t> span) {
-    std::string result;
-    result.reserve(span.size());
-    for (const uint8_t byte : span) {
-        result += static_cast<char>(byte);
-    }
-    return result;
+    return std::string(reinterpret_cast<const char*>(span.data()), span.size());
 }

src/processing/typed_buffer_codecs.h

@@ -27,6 +27,8 @@
 
 namespace dbps::processing {
 
+// The values in Int96 of low/mid/hi are stored in little endian order.
+// The order of low/mid/hi in the C++ struct should be kept, otherwise the codec will yield incorrect values.
 struct Int96 {
     int32_t lo;
     int32_t mid;