Fix issue with converting uint32_t to int32_t in message params

storm-devs · Mar 5, 2022 · a0d53db · a0d53db
1 parent 4bdc11f
commit a0d53db
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/libs/battle_interface/src/log_and_action.cpp b/src/libs/battle_interface/src/log_and_action.cpp
@@ -48,7 +48,7 @@ bool ILogAndActions::Init()
     }
     D3DVIEWPORT9 vp;
     rs->GetViewport(&vp);
-    core.Event("SetWindowSize", "lll", static_cast<uint32_t>(vp.Width), static_cast<uint32_t>(vp.Height), false);
+    core.Event("SetWindowSize", "lll", static_cast<int32_t>(vp.Width), static_cast<int32_t>(vp.Height), false);
     g_ILogAndActions = GetId();
     return true;
 }

diff --git a/src/libs/common/include/message.h b/src/libs/common/include/message.h
@@ -28,6 +28,13 @@ inline MessageParam convertMessageParam(const bool value)
     return {value ? 1 : 0};
 }
 
+// Convert uint32_t to int32_t to prevent conversion issues
+template<>
+inline MessageParam convertMessageParam(const uint32_t value)
+{
+    return MessageParam(static_cast<int32_t>(value));
+}
+
 } // namespace detail
 
 } // namespace storm