vcl/source/font/PhysicalFontFace.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 37e95b4d6754d2c77a76a84ce51a0b18df93927f
Author: Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Thu Nov 10 18:42:32 2022 +0100
Commit: Stephan Bergmann <sberg...@redhat.com>
CommitDate: Fri Nov 11 14:10:49 2022 +0100
Drop a counterproductive top-level const
...that was apparently left by accident by
caa10a1f80da856debf4f397503f27c86199208f "vcl: Use a hb_face_t wrapper in
TrueTypeFace", thwarting the use of std::move here.
(I came across this code with an upcoming loplugin:constmove that flags
suspicious uses of std::move involving const-qualified types.)
Change-Id: I460b11738b0f480a91c1212632cb428cb1331cd6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142560
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sberg...@redhat.com>
diff --git a/vcl/source/font/PhysicalFontFace.cxx
b/vcl/source/font/PhysicalFontFace.cxx
index 05f9a4519b61..975ace9ad264 100644
--- a/vcl/source/font/PhysicalFontFace.cxx
+++ b/vcl/source/font/PhysicalFontFace.cxx
@@ -344,7 +344,7 @@ class TrueTypeFace final : public AbstractTrueTypeFont
}
public:
- TrueTypeFace(const RawFace aFace, const FontCharMapRef rCharMap)
+ TrueTypeFace(RawFace aFace, const FontCharMapRef rCharMap)
: AbstractTrueTypeFont(nullptr, rCharMap)
, m_aFace(std::move(aFace))
{
