QmlDesigner: More const ref usage

Change-Id: Id54eb4af54bb31c51d1fdbc52ecb0976147fe7d0
Reviewed-by: Marco Bubke <marco.bubke@qt.io>
This commit is contained in:
Burak Hancerli
2025-03-25 13:54:36 +01:00
parent fbe8340861
commit b0a13b9213
2 changed files with 12 additions and 12 deletions

View File

@@ -73,7 +73,7 @@ QString previewBrokenImagePath()
return Core::ICore::resourcePath("qmldesigner/welcomepage/images/noPreview.png").toUrlishString(); return Core::ICore::resourcePath("qmldesigner/welcomepage/images/noPreview.png").toUrlishString();
} }
::QmlProjectManager::QmlBuildSystem *getQmlBuildSystem(::ProjectExplorer::Target *target) ::QmlProjectManager::QmlBuildSystem *getQmlBuildSystem(const ::ProjectExplorer::Target *target)
{ {
return qobject_cast<::QmlProjectManager::QmlBuildSystem *>(target->buildSystem()); return qobject_cast<::QmlProjectManager::QmlBuildSystem *>(target->buildSystem());
} }
@@ -193,7 +193,7 @@ Sqlite::JournalMode projectStorageJournalMode()
return QLibraryInfo::path(QLibraryInfo::QmlImportsPath); return QLibraryInfo::path(QLibraryInfo::QmlImportsPath);
} }
[[maybe_unused]] QString qmlPath(::ProjectExplorer::Project *project) [[maybe_unused]] QString qmlPath(const ::ProjectExplorer::Project *project)
{ {
auto qt = QtSupport::QtKitAspect::qtVersion(project->activeKit()); auto qt = QtSupport::QtKitAspect::qtVersion(project->activeKit());
if (qt) if (qt)
@@ -205,7 +205,7 @@ Sqlite::JournalMode projectStorageJournalMode()
class ProjectStorageData class ProjectStorageData
{ {
public: public:
ProjectStorageData(::ProjectExplorer::Project *project, PathCacheType &pathCache) ProjectStorageData(const ::ProjectExplorer::Project *project, PathCacheType &pathCache)
: database{project->projectDirectory().pathAppended("projectstorage.db").toUrlishString(), : database{project->projectDirectory().pathAppended("projectstorage.db").toUrlishString(),
projectStorageJournalMode()} projectStorageJournalMode()}
, errorNotifier{pathCache} , errorNotifier{pathCache}
@@ -242,7 +242,7 @@ public:
ProjectStorageUpdater updater; ProjectStorageUpdater updater;
}; };
std::unique_ptr<ProjectStorageData> createProjectStorageData(::ProjectExplorer::Project *project, std::unique_ptr<ProjectStorageData> createProjectStorageData(const ::ProjectExplorer::Project *project,
PathCacheType &pathCache) PathCacheType &pathCache)
{ {
if constexpr (useProjectStorage()) { if constexpr (useProjectStorage()) {
@@ -272,7 +272,7 @@ class QmlDesignerProjectManager::QmlDesignerProjectManagerProjectData
{ {
public: public:
QmlDesignerProjectManagerProjectData(ImageCacheStorage<Sqlite::Database> &storage, QmlDesignerProjectManagerProjectData(ImageCacheStorage<Sqlite::Database> &storage,
::ProjectExplorer::Project *project, const ::ProjectExplorer::Project *project,
PathCacheType &pathCache, PathCacheType &pathCache,
ExternalDependenciesInterface &externalDependencies) ExternalDependenciesInterface &externalDependencies)
: collector{QSize{300, 300}, : collector{QSize{300, 300},
@@ -281,6 +281,7 @@ public:
ImageCacheCollectorNullImageHandling::CaptureNullImage} ImageCacheCollectorNullImageHandling::CaptureNullImage}
, factory{storage, timeStampProvider, collector} , factory{storage, timeStampProvider, collector}
, projectStorageData{createProjectStorageData(project, pathCache)} , projectStorageData{createProjectStorageData(project, pathCache)}
, activeTarget{project->activeTarget()}
{} {}
ImageCacheCollector collector; ImageCacheCollector collector;
@@ -461,11 +462,10 @@ QString qtCreatorItemLibraryPath()
} // namespace } // namespace
void QmlDesignerProjectManager::projectAdded(::ProjectExplorer::Project *project) void QmlDesignerProjectManager::projectAdded(const ::ProjectExplorer::Project *project)
{ {
m_projectData = std::make_unique<QmlDesignerProjectManagerProjectData>( m_projectData = std::make_unique<QmlDesignerProjectManagerProjectData>(
m_previewImageCacheData->storage, project, m_data->pathCache, m_externalDependencies); m_previewImageCacheData->storage, project, m_data->pathCache, m_externalDependencies);
m_projectData->activeTarget = project->activeTarget();
QObject::connect(project, &::ProjectExplorer::Project::fileListChanged, [&]() { QObject::connect(project, &::ProjectExplorer::Project::fileListChanged, [&]() {
fileListChanged(); fileListChanged();
@@ -483,7 +483,7 @@ void QmlDesignerProjectManager::projectAdded(::ProjectExplorer::Project *project
activeTargetChanged(target); activeTargetChanged(target);
} }
void QmlDesignerProjectManager::aboutToRemoveProject(::ProjectExplorer::Project *) void QmlDesignerProjectManager::aboutToRemoveProject(const ::ProjectExplorer::Project *)
{ {
if (m_projectData) { if (m_projectData) {
m_previewImageCacheData->collector.setTarget(m_projectData->activeTarget); m_previewImageCacheData->collector.setTarget(m_projectData->activeTarget);
@@ -491,7 +491,7 @@ void QmlDesignerProjectManager::aboutToRemoveProject(::ProjectExplorer::Project
} }
} }
void QmlDesignerProjectManager::projectRemoved(::ProjectExplorer::Project *) {} void QmlDesignerProjectManager::projectRemoved(const ::ProjectExplorer::Project *) {}
void QmlDesignerProjectManager::generatePreview() void QmlDesignerProjectManager::generatePreview()
{ {

View File

@@ -49,9 +49,9 @@ private:
void editorOpened(::Core::IEditor *editor); void editorOpened(::Core::IEditor *editor);
void currentEditorChanged(::Core::IEditor *); void currentEditorChanged(::Core::IEditor *);
void editorsClosed(const QList<Core::IEditor *> &editor); void editorsClosed(const QList<Core::IEditor *> &editor);
void projectAdded(::ProjectExplorer::Project *project); void projectAdded(const ::ProjectExplorer::Project *project);
void aboutToRemoveProject(::ProjectExplorer::Project *project); void aboutToRemoveProject(const ::ProjectExplorer::Project *project);
void projectRemoved(::ProjectExplorer::Project *project); void projectRemoved(const ::ProjectExplorer::Project *project);
void generatePreview(); void generatePreview();
ImageCacheData *imageCacheData(); ImageCacheData *imageCacheData();