[Libreoffice-commits] core.git: vcl/source

[Noel Grandin (via logerrit)]

 vcl/source/gdi/impglyphitem.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit fe1681b902caed0aaf9157ec31062d7627a1e1b9
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Tue Oct 25 09:46:49 2022 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Tue Oct 25 11:56:21 2022 +0200

    no need to call GetGlyphs() again
    
    removes a copy from a hot path
    
    Change-Id: I28c7b49643448232b6a6a4ab409dd3e36c751aeb
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/141795
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/vcl/source/gdi/impglyphitem.cxx b/vcl/source/gdi/impglyphitem.cxx
index a8eed4a02020..016e7c9b9b39 100644
--- a/vcl/source/gdi/impglyphitem.cxx
+++ b/vcl/source/gdi/impglyphitem.cxx
@@ -458,7 +458,7 @@ SalLayoutGlyphsCache::GetLayoutGlyphs(VclPtr<const 
OutputDevice> outputDevice, c
                 mLastTemporaryKey.reset();
                 return &mLastTemporaryGlyphs;
             }
-            mCachedGlyphs.insert(std::make_pair(key, layout->GetGlyphs()));
+            mCachedGlyphs.insert(std::make_pair(key, std::move(glyphs)));
             assert(mCachedGlyphs.find(key)
                    == mCachedGlyphs.begin()); // newly inserted item is first
             return &mCachedGlyphs.begin()->second;