commit 42f0422d19c4d0a2fb09654fb6d1d92e50f6dd5c
author Khaled Hosny <khaled@libreoffice.org> Tue Jul 25 03:14:37 2023 +0300
committer خالد حسني <khaled@libreoffice.org> Tue Jul 25 04:16:11 2023 +0200
tree 672d8025054aa424923ad9f131b9b4a558548ff2
parent 529dce3d5b695637a1ccc8b4b697d87c5db0d3a2

tdf#156437: Fix missing name inside embedded Type 1 font in PDF

Regression from:

commit 7415ce2b9795a78f836bc5e5bffb4b56eee26b21
Author: Khaled Hosny <khaled@aliftype.com>
Date:   Mon Dec 5 20:52:13 2022 +0200

    tdf#113048: Fix PDF ascender and descender of some fonts

We were setting the font in PDF /Font dictionary, but not inside the
font itself.

Change-Id: I1936f615f62fd7afa8f31d66d35dbded879328c5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154880
Tested-by: Jenkins
Reviewed-by: خالد حسني <khaled@libreoffice.org>

vcl/source/fontsubset/fontsubset.cxx

diff --git a/vcl/source/fontsubset/fontsubset.cxx b/vcl/source/fontsubset/fontsubset.cxx
index 4e6e03c..1ecd358 100644
--- a/vcl/source/fontsubset/fontsubset.cxx
+++ b/vcl/source/fontsubset/fontsubset.cxx
@@ -79,6 +79,10 @@ bool FontSubsetInfo::CreateFontSubset(
    mpReqEncodedIds = pReqEncodedIds;
    mnReqGlyphCount = nReqGlyphCount;

    OString aPSName = m_aPSName.toUtf8();
    if (!mpReqFontName)
        mpReqFontName = aPSName.getStr();

    // TODO: move the glyphid/encid/notdef reshuffling from the callers to here

    // dispatch to underlying subsetters