diff options
author | Albert Astals Cid <aacid@kde.org> | 2019-10-05 18:48:28 +0200 |
---|---|---|
committer | Miklos Vajna <vmiklos@collabora.com> | 2019-10-09 09:24:54 +0200 |
commit | 4b28c1a10f06e0a610d0a740b8a5839dcec9dae4 (patch) | |
tree | 4fcaa07df574101c8565ca4393190f789e21f5bd | |
parent | 012bce64e08d45c9ce37c9c6758e8556cebf8688 (diff) |
CDROutputElementList::draw: Micro perfomance improvement
make the auto variable of the for loop const &, otherwise there's a copy
involved. Copying a shared_ptr is not very expensive but not copying it
is cheaper :)
Change-Id: I861e8b58441b45e2e7aa045e2aa7c0521c73ebbe
Reviewed-on: https://gerrit.libreoffice.org/80303
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Miklos Vajna <vmiklos@collabora.com>
-rw-r--r-- | src/lib/CDROutputElementList.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/CDROutputElementList.cpp b/src/lib/CDROutputElementList.cpp index 001f9dc..c970bd6 100644 --- a/src/lib/CDROutputElementList.cpp +++ b/src/lib/CDROutputElementList.cpp @@ -351,7 +351,7 @@ CDROutputElementList::~CDROutputElementList() void CDROutputElementList::draw(librevenge::RVNGDrawingInterface *painter) const { - for (auto element : m_elements) + for (const auto &element : m_elements) element->draw(painter); } |