diff options
author | Ivan Komissarov <abbapoh@gmail.com> | 2020-05-14 18:49:05 +0200 |
---|---|---|
committer | Ivan Komissarov <ABBAPOH@gmail.com> | 2020-05-25 08:59:04 +0000 |
commit | ff61ba1702644bd558ba5d65821b885d39a21e1d (patch) | |
tree | 3938f243e9126bf1163cce06264ca46b02132ebf /src/app/qbs/commandlinefrontend.cpp | |
parent | 6bba36e5e76820409bd3afad5da6883a79f79904 (diff) | |
download | qbs-ff61ba1702644bd558ba5d65821b885d39a21e1d.tar.gz |
ProjectData: return const values/references
Qbs does not care that much about binary compatilibity (because of qbs
session) and thus is is preferable to return members by const-ref
(saving 2 atomic operations and preventing from accidental detaches). In
case when method needs to compute some value instead of returning member
directly, the function signature can be changed.
Change-Id: I4e8d228761a990d274e77652c94adf9f43a758f1
Reviewed-by: Christian Kandeler <christian.kandeler@qt.io>
Diffstat (limited to 'src/app/qbs/commandlinefrontend.cpp')
-rw-r--r-- | src/app/qbs/commandlinefrontend.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/src/app/qbs/commandlinefrontend.cpp b/src/app/qbs/commandlinefrontend.cpp index d8b4d9ca8..4a28b93ec 100644 --- a/src/app/qbs/commandlinefrontend.cpp +++ b/src/app/qbs/commandlinefrontend.cpp @@ -351,8 +351,7 @@ CommandLineFrontend::ProductMap CommandLineFrontend::productsToUse() const for (const Project &project : qAsConst(m_projects)) { QList<ProductData> &productList = products[project]; const ProjectData projectData = project.projectData(); - const auto products = projectData.allProducts(); - for (const ProductData &product : products) { + for (const ProductData &product : projectData.allProducts()) { productNames << product.name(); if (useAll || m_parser.products().contains(product.name())) { productList.push_back(product); @@ -647,8 +646,7 @@ ProductData CommandLineFrontend::getTheOneRunnableProduct() QBS_CHECK(m_projects.size() == 1); // Has been checked earlier. if (m_parser.products().size() == 1) { - const auto products = m_projects.front().projectData().allProducts(); - for (const ProductData &p : products) { + for (const ProductData &p : m_projects.front().projectData().allProducts()) { if (p.name() == m_parser.products().constFirst()) return p; } @@ -657,8 +655,7 @@ ProductData CommandLineFrontend::getTheOneRunnableProduct() QBS_CHECK(m_parser.products().isEmpty()); QList<ProductData> runnableProducts; - const auto products = m_projects.front().projectData().allProducts(); - for (const ProductData &p : products) { + for (const ProductData &p : m_projects.front().projectData().allProducts()) { if (p.isRunnable()) runnableProducts.push_back(p); } |