Fix warnings

olympus-robotics · Sep 30, 2024 · d9f4afc · d9f4afc
1 parent 602216b
commit d9f4afc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/modules/serdes/include/hephaestus/serdes/protobuf/enums.h b/modules/serdes/include/hephaestus/serdes/protobuf/enums.h
@@ -61,10 +61,11 @@ template <EnumType ProtoT, EnumType T>
       fmt::format("{}_{}", proto_prefix, magic_enum::enum_name(e));  // ClassName_EnumName_ENUM_VALUE
 
   auto proto_enum = magic_enum::enum_cast<ProtoT>(proto_enum_name);
-  heph::throwExceptionIf<heph::InvalidParameterException>(
-      !proto_enum.has_value(),
-      fmt::format("The proto enum does not contain the requested key {}. Proto enum keys are\n{}",
-                  proto_enum_name, utils::format::toString(magic_enum::enum_names<ProtoT>())));
+  if (!proto_enum.has_value()) {
+    heph::throwException<heph::InvalidParameterException>(
+        fmt::format("The proto enum does not contain the requested key {}. Proto enum keys are\n{}",
+                    proto_enum_name, utils::format::toString(magic_enum::enum_names<ProtoT>())));
+  }
 
   return proto_enum.value();
 }

diff --git a/modules/types_proto/src/dummy_type.cpp b/modules/types_proto/src/dummy_type.cpp
@@ -7,9 +7,6 @@
 #include <cstddef>
 #include <vector>
 
-#include <fmt/core.h>
-#include <magic_enum.hpp>
-
 #include "hephaestus/serdes/protobuf/enums.h"
 #include "hephaestus/types/dummy_type.h"
 #include "hephaestus/types/proto/dummy_type.pb.h"