Skip to content

Commit

Permalink
[WINDOWS] Make sure warning reporting is not enabled from the core pr…
Browse files Browse the repository at this point in the history
…oject file. (#1419)

As warning reporting is optional, I think it is good to enable WarningReporting builds from the outside. If so the build (by default) does not get
to bulky (lean and mean) and if you want WarningReporting just pass the flag from the outside (if possible :-).

Also the MessageDispatcher.h is no longer part of this deployment, lets kill it :-)
  • Loading branch information
pwielders authored Oct 4, 2023
1 parent ce43523 commit 7f0fd3d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
9 changes: 4 additions & 5 deletions Source/core/core.vcxproj
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@
<ClInclude Include="LockableContainer.h" />
<ClInclude Include="Measurement.h" />
<ClInclude Include="Media.h" />
<ClInclude Include="MessageDispatcher.h" />
<ClInclude Include="MessageException.h" />
<ClInclude Include="MessageStore.h" />
<ClInclude Include="Module.h" />
Expand Down Expand Up @@ -216,7 +215,7 @@
<WarningLevel>Level3</WarningLevel>
<Optimization>Disabled</Optimization>
<SDLCheck>true</SDLCheck>
<PreprocessorDefinitions>__CORE_WARNING_REPORTING__;CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;_DEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions);</PreprocessorDefinitions>
<PreprocessorDefinitions>CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;_DEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions);</PreprocessorDefinitions>
<ConformanceMode>true</ConformanceMode>
<AdditionalIncludeDirectories>
</AdditionalIncludeDirectories>
Expand All @@ -235,7 +234,7 @@
<WarningLevel>Level3</WarningLevel>
<Optimization>Disabled</Optimization>
<SDLCheck>true</SDLCheck>
<PreprocessorDefinitions>__CORE_WARNING_REPORTING__;CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;DEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<PreprocessorDefinitions>CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;DEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<ConformanceMode>true</ConformanceMode>
<AdditionalIncludeDirectories>$(WindowsPath)</AdditionalIncludeDirectories>
<UseStandardPreprocessor>true</UseStandardPreprocessor>
Expand All @@ -255,7 +254,7 @@
<FunctionLevelLinking>true</FunctionLevelLinking>
<IntrinsicFunctions>true</IntrinsicFunctions>
<SDLCheck>true</SDLCheck>
<PreprocessorDefinitions>__CORE_WARNING_REPORTING__;CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;IN32;NDEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<PreprocessorDefinitions>CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;IN32;NDEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<ConformanceMode>true</ConformanceMode>
<AdditionalIncludeDirectories>
</AdditionalIncludeDirectories>
Expand All @@ -278,7 +277,7 @@
<FunctionLevelLinking>true</FunctionLevelLinking>
<IntrinsicFunctions>true</IntrinsicFunctions>
<SDLCheck>true</SDLCheck>
<PreprocessorDefinitions>__CORE_WARNING_REPORTING__;CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;NDEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<PreprocessorDefinitions>CORE_EXPORTS;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;NDEBUG;_WINDOWS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
<ConformanceMode>true</ConformanceMode>
<AdditionalIncludeDirectories>
</AdditionalIncludeDirectories>
Expand Down
3 changes: 0 additions & 3 deletions Source/core/core.vcxproj.filters
Original file line number Diff line number Diff line change
Expand Up @@ -246,9 +246,6 @@
<ClInclude Include="ThreadPool.h">
<Filter>Header Files</Filter>
</ClInclude>
<ClInclude Include="MessageDispatcher.h">
<Filter>Header Files</Filter>
</ClInclude>
<ClInclude Include="MessageStore.h">
<Filter>Header Files</Filter>
</ClInclude>
Expand Down

0 comments on commit 7f0fd3d

Please sign in to comment.