add --deduplicate option for controlling deduplication of FileSettings

Author: ludviggunne

cli/cmdlineparser.cpp

@@ -250,7 +250,7 @@ bool CmdLineParser::fillSettingsFromArgs(int argc, const char* const argv[])
             fs.computeHash();
         }
 
-        {
+        if (mSettings.deduplicate) {
             auto it = fileSettings.begin();
             while (it != fileSettings.end()) {
                 fileSettings.erase(std::remove_if(std::next(it), fileSettings.end(), [&](const FileSettings& fs) {
@@ -651,6 +651,9 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
         else if (std::strcmp(argv[i], "--debug-clang-output") == 0)
             mSettings.debugClangOutput = true;
 
+        else if (std::strcmp(argv[i], "--deduplicate") == 0)
+            mSettings.deduplicate = true;
+
         // Show debug messages for ignored files
         else if (std::strcmp(argv[i], "--debug-ignore") == 0)
             mSettings.debugignore = true;
@@ -1718,6 +1721,8 @@ void CmdLineParser::printHelp() const
         "                         be considered for evaluation.\n"
         "    --config-excludes-file=<file>\n"
         "                         A file that contains a list of config-excludes\n"
+        "    --deduplicate        Remove duplicate entries when reading project files.\n"
+        "                         Takes both compilation flags and filenames in to account.\n"
         "    --disable=<id>       Disable individual checks.\n"
         "                         Please refer to the documentation of --enable=<id>\n"
         "                         for further details.\n"

lib/settings.h

@@ -220,6 +220,9 @@ class CPPCHECKLIB WARN_UNUSED Settings {
     /** @brief Do not filter duplicated errors. */
     bool emitDuplicates{};
 
+    /** @brief De-duplicate files with identical path and config. */
+    bool deduplicate{};
+
     /** @brief Name of the language that is enforced. Empty per default. */
     Standards::Language enforcedLang{};
 

man/cppcheck.1.xml

@@ -150,6 +150,9 @@ man(1), man(7), http://www.tldp.org/HOWTO/Man-Page/
       <arg choice="opt">
         <option>--config-excludes-file=&lt;file&gt;</option>
       </arg>
+      <arg choice="opt">
+        <option>--deduplicate</option>
+      </arg>
       <arg choice="opt">
         <option>--include=&lt;file&gt;</option>
       </arg>
@@ -441,6 +444,14 @@ First given path is searched for contained header files first. If paths are rela
           <para>A file that contains a list of config-excludes.</para>
         </listitem>
       </varlistentry>
+      <varlistentry>
+        <term>
+          <option>--deduplicate</option>
+        </term>
+        <listitem>
+          <para>Remove duplicate entries when reading project files. Takes both compilation flags and filenames in to account.</para>
+        </listitem>
+      </varlistentry>
       <varlistentry>
         <term>
           <option>--include=&lt;file&gt;</option>