walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

GUI: cleanup the code

This commit is contained in:
Daniel Marjamäki 2018-10-22 21:00:04 +02:00
parent ae657d49e8
commit 4533eb0bd6
1 changed files with 19 additions and 53 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
gui/projectfiledialog.cpp
View File

@ -34,6 +34,18 @@
#include "library.h" #include "library.h"
#include "platforms.h" #include "platforms.h"
/** Return paths from QListWidget */
static QStringList getPaths(const QListWidget *list)
{
const int count = list->count();
QStringList paths;
for (int i = 0; i < count; i++) {
QListWidgetItem *item = list->item(i);
paths << QDir::fromNativeSeparators(item->text());
}
return paths;
}
/** Platforms shown in the platform combobox */ /** Platforms shown in the platform combobox */
static const cppcheck::Platform::PlatformType builtinPlatforms[] = { static const cppcheck::Platform::PlatformType builtinPlatforms[] = {
cppcheck::Platform::Native, cppcheck::Platform::Native,
@ -253,14 +265,7 @@ void ProjectFileDialog::loadFromProjectFile(const ProjectFile *projectFile)
mUI.mToolClangTidy->setText(tr("Clang-tidy (not found)")); mUI.mToolClangTidy->setText(tr("Clang-tidy (not found)"));
mUI.mToolClangTidy->setEnabled(false); mUI.mToolClangTidy->setEnabled(false);
} }
QString tags; mUI.mEditTags->setText(projectFile->getTags().join(';'));
foreach (const QString tag, projectFile->getTags()) {
if (tags.isEmpty())
tags = tag;
else
tags += ';' + tag;
}
mUI.mEditTags->setText(tags);
updatePathsAndDefines(); updatePathsAndDefines();
} }
@ -298,9 +303,7 @@ void ProjectFileDialog::saveToProjectFile(ProjectFile *projectFile) const
projectFile->setAddons(list); projectFile->setAddons(list);
projectFile->setClangAnalyzer(mUI.mToolClangAnalyzer->isChecked()); projectFile->setClangAnalyzer(mUI.mToolClangAnalyzer->isChecked());
projectFile->setClangTidy(mUI.mToolClangTidy->isChecked()); projectFile->setClangTidy(mUI.mToolClangTidy->isChecked());
QStringList tags(mUI.mEditTags->text().split(";")); projectFile->setTags(mUI.mEditTags->text().split(";", QString::SkipEmptyParts));
tags.removeAll(QString());
projectFile->setTags(tags);
} }
void ProjectFileDialog::ok() void ProjectFileDialog::ok()
@ -436,30 +439,14 @@ QString ProjectFileDialog::getBuildDir() const
return mUI.mEditBuildDir->text(); return mUI.mEditBuildDir->text();
} }
QStringList ProjectFileDialog::getIncludePaths() const QStringList ProjectFileDialog::getIncludePaths() const
{ {
const int count = mUI.mListIncludeDirs->count(); return getPaths(mUI.mListIncludeDirs);
QStringList includePaths;
for (int i = 0; i < count; i++) {
QListWidgetItem *item = mUI.mListIncludeDirs->item(i);
includePaths << QDir::fromNativeSeparators(item->text());
}
return includePaths;
} }
QStringList ProjectFileDialog::getDefines() const QStringList ProjectFileDialog::getDefines() const
{ {
QString define = mUI.mEditDefines->text(); return mUI.mEditDefines->text().trimmed().split(QRegExp("\\s*;\\s*"), QString::SkipEmptyParts);
QStringList defines;
if (!define.isEmpty()) {
define = define.trimmed();
if (define.indexOf(';') != -1)
defines = define.split(";");
else
defines.append(define);
}
return defines;
} }
QStringList ProjectFileDialog::getUndefines() const QStringList ProjectFileDialog::getUndefines() const
@ -472,24 +459,12 @@ QStringList ProjectFileDialog::getUndefines() const
QStringList ProjectFileDialog::getCheckPaths() const QStringList ProjectFileDialog::getCheckPaths() const
{ {
const int count = mUI.mListCheckPaths->count(); return getPaths(mUI.mListCheckPaths);
QStringList paths;
for (int i = 0; i < count; i++) {
QListWidgetItem *item = mUI.mListCheckPaths->item(i);
paths << QDir::fromNativeSeparators(item->text());
}
return paths;
} }
QStringList ProjectFileDialog::getExcludedPaths() const QStringList ProjectFileDialog::getExcludedPaths() const
{ {
const int count = mUI.mListExcludedPaths->count(); return getPaths(mUI.mListExcludedPaths);
QStringList paths;
for (int i = 0; i < count; i++) {
QListWidgetItem *item = mUI.mListExcludedPaths->item(i);
paths << QDir::fromNativeSeparators(item->text());
}
return paths;
} }
QStringList ProjectFileDialog::getLibraries() const QStringList ProjectFileDialog::getLibraries() const
@ -526,16 +501,7 @@ void ProjectFileDialog::setIncludepaths(const QStringList &includes)
void ProjectFileDialog::setDefines(const QStringList &defines) void ProjectFileDialog::setDefines(const QStringList &defines)
{ {
QString definestr; mUI.mEditDefines->setText(defines.join(";"));
QString define;
foreach (define, defines) {
definestr += define;
definestr += ";";
}
// Remove ; from the end of the string
if (definestr.endsWith(';'))
definestr = definestr.left(definestr.length() - 1);
mUI.mEditDefines->setText(definestr);
} }
void ProjectFileDialog::setUndefines(const QStringList &undefines) void ProjectFileDialog::setUndefines(const QStringList &undefines)