From f866c80222da84dda3917e56b6d1b14beeeedb6c Mon Sep 17 00:00:00 2001 From: Ben Wiederhake Date: Sat, 13 May 2023 21:23:18 +0200 Subject: [PATCH] LibPDF: Avoid unnecessary HashMap copy, mark other copies --- Userland/Libraries/LibPDF/Encoding.cpp | 4 ++-- Userland/Libraries/LibPDF/Parser.cpp | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Userland/Libraries/LibPDF/Encoding.cpp b/Userland/Libraries/LibPDF/Encoding.cpp index de4a826d7d..549c71d4c9 100644 --- a/Userland/Libraries/LibPDF/Encoding.cpp +++ b/Userland/Libraries/LibPDF/Encoding.cpp @@ -43,8 +43,8 @@ PDFErrorOr> Encoding::from_object(Document* document, No auto encoding = adopt_ref(*new Encoding()); - encoding->m_descriptors = base_encoding->m_descriptors; - encoding->m_name_mapping = base_encoding->m_name_mapping; + encoding->m_descriptors = TRY(base_encoding->m_descriptors.clone()); + encoding->m_name_mapping = TRY(base_encoding->m_name_mapping.clone()); auto differences_array = TRY(dict->get_array(document, CommonNames::Differences)); diff --git a/Userland/Libraries/LibPDF/Parser.cpp b/Userland/Libraries/LibPDF/Parser.cpp index 4f90d06471..3f878bd8c5 100644 --- a/Userland/Libraries/LibPDF/Parser.cpp +++ b/Userland/Libraries/LibPDF/Parser.cpp @@ -432,7 +432,7 @@ PDFErrorOr> Parser::parse_dict() return error("Expected dict to end with \">>\""); m_reader.consume_whitespace(); - return make_object(map); + return make_object(move(map)); } PDFErrorOr> Parser::parse_stream(NonnullRefPtr dict)