chore: Fix errors/warnings in GCC13 (#1155)

* chore: Fix errors/warnings in GCC13

* OpResult: Add non const overloads to */-> so we can std::move from the result

* Run clang-format on file

src/core/small_string.h

@@ -3,6 +3,7 @@
 //
 #pragma once
 
+#include <cstdint>
 #include <string_view>
 
 #include "core/core_types.h"

src/core/string_map.h

@@ -4,6 +4,8 @@
 
 #pragma once
 
+#include <string_view>
+
 #include "core/dense_set.h"
 
 extern "C" {

src/core/string_set.h

@@ -7,6 +7,8 @@
 #include <cstdint>
 #include <functional>
 #include <optional>
+#include <string>
+#include <string_view>
 
 #include "core/dense_set.h"
 

src/facade/memcache_parser.h

@@ -4,6 +4,7 @@
 
 #pragma once
 
+#include <cstdint>
 #include <string_view>
 #include <vector>
 
@@ -48,7 +49,7 @@ class MemcacheParser {
 
     union {
       uint64_t cas_unique = 0;  // for CAS COMMAND
-      uint64_t delta;          // for DECR/INCR commands.
+      uint64_t delta;           // for DECR/INCR commands.
     };
 
     uint32_t expire_ts = 0;  // relative time in seconds.
@@ -75,4 +76,4 @@ class MemcacheParser {
  private:
 };
 
-}  // namespace dfly
+}  // namespace facade

src/facade/op_status.h

@@ -4,6 +4,7 @@
 
 #pragma once
 
+#include <cstdint>
 #include <ostream>
 
 namespace facade {
@@ -75,6 +76,14 @@ template <typename V> class OpResult : public OpResultBase {
     return status() == OpStatus::OK ? v_ : v;
   }
 
+  V* operator->() {
+    return &v_;
+  }
+
+  V& operator*() {
+    return v_;
+  }
+
   const V* operator->() const {
     return &v_;
   }

src/server/main_service.cc

@@ -1057,7 +1057,7 @@ optional<CapturingReplyBuilder::Payload> Service::FlushEvalAsyncCmds(ConnectionC
   info->async_cmds_heap_mem = 0;
   info->async_cmds.clear();
 
-  auto reply = move(crb.Take());
+  auto reply = crb.Take();
   return CapturingReplyBuilder::GetError(reply) ? make_optional(move(reply)) : nullopt;
 }