Skip to content

Commit c84857f

Browse files
committed
Fixed compilation as shared lib with PCRE: Ensure that sizeof(Settings) is the same with and without HAVE_RULES
Fixed VS10 solution when TinyXml is included (define TIXML_USE_STL).
1 parent 6c2563c commit c84857f

3 files changed

Lines changed: 4 additions & 6 deletions

File tree

cli/cli.vcxproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@
174174
<BufferSecurityCheck>true</BufferSecurityCheck>
175175
<DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
176176
<Optimization>Disabled</Optimization>
177-
<PreprocessorDefinitions>CPPCHECKLIB_IMPORT;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;%(PreprocessorDefinitions)</PreprocessorDefinitions>
177+
<PreprocessorDefinitions>CPPCHECKLIB_IMPORT;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;TIXML_USE_STL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
178178
<RuntimeLibrary>MultiThreadedDebugDLL</RuntimeLibrary>
179179
<SuppressStartupBanner>true</SuppressStartupBanner>
180180
<WarningLevel>Level4</WarningLevel>
@@ -266,7 +266,7 @@
266266
<AdditionalIncludeDirectories>..\lib;..\externals;..\externals\tinyxml;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
267267
<BufferSecurityCheck>false</BufferSecurityCheck>
268268
<Optimization>MaxSpeed</Optimization>
269-
<PreprocessorDefinitions>CPPCHECKLIB_IMPORT;NDEBUG;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;%(PreprocessorDefinitions)</PreprocessorDefinitions>
269+
<PreprocessorDefinitions>CPPCHECKLIB_IMPORT;NDEBUG;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;TIXML_USE_STL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
270270
<RuntimeLibrary>MultiThreadedDLL</RuntimeLibrary>
271271
<SuppressStartupBanner>true</SuppressStartupBanner>
272272
<WarningLevel>Level4</WarningLevel>

lib/cppcheck.vcxproj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@
245245
<DebugInformationFormat>ProgramDatabase</DebugInformationFormat>
246246
<MultiProcessorCompilation>true</MultiProcessorCompilation>
247247
<Optimization>Disabled</Optimization>
248-
<PreprocessorDefinitions>CPPCHECKLIB_EXPORT;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;%(PreprocessorDefinitions)</PreprocessorDefinitions>
248+
<PreprocessorDefinitions>CPPCHECKLIB_EXPORT;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;TIXML_USE_STL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
249249
<WarningLevel>Level4</WarningLevel>
250250
<AdditionalIncludeDirectories>..\externals;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
251251
<DisableSpecificWarnings>4251;4512</DisableSpecificWarnings>
@@ -331,7 +331,7 @@
331331
<FunctionLevelLinking>true</FunctionLevelLinking>
332332
<AdditionalIncludeDirectories>..\externals;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
333333
<DisableSpecificWarnings>4251;4512</DisableSpecificWarnings>
334-
<PreprocessorDefinitions>CPPCHECKLIB_EXPORT;NDEBUG;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;%(PreprocessorDefinitions)</PreprocessorDefinitions>
334+
<PreprocessorDefinitions>CPPCHECKLIB_EXPORT;NDEBUG;WIN32;HAVE_RULES;_CRT_SECURE_NO_WARNINGS;WIN32_LEAN_AND_MEAN;TIXML_USE_STL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
335335
<RuntimeLibrary>MultiThreadedDLL</RuntimeLibrary>
336336
<DebugInformationFormat>
337337
</DebugInformationFormat>

lib/settings.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,6 @@ class CPPCHECKLIB Settings {
171171
/** @brief --report-progress */
172172
bool reportProgress;
173173

174-
#ifdef HAVE_RULES
175174
/** Rule */
176175
class CPPCHECKLIB Rule {
177176
public:
@@ -190,7 +189,6 @@ class CPPCHECKLIB Settings {
190189
* @brief Extra rules
191190
*/
192191
std::list<Rule> rules;
193-
#endif
194192

195193
/** Is the 'configuration checking' wanted? */
196194
bool checkConfiguration;

0 commit comments

Comments
 (0)