From ba4df4555bfd1a67908a9fbe19270dc0500760e7 Mon Sep 17 00:00:00 2001 From: Jarek Kobus Date: Tue, 30 Jan 2024 15:52:33 +0100 Subject: [PATCH] Axivion: Use more const references Instead of copying potentially big vectors / maps. Change-Id: Ib367d18b7afb8f00551f54e8f2b2dd5c18a018c3 Reviewed-by: Reviewed-by: Christian Stenger --- src/plugins/axivion/axivionoutputpane.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/plugins/axivion/axivionoutputpane.cpp b/src/plugins/axivion/axivionoutputpane.cpp index 87b3dc5d9ac..1efffa79bad 100644 --- a/src/plugins/axivion/axivionoutputpane.cpp +++ b/src/plugins/axivion/axivionoutputpane.cpp @@ -456,12 +456,12 @@ void IssuesWidget::addIssues(const Dto::IssueTableDto &dto) if (dto.totalRemovedCount.has_value()) m_removedFilter->setText(QString::number(dto.totalRemovedCount.value())); - const std::vector tableColumns = m_currentTableInfo->columns; - const std::vector> rows = dto.rows; - for (auto row : rows) { + const std::vector &tableColumns = m_currentTableInfo->columns; + const std::vector> &rows = dto.rows; + for (const auto &row : rows) { QStringList data; - for (auto column : tableColumns) { - auto it = row.find(column.key); + for (const auto &column : tableColumns) { + const auto it = row.find(column.key); if (it != row.end()) { QString value = anyToSimpleString(it->second); if (column.key == "id")