Unnecessary <WinSock2.h> include in hkMemoryRouter causing community false alarms
<img width="1815" height="1202" alt="Image" src="https://github.com/user-attachments/assets/cad2978c-8de2-48e2-a63f-ab800478322d" />
<img width="1495" height="663" alt="Image" src="https://github.com/user-attachments/assets/474bcba2-88b0-4e57-80f0-62ff77f8c18a" />
Hi, thanks for the great work on this library.
I noticed that in `hkMemoryRouter` (header/cpp), there is an `#include <WinSock2.h>`.
It appears that this include is completely unused for the actual Havok memory routing logic, as `TlsGetValue` and the `DWORD` type only require `processthreadsapi.h` (or the standard `<windows.h>`).
**The Issue:**
Because this networking header is included in the compiled DLLs of mods using this template, some hardcore players and community members are reverse-engineering or scanning the DLLs and falsely accusing mod authors of sneaking in telemetry, spyware, or network backdoors into a single-player game. This is currently causing a lot of unnecessary drama in the modding community.
**Suggestion:**
Could we please remove the `#include <WinSock2.h>` if it is indeed redundant? This would greatly help downstream modders avoid baseless paranoia from players.
I've attached a screenshot for reference. Thanks!
3 条评论