remove Pragma.h

Author: ab9rf

CMakeLists.txt

@@ -94,16 +94,17 @@ if(MSVC)
     add_compile_options("/wd4267")
 
     # suppress C4251 - VC++ will warn when exporting an entire class which contains members
-    # referencing unexported compound types is unsafe. because we don't guarantee a stable API
-    # for our exports, we don't really care about this, so we choose to be lazy and continue to
-    # export entire classes instead of exporting on a method-by-method basis
+    # referencing unexported compound types as this is potentially unsafe. because we don't
+    # guarantee a stable ABI for exports, we don't really care about this, and so we choose to
+    # be lazy and continue to export entire classes instead of exporting on a method-by-method basis
     add_compile_options("/wd4251")
 
     # suppress C4068 - VC++ will warn for unknown pragmas by default. this is equivalent to gcc
-    # -Wno-unknown-pragmas. we could work around this with sufficiently complex macros
+    # -Wno-unknown-pragmas (which is enabled for gcc below).
+    # we could work around this with sufficiently complex macros
     add_compile_options("/wd4068")
 
-    # suppress C4244 - VC++ warns by default (with /Wall) about narrowing conversions that may lose data
+    # suppress C4244 - VC++ warns by default (with /W3) about narrowing conversions that may lose data
     # (such as double -> int or int32_t -> int16_t). dfhack has many of these, mostly related to Lua
     # this is equivalent to gcc -Wno_conversions which is the default as gcc -Wall doesn't enable -Wconversions
     add_compile_options("/wd4244")

library/CMakeLists.txt

@@ -69,12 +69,10 @@ set(MAIN_HEADERS
     include/Memory.h
     include/MiscUtils.h
     include/Module.h
-    include/Pragma.h
     include/MemAccess.h
     include/ModuleFactory.h
     include/PluginManager.h
     include/PluginStatics.h
-    include/Pragma.h
     include/RemoteClient.h
     include/RemoteServer.h
     include/RemoteTools.h

library/include/BitArray.h

@@ -23,7 +23,6 @@ distribution.
 */
 
 #pragma once
-#include "Pragma.h"
 #include "Export.h"
 #include "Error.h"
 #include <stdint.h>

library/include/ColorText.h

@@ -23,7 +23,6 @@ distribution.
 */
 
 #pragma once
-#include "Pragma.h"
 #include "Export.h"
 
 #include <list>

library/include/Console.h

@@ -23,7 +23,6 @@ distribution.
 */
 
 #pragma once
-#include "Pragma.h"
 #include "Export.h"
 #include "ColorText.h"
 #include <atomic>

library/include/Core.h

@@ -24,8 +24,6 @@ distribution.
 
 #pragma once
 
-#include "Pragma.h"
-
 #include "Console.h"
 #include "Export.h"
 #include "Hooks.h"

library/include/Error.h

@@ -30,7 +30,6 @@ distribution.
 
 #include "Export.h"
 #include "MiscUtils.h"
-#include "Pragma.h"
 
 namespace DFHack
 {

library/include/Internal.h

@@ -34,8 +34,5 @@ distribution.
     #define _FILE_OFFSET_BITS 64
 #endif
 
-// one file for telling the MSVC compiler where it can shove its pointless warnings
-#include "Pragma.h"
-
 // C99 integer types
 #include <stdint.h>

library/include/MemAccess.h

@@ -28,7 +28,6 @@ distribution.
 #ifndef PROCESS_H_INCLUDED
 #define PROCESS_H_INCLUDED
 
-#include "Pragma.h"
 #include "Export.h"
 #include <iostream>
 #include <cstring>