From 6c63c42799e8deeb8877944711a2c29213b70922 Mon Sep 17 00:00:00 2001 From: Cristian Adam Date: Tue, 9 Jul 2024 16:36:24 +0200 Subject: [PATCH] CMakePM: Do not open CMakeLists.txt for adding/renaming/removing files Other build systems like qmake / qbs do not do this. Change-Id: I615ea3556c694e7a18426558c5bf6f3ea142068d Reviewed-by: Alessandro Portale --- .../cmakeprojectmanager/cmakebuildsystem.cpp | 26 +++++++++---------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp b/src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp index 540192d2d21..8a65c45b7f9 100644 --- a/src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp +++ b/src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp @@ -399,12 +399,10 @@ static SnippetAndLocation generateSnippetAndLocationForSources( static expected_str insertSnippetSilently(const FilePath &cmakeFile, const SnippetAndLocation &snippetLocation) { - BaseTextEditor *editor = qobject_cast( - Core::EditorManager::openEditorAt({cmakeFile, - int(snippetLocation.line), - int(snippetLocation.column)}, - Constants::CMAKE_EDITOR_ID, - Core::EditorManager::DoNotMakeVisible)); + BaseTextEditor *editor = qobject_cast(Core::EditorManager::openEditorAt( + {cmakeFile, int(snippetLocation.line), int(snippetLocation.column)}, + Constants::CMAKE_EDITOR_ID, + Core::EditorManager::DoNotMakeVisible | Core::EditorManager::DoNotChangeCurrentEditor)); if (!editor) { return make_unexpected("BaseTextEditor cannot be obtained for " + cmakeFile.toUserOutput() + ":" + QString::number(snippetLocation.line) + ":" @@ -864,12 +862,13 @@ RemovedFilesFromProject CMakeBuildSystem::removeFiles(Node *context, } BaseTextEditor *editor = qobject_cast( - Core::EditorManager::openEditorAt({filePos.value().cmakeFile, - static_cast(filePos.value().argumentPosition.Line), - static_cast(filePos.value().argumentPosition.Column - - 1)}, - Constants::CMAKE_EDITOR_ID, - Core::EditorManager::DoNotMakeVisible)); + Core::EditorManager::openEditorAt( + {filePos.value().cmakeFile, + static_cast(filePos.value().argumentPosition.Line), + static_cast(filePos.value().argumentPosition.Column - 1)}, + Constants::CMAKE_EDITOR_ID, + Core::EditorManager::DoNotMakeVisible + | Core::EditorManager::DoNotChangeCurrentEditor)); if (!editor) { badFiles << file; @@ -976,7 +975,8 @@ bool CMakeBuildSystem::renameFile(Node *context, static_cast(fileToRename->argumentPosition.Line), static_cast(fileToRename->argumentPosition.Column - 1)}, Constants::CMAKE_EDITOR_ID, - Core::EditorManager::DoNotMakeVisible)); + Core::EditorManager::DoNotMakeVisible + | Core::EditorManager::DoNotChangeCurrentEditor)); if (!editor) { qCCritical(cmakeBuildSystemLog).noquote() << "BaseTextEditor cannot be obtained for" << fileToRename->cmakeFile.path()