Minor changes

Author: RenardDev

Detours.cpp

@@ -7051,7 +7051,7 @@ namespace Detours {
 
 			m_unRegionCapacity = ri.m_unSize;
 			m_pRegionAddress = pBaseAddress;
-			
+
 			auto vecPages = __get_pages_info(pBaseAddress, ri.m_unSize);
 			for (auto& PageInfo : vecPages) {
 				m_Pages.emplace_back(PageInfo.m_pBaseAddress, false);

Detours.h

@@ -48,40 +48,35 @@
 
 // MSVC - Sections
 
-#define DECLARE_DATA_SECTION(NAME) __pragma(section(NAME, read))
-#define DECLARE_CODE_SECTION(NAME) __pragma(section(NAME, execute))
- 
+#define DECLARE_SECTION(NAME) __pragma(section(NAME, execute))
+
 #define SECTION_READONLY "R"
 #define SECTION_READWRITE "RW"
 #define SECTION_EXECUTE_READ "ER"
 #define SECTION_EXECUTE_READWRITE "ERW"
 #define CHANGE_SECTION_ATTRIBUTES(NAME, ATTRIBUTES) LINKER_OPTION("/SECTION:" NAME "," ATTRIBUTES)
- 
-#define DEFINE_DATA_SECTION(NAME, ATTRIBUTES)   \
-	DECLARE_DATA_SECTION(NAME)                  \
-	CHANGE_SECTION_ATTRIBUTES(NAME, ATTRIBUTES)
- 
-#define DEFINE_CODE_SECTION(NAME, ATTRIBUTES)   \
-	DECLARE_CODE_SECTION(NAME)                  \
+
+#define DEFINE_SECTION(NAME, ATTRIBUTES)   \
+	DECLARE_SECTION(NAME)                  \
 	CHANGE_SECTION_ATTRIBUTES(NAME, ATTRIBUTES)
- 
+
 #define MERGE_SECTION(FROM, TO) LINKER_OPTION("/MERGE:" FROM "=" TO)
- 
+
 #define DEFINE_DATA_IN_SECTION(NAME) __declspec(allocate(NAME))
 #define DEFINE_CODE_IN_SECTION(NAME) __declspec(code_seg(NAME))
- 
+
 // MSVC - Optimization
- 
+
 #define DISABLE_OPTIMIZATION(OPTION) \
 	__pragma(optimize(OPTION, off))
- 
+
 #define ENABLE_OPTIMIZATION(OPTION) \
 	__pragma(optimize(OPTION, on))
- 
+
 #define DISABLE_OPTIMIZATION_BEGIN(OPTION) \
 	DISABLE_OPTIMIZATION(OPTION)           \
 	extern "C++"
- 
+
 #define DISABLE_OPTIMIZATION_END(OPTION) \
 	ENABLE_OPTIMIZATION(OPTION)
 

main.cpp

@@ -155,8 +155,8 @@ static inline const char* nameof() {
 	return typeid(T).raw_name();
 }
 
-DEFINE_DATA_SECTION(".cdata", SECTION_READWRITE)
-DEFINE_CODE_SECTION(".ctext", SECTION_EXECUTE_READ)
+DEFINE_SECTION(".cdata", SECTION_READWRITE)
+DEFINE_SECTION(".ctext", SECTION_EXECUTE_READ)
 
 DEFINE_DATA_IN_SECTION(".cdata") __declspec(dllexport) BaseTestingRTTI* g_pBaseTestingRTTI = nullptr;
 DEFINE_DATA_IN_SECTION(".cdata") __declspec(dllexport) TestingRTTI* g_pTestingRTTI = nullptr;
@@ -2417,7 +2417,7 @@ TEST_SUITE("Detours::Hook") {
 		CHECK(reinterpret_cast<fnFoo>(pVTable[0])(g_pTestingRTTI, nullptr) == false);
 		CHECK(fooHook.UnHook() == true);
 		CHECK(fooHook.Release() == true);
-		
+
 		CHECK(booHook.Set(pVTable, 1) == true);
 		CHECK(booHook.Hook(boo_Hook) == true);
 		CHECK(reinterpret_cast<fnBoo>(pVTable[1])(g_pTestingRTTI, nullptr) == true);