#3

All hooks are installed now.
Set warning level to 4, all warnings treats as errors now.

CMakeLists.txt

@@ -7,6 +7,7 @@ file(GLOB_RECURSE INC src/*.hpp src/*.h)
 
 add_library(${PROJECT_NAME} SHARED ${SRC} ${INC})
 target_include_directories(${PROJECT_NAME} PRIVATE src)
+target_compile_options(${PROJECT_NAME} PRIVATE /WX /W4)
 
 source_group(TREE ${CMAKE_CURRENT_SOURCE_DIR} FILES ${SRC} ${INC})
 

src/dllmain.cpp

@@ -4,7 +4,7 @@
 #include "ep_patch.hpp"
 
 
-BOOL APIENTRY DllMain(HMODULE hModule, DWORD ul_reason_for_call, LPVOID lpReserved)
+BOOL APIENTRY DllMain(HMODULE /*hModule*/, DWORD ul_reason_for_call, LPVOID /*lpReserved*/)
 {
     switch (ul_reason_for_call)
     {

src/hooks/base_hook.cpp

@@ -10,8 +10,7 @@ namespace hooks
     CBaseHook::CBaseHook(uint32_t Address_, uint32_t Size_)
         : m_StartAddress(Address_)
         , m_EndAddress(Address_ + Size_ - 1)
-    {
-    }
+    {}
 
 
     CBaseHook::~CBaseHook()

src/hooks/base_hook.hpp

@@ -67,8 +67,8 @@ namespace hooks
         void SetInstalled(bool bState_);
 
     private:
-        uint32_t m_StartAddress;    // Address of first byte.
-        uint32_t m_EndAddress;      // Address of last byte.
-        bool m_bInstalled;      // This hook is installed.
+        uint32_t m_StartAddress = 0;    // Address of first byte.
+        uint32_t m_EndAddress = 0;      // Address of last byte.
+        bool m_bInstalled = false;      // This hook is installed.
     };
 }

src/hooks/jump_hook.hpp

@@ -49,7 +49,7 @@ namespace hooks
 #pragma pack(pop)
 
         SJumpHook m_OriginalInstruction;
-        uint32_t m_iRawTarget;
-        bool m_bCallHook;
+        uint32_t m_iRawTarget = 0;
+        bool m_bCallHook = false;
     };
 }

src/hooks/native_function.hpp

@@ -39,6 +39,6 @@ namespace hooks
             m_Pointer = reinterpret_cast<Type_>(StartAddress_);
         }
 
-        Type_ m_Pointer;
+        Type_ m_Pointer = nullptr;
     };
 }

src/hooks/replace_function.hpp

@@ -49,6 +49,6 @@ namespace hooks
 #pragma pack(pop)
 
         SJumpHook m_Original;
-        uint32_t m_iRawTarget;
+        uint32_t m_iRawTarget = 0;
     };
 }

src/module_manager.hpp

@@ -32,5 +32,6 @@ class CModuleManager
 private:
     bool isModuleExist(const std::type_index& TI_) const;
 
+private:
     std::vector<std::pair<std::type_index, std::shared_ptr<CModule>>> m_vModules;
 };

src/modules/database.cpp

@@ -139,7 +139,7 @@ namespace modules
                 *Dst_ = nullptr;
                 if (g_streamPosIndex != 1)
                 {
-                    QASSERTFMT(g_streamDelayIndex < 0x1000, "g_streamDelayIndex doesn't index ARRAY_COUNT( g_streamDelayArray )\n\t%i not in [0, %i)", g_streamDelayIndex, 4096);
+                    QASSERTFMT(g_streamDelayIndex < 0x1000, "g_streamDelayIndex doesn't index ARRAY_COUNT( g_streamDelayArray )\n\t%i not in [0, %i)", g_streamDelayIndex.Raw(), 4096);
                     g_streamDelayArray[2 * g_streamDelayIndex] = (int)Src_;
                     g_streamDelayArray[2 * g_streamDelayIndex + 1] = Size_;
                     g_streamDelayIndex += 1;

src/proxy_lib.hpp

@@ -17,5 +17,5 @@ class CProxyLib
     CProxyLib();
     ~CProxyLib();
 
-    HMODULE m_hOriginalLib;
+    HMODULE m_hOriginalLib = nullptr;
 };