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

StudioWelcome: use Utils::FilePath and avoid QString round‑trips

Browse Source 
setProjectLocation() is invoked from QML and therefore keeps its QString
parameter. The value is now converted only once to Utils::FilePath.

No functional change.

Change-Id: I1d12f1b5493c6b9be3bc8b6d9e9c7029671b80eb
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
This commit is contained in:
Tim Jenßen
2025-05-09 14:49:45 +02:00
committed by Tim Jenssen
parent 719854f8e8
commit 78c2a142dd
1 changed files with 5 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/plugins/studiowelcome/qdsnewdialog.cpp
View File

@@ -133,7 +133,7 @@ void QdsNewDialog::setProjectName(const QString &name)
void QdsNewDialog::setProjectLocation(const QString &location)
{
m_qmlProjectLocation = Utils::FilePath::fromString(QDir::toNativeSeparators(location));
m_qmlProjectLocation = Utils::FilePath::fromString(location);
m_wizard.setProjectLocation(m_qmlProjectLocation);
}
@@ -238,8 +238,8 @@ void QdsNewDialog::onWizardCreated(QStandardItemModel *screenSizeModel, QStandar
updateScreenSizes();
setProjectName(m_qmlProjectName);
setProjectLocation(m_qmlProjectLocation.toUrlishString());
m_wizard.setProjectName(m_qmlProjectName);
m_wizard.setProjectLocation(m_qmlProjectLocation);
}
}
@@ -330,12 +330,9 @@ void QdsNewDialog::setWizardFactories(QList<Core::IWizardFactory *> factories_,
return; // TODO: some message box?
const Core::IWizardFactory *first = factories.front();
Utils::FilePath projectLocation = first->runPath(defaultLocation);
m_qmlProjectName = uniqueProjectName(projectLocation.toUrlishString());
m_qmlProjectLocation = first->runPath(defaultLocation);
m_qmlProjectName = uniqueProjectName(m_qmlProjectLocation.toUrlishString());
emit projectNameChanged(); // So that QML knows to update the field
m_qmlProjectLocation = Utils::FilePath::fromString(QDir::toNativeSeparators(projectLocation.toUrlishString()));
emit projectLocationChanged(); // So that QML knows to update the field
/* NOTE: