[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - sdext/source

[Stephan Bergmann]

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

New commits:
commit 4ca9473d824fe37c2a63d48917cbaee79f5811e1
Author: Stephan Bergmann <sberg...@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
    (cherry picked from commit b51ba9e383243e8248d6da1b71a5b06d841d165b)
    Reviewed-on: https://gerrit.libreoffice.org/16643
    Reviewed-by: Miklos Vajna <vmik...@collabora.co.uk>
    Tested-by: Miklos Vajna <vmik...@collabora.co.uk>

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
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits