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

Nim: Fix handling of project settings

Browse Source 
Removes unused duplicate of nim specific project settings
and uses the nimble specific project settings additionally
inside nimble projects.

Change-Id: Ibbda76ab41538a0ebdf0a395503c519fcac93460
Reviewed-by: Filippo Cucchetto <filippocucchetto@gmail.com>
Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
Christian Stenger
2020-09-09 16:14:16 +02:00
parent 495c08a3eb
commit 512c0381f3
3 changed files with 7 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/plugins/nim/project/nimblebuildsystem.cpp
View File

@@ -130,6 +130,10 @@ NimbleBuildSystem::NimbleBuildSystem(Target *target)
requestDelayedParse();
});
connect(target->project(), &ProjectExplorer::Project::settingsLoaded,
this, &NimbleBuildSystem::loadSettings);
connect(target->project(), &ProjectExplorer::Project::aboutToSaveSettings,
this, &NimbleBuildSystem::saveSettings);
requestDelayedParse();
}
@@ -190,6 +194,7 @@ NimbleMetadata NimbleBuildSystem::metadata() const
void NimbleBuildSystem::saveSettings()
{
// only handles nimble specific settings - NimProjectScanner handles general settings
QStringList result;
for (const NimbleTask &task : m_tasks) {
result.push_back(task.name);
@@ -201,17 +206,15 @@ void NimbleBuildSystem::saveSettings()
void NimbleBuildSystem::loadSettings()
{
// only handles nimble specific settings - NimProjectScanner handles general settings
QStringList list = project()->namedSettings(C_NIMBLEPROJECT_TASKS).toStringList();
m_tasks.clear();
if (list.size() % 2 != 0)
return;
std::vector<NimbleTask> result;
for (int i = 0; i < list.size(); i += 2)
result.push_back({list[i], list[i + 1]});
requestParse();
m_tasks.push_back({list[i], list[i + 1]});
}
bool NimbleBuildSystem::supportsAction(Node *context, ProjectAction action, const Node *node) const

16
src/plugins/nim/project/nimbuildsystem.cpp
View File

@@ -213,22 +213,6 @@ FilePath nimblePathFromKit(Kit *kit)
return nimbleFromKit.exists() ? nimbleFromKit.canonicalPath() : FilePath::fromString(nimbleFromPath);
}
void NimBuildSystem::loadSettings()
{
QVariantMap settings = project()->namedSettings(SETTINGS_KEY).toMap();
if (settings.contains(EXCLUDED_FILES_KEY))
m_projectScanner.setExcludedFiles(settings.value(EXCLUDED_FILES_KEY, m_projectScanner.excludedFiles()).toStringList());
requestParse();
}
void NimBuildSystem::saveSettings()
{
QVariantMap settings;
settings.insert(EXCLUDED_FILES_KEY, m_projectScanner.excludedFiles());
project()->setNamedSettings(SETTINGS_KEY, settings);
}
bool NimBuildSystem::supportsAction(Node *context, ProjectAction action, const Node *node) const
{
if (node->asFileNode()) {

3
src/plugins/nim/project/nimbuildsystem.h
View File

@@ -92,9 +92,6 @@ public:
void triggerParsing() override;
protected:
void loadSettings();
void saveSettings();
ParseGuard m_guard;
NimProjectScanner m_projectScanner;
};