feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

QMakeProjectManager: Avoid using keys()

Browse Source 
Instead, iterate directly over the container.

Change-Id: I589e87b0ed86be778aec7679bedab28d709a7697
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
This commit is contained in:
Jarek Kobus
2024-01-09 00:15:38 +01:00
parent 2d422e33f0
commit 7b32c47494
1 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/plugins/qmakeprojectmanager/qmakeparsernodes.cpp
View File

@@ -572,18 +572,15 @@ bool QmakePriFile::addFiles(const FilePaths &filePaths, FilePaths *notAdded)
bool QmakePriFile::removeFiles(const FilePaths &filePaths, FilePaths *notRemoved)
{
FilePaths failedFiles;
using TypeFileMap = QMap<QString, FilePaths>;
// Split into lists by file type and bulk-add them.
TypeFileMap typeFileMap;
QMap<QString, FilePaths> typeFileMap;
for (const FilePath &file : filePaths) {
const MimeType mt = Utils::mimeTypeForFile(file);
typeFileMap[mt.name()] << file;
}
const QStringList types = typeFileMap.keys();
for (const QString &type : types) {
const FilePaths typeFiles = typeFileMap.value(type);
changeFiles(type, typeFiles, &failedFiles, RemoveFromProFile);
FilePaths failedFiles;
for (auto it = typeFileMap.cbegin(); it != typeFileMap.cend(); ++it) {
changeFiles(it.key(), *it, &failedFiles, RemoveFromProFile);
if (notRemoved)
*notRemoved = failedFiles;
}