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

Stephan Bergmann sbergman at redhat.com
Wed Jul 1 00:21:50 PDT 2015


 sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx |   12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

New commits:
commit b51ba9e383243e8248d6da1b71a5b06d841d165b
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Wed Jul 1 09:20:18 2015 +0200

    Work around const-ness issues in the GooString API
    
    cf. <https://gerrit.libreoffice.org/#/c/16357> "Revert the removal of cast to
    build on old poppler"
    
    Change-Id: I75184cec64b4f32e0cf9adedeb84c8591be4fc73

diff --git a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
index f4fa810..b5dd4e4 100644
--- a/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
+++ b/sdext/source/pdfimport/xpdfwrapper/pdfioutdev_gpl.hxx
@@ -103,13 +103,13 @@ namespace pdfi
             isUnderline(rSrc.isUnderline),
             size(rSrc.size)
         {
-            familyName.append(const_cast<GooString*>(&rSrc.familyName));
+            familyName.append(&rSrc.getFamilyName());
         }
 
         FontAttributes& operator=( const FontAttributes& rSrc )
         {
             familyName.clear();
-            familyName.append(const_cast<GooString*>(&rSrc.familyName));
+            familyName.append(&rSrc.getFamilyName());
 
             isEmbedded  = rSrc.isEmbedded;
             isBold      = rSrc.isBold;
@@ -122,8 +122,7 @@ namespace pdfi
 
         bool operator==(const FontAttributes& rFont) const
         {
-            return familyName.cmp(
-                const_cast<GooString*>(&rFont.familyName))==0 &&
+            return getFamilyName().cmp(&rFont.getFamilyName())==0 &&
                 isEmbedded == rFont.isEmbedded &&
                 isBold == rFont.isBold &&
                 isItalic == rFont.isItalic &&
@@ -137,6 +136,11 @@ namespace pdfi
         bool        isItalic;
         bool        isUnderline;
         double      size;
+
+    private:
+        // Work around const-ness issues in the GooString API:
+        GooString & getFamilyName() const
+        { return const_cast<GooString &>(familyName); }
     };
 
     class PDFOutDev : public OutputDev


More information about the Libreoffice-commits mailing list