refs #12834 - du-duplicate some input paths by comparing simplified paths

Author: firewave

cli/cmdlineparser.cpp

@@ -269,10 +269,10 @@ bool CmdLineParser::fillSettingsFromArgs(int argc, const char* const argv[])
         {
             auto it = filesResolved.begin();
             while (it != filesResolved.end()) {
-                const std::string& name = it->path();
+                const std::string& sname = it->spath();
                 // TODO: log if duplicated files were dropped
                 filesResolved.erase(std::remove_if(std::next(it), filesResolved.end(), [&](const FileWithDetails& entry) {
-                    return entry.path() == name;
+                    return entry.spath() == sname;
                 }), filesResolved.end());
                 ++it;
             }

test/cli/more-projects_test.py

@@ -687,28 +687,13 @@ def test_project_file_duplicate_3(tmpdir):
 </project>""".format(in_file_a, in_file_b, in_file_c, in_file_d, in_file_e, in_file_f, tmpdir))
 
     args = ['--project={}'.format(project_file)]
-    args.append('-j1') # TODO: remove when fixed
 
     exitcode, stdout, stderr = cppcheck(args, cwd=tmpdir)
     assert exitcode == 0
     lines = stdout.splitlines()
-    # TODO: only a single file should be checked
-    if sys.platform == 'win32':
-        assert lines == [
-            'Checking {} ...'.format(test_file_a),
-            '1/3 files checked 33% done',
-            'Checking {} ...'.format(test_file_a),
-            '2/3 files checked 66% done',
-            'Checking {} ...'.format(test_file_a),
-            '3/3 files checked 100% done'
-        ]
-    else:
-        assert lines == [
-            'Checking {} ...'.format(test_file_a),
-            '1/2 files checked 50% done',
-            'Checking {} ...'.format(test_file_a),
-            '2/2 files checked 100% done'
-        ]
+    assert lines == [
+        'Checking {} ...'.format(test_file_a)
+    ]
     assert stderr == ''
 
 
@@ -756,19 +741,12 @@ def test_project_file_duplicate_4(tmpdir):
                      args2[0], args2[1], args2[2], args2[3], args2[4], args2[5], args2[6]))
 
     args = ['--project={}'.format(project_file)]
-    args.append('-j1') # TODO: remove when fixed
 
     exitcode, stdout, stderr = cppcheck(args, cwd=tmpdir)
     assert exitcode == 0
     lines = stdout.splitlines()
-    # TODO: only a single file should be checked
     assert lines == [
-        'Checking {} ...'.format(test_file_a),
-        '1/3 files checked 33% done',
-        'Checking {} ...'.format(test_file_a),
-        '2/3 files checked 66% done',
-        'Checking {} ...'.format(test_file_a),
-        '3/3 files checked 100% done'
+        'Checking {} ...'.format(test_file_a)
     ]
     assert stderr == ''
 

test/cli/other_test.py

@@ -1236,32 +1236,12 @@ def test_file_duplicate_3(tmpdir):
     assert exitcode == 0, stdout if stdout else stderr
     lines = stdout.splitlines()
     # TODO: only a single file should be checked
-    if sys.platform == 'win32':
-        assert lines == [
-            'Checking {} ...'.format('a.c'),
-            '1/6 files checked 16% done',
-            'Checking {} ...'.format('a.c'),
-            '2/6 files checked 33% done',
-            'Checking {} ...'.format('a.c'),
-            '3/6 files checked 50% done',
-            'Checking {} ...'.format(test_file_a),
-            '4/6 files checked 66% done',
-            'Checking {} ...'.format(test_file_a),
-            '5/6 files checked 83% done',
-            'Checking {} ...'.format(test_file_a),
-            '6/6 files checked 100% done'
-        ]
-    else:
-        assert lines == [
-            'Checking {} ...'.format('a.c'),
-            '1/4 files checked 25% done',
-            'Checking {} ...'.format('a.c'),
-            '2/4 files checked 50% done',
-            'Checking {} ...'.format(test_file_a),
-            '3/4 files checked 75% done',
-            'Checking {} ...'.format(test_file_a),
-            '4/4 files checked 100% done'
-        ]
+    assert lines == [
+        'Checking {} ...'.format('a.c'),
+        '1/2 files checked 50% done',
+        'Checking {} ...'.format(test_file_a),
+        '2/2 files checked 100% done'
+    ]
     assert stderr == ''
 
 
@@ -1292,17 +1272,9 @@ def test_file_duplicate_4(tmpdir):
     # TODO: only a single file should be checked
     assert lines == [
         'Checking {} ...'.format('a.c'),
-        '1/6 files checked 16% done',
-        'Checking {} ...'.format('a.c'),
-        '2/6 files checked 33% done',
-        'Checking {} ...'.format('a.c'),
-        '3/6 files checked 50% done',
-        'Checking {} ...'.format(test_file_a),
-        '4/6 files checked 66% done',
-        'Checking {} ...'.format(test_file_a),
-        '5/6 files checked 83% done',
+        '1/2 files checked 50% done',
         'Checking {} ...'.format(test_file_a),
-        '6/6 files checked 100% done'
+        '2/2 files checked 100% done'
     ]
     assert stderr == ''