[PATCH] Change public variables of class ImplFontAttributes to priva...

navin patidar (via Code Review) gerrit at gerrit.libreoffice.org
Sat Jan 5 11:38:28 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1551

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/51/1551/1

Change public variables of class ImplFontAttributes to private.

Change-Id: I56ff5d7d2420c334c6fc9e4c552f8b5c4715d93f
---
M vcl/generic/fontmanager/fontconfig.cxx
M vcl/generic/fontmanager/fontsubst.cxx
M vcl/generic/glyphs/gcach_ftyp.cxx
M vcl/generic/glyphs/glyphcache.cxx
M vcl/generic/print/genpspgraphics.cxx
M vcl/inc/outfont.hxx
M vcl/source/gdi/font.cxx
M vcl/source/gdi/outdev3.cxx
M vcl/source/gdi/pdfwriter_impl.cxx
9 files changed, 163 insertions(+), 148 deletions(-)



diff --git a/vcl/generic/fontmanager/fontconfig.cxx b/vcl/generic/fontmanager/fontconfig.cxx
index b707b78..0efdbe4 100644
--- a/vcl/generic/fontmanager/fontconfig.cxx
+++ b/vcl/generic/fontmanager/fontconfig.cxx
@@ -946,8 +946,8 @@
        FcCharSetDestroy(unicodes);
     }
 
-    addtopattern(pPattern, rPattern.meItalic, rPattern.meWeight,
-        rPattern.meWidthType, rPattern.mePitch);
+    addtopattern(pPattern, rPattern.GetSlant(), rPattern.GetWeight(),
+        rPattern.GetWidthType(), rPattern.GetPitch());
 
     // query fontconfig for a substitute
     FcConfigSubstitute(FcConfigGetCurrent(), pPattern, FcMatchPattern);
@@ -1017,13 +1017,13 @@
             {
                 int val = 0;
                 if (FcResultMatch == FcPatternGetInteger(pSet->fonts[0], FC_WEIGHT, 0, &val))
-                    rPattern.meWeight = convertWeight(val);
+                    rPattern.SetWeight( convertWeight(val) );
                 if (FcResultMatch == FcPatternGetInteger(pSet->fonts[0], FC_SLANT, 0, &val))
-                    rPattern.meItalic = convertSlant(val);
+                    rPattern.SetItalic( convertSlant(val) );
                 if (FcResultMatch == FcPatternGetInteger(pSet->fonts[0], FC_SPACING, 0, &val))
-                    rPattern.mePitch = convertSpacing(val);
+                    rPattern.SetPitch ( convertSpacing(val) );
                 if (FcResultMatch == FcPatternGetInteger(pSet->fonts[0], FC_WIDTH, 0, &val))
-                    rPattern.meWidthType = convertWidth(val);
+                    rPattern.SetWidthType ( convertWidth(val) );
                 FcBool bEmbolden;
                 if (FcResultMatch == FcPatternGetBool(pSet->fonts[0], FC_EMBOLDEN, 0, &bEmbolden))
                     rPattern.mbEmbolden = bEmbolden;
diff --git a/vcl/generic/fontmanager/fontsubst.cxx b/vcl/generic/fontmanager/fontsubst.cxx
index 23ac968..a026ee9 100644
--- a/vcl/generic/fontmanager/fontsubst.cxx
+++ b/vcl/generic/fontmanager/fontsubst.cxx
@@ -132,10 +132,10 @@
         return
           (
             rOrig.maTargetName == rNew.maSearchName &&
-            rOrig.meWeight == rNew.meWeight &&
-            rOrig.meItalic == rNew.meItalic &&
-            rOrig.mePitch == rNew.mePitch &&
-            rOrig.meWidthType == rNew.meWidthType
+            rOrig.GetWeight() == rNew.GetWeight() &&
+            rOrig.GetSlant() == rNew.GetSlant() &&
+            rOrig.GetPitch() == rNew.GetPitch() &&
+            rOrig.GetWidthType() == rNew.GetWidthType()
           );
     }
 
diff --git a/vcl/generic/glyphs/gcach_ftyp.cxx b/vcl/generic/glyphs/gcach_ftyp.cxx
index d7e693b..de44e31 100644
--- a/vcl/generic/glyphs/gcach_ftyp.cxx
+++ b/vcl/generic/glyphs/gcach_ftyp.cxx
@@ -788,8 +788,8 @@
     mnLoadFlags |= FT_LOAD_IGNORE_TRANSFORM;
 #endif
 
-    mbArtItalic = (rFSD.meItalic != ITALIC_NONE && pFI->GetFontAttributes().GetSlant() == ITALIC_NONE);
-    mbArtBold = (rFSD.meWeight > WEIGHT_MEDIUM && pFI->GetFontAttributes().GetWeight() <= WEIGHT_MEDIUM);
+    mbArtItalic = (rFSD.GetSlant() != ITALIC_NONE && pFI->GetFontAttributes().GetSlant() == ITALIC_NONE);
+    mbArtBold = (rFSD.GetWeight() > WEIGHT_MEDIUM && pFI->GetFontAttributes().GetWeight() <= WEIGHT_MEDIUM);
     mbUseGamma = false;
     if( mbArtBold )
     {
@@ -934,7 +934,7 @@
          (rTo.GetFamilyName().EqualsAscii("StarSymbol"))
        )
     {
-        rTo.mbSymbolFlag = true;
+        rTo.SetSymbolFlag( true );
     }
 
     if( maSizeFT )
@@ -964,11 +964,11 @@
         // map the panose info from the OS2 table to their VCL counterparts
         switch( pOS2->panose[0] )
         {
-            case 1: rTo.meFamily = FAMILY_ROMAN; break;
-            case 2: rTo.meFamily = FAMILY_SWISS; break;
-            case 3: rTo.meFamily = FAMILY_MODERN; break;
-            case 4: rTo.meFamily = FAMILY_SCRIPT; break;
-            case 5: rTo.meFamily = FAMILY_DECORATIVE; break;
+            case 1: rTo.SetFamilyType( FAMILY_ROMAN ); break;
+            case 2: rTo.SetFamilyType( FAMILY_SWISS ); break;
+            case 3: rTo.SetFamilyType( FAMILY_MODERN ); break;
+            case 4: rTo.SetFamilyType( FAMILY_SCRIPT ); break;
+            case 5: rTo.SetFamilyType( FAMILY_DECORATIVE ); break;
             // TODO: is it reasonable to override the attribute with DONTKNOW?
             case 0: // fall through
             default: rTo.meFamilyType = FAMILY_DONTKNOW; break;
@@ -982,12 +982,12 @@
             case 5: // fall through
             case 6: // fall through
             case 7: // fall through
-            case 8: rTo.mePitch = PITCH_VARIABLE; break;
-            case 9: rTo.mePitch = PITCH_FIXED; break;
+            case 8: rTo.SetPitch( PITCH_VARIABLE ); break;
+            case 9: rTo.SetPitch( PITCH_FIXED ); break;
             // TODO: is it reasonable to override the attribute with DONTKNOW?
             case 0: // fall through
             case 1: // fall through
-            default: rTo.mePitch = PITCH_DONTKNOW; break;
+            default: rTo.SetPitch( PITCH_DONTKNOW ); break;
         }
 
         const double fScale = (double)GetFontSelData().mnHeight / maFaceFT->units_per_EM;
diff --git a/vcl/generic/glyphs/glyphcache.cxx b/vcl/generic/glyphs/glyphcache.cxx
index deef6be..79a2d7a 100644
--- a/vcl/generic/glyphs/glyphcache.cxx
+++ b/vcl/generic/glyphs/glyphcache.cxx
@@ -97,8 +97,8 @@
     nHash   += rFontSelData.mnHeight;
     nHash   += rFontSelData.mnOrientation;
     nHash   += rFontSelData.mbVertical;
-    nHash   += rFontSelData.meItalic;
-    nHash   += rFontSelData.meWeight;
+    nHash   += rFontSelData.GetSlant();
+    nHash   += rFontSelData.GetWeight();
 #ifdef ENABLE_GRAPHITE
     nHash   += rFontSelData.meLanguage;
 #endif
@@ -122,8 +122,8 @@
     ||  (rA.mbNonAntialiased != rB.mbNonAntialiased) )
         return false;
 
-    if( (rA.meItalic != rB.meItalic)
-    ||  (rA.meWeight != rB.meWeight) )
+    if( (rA.GetSlant() != rB.GetSlant())
+    ||  (rA.GetWeight() != rB.GetWeight()) )
         return false;
 
     // NOTE: ignoring meFamily deliberately
diff --git a/vcl/generic/print/genpspgraphics.cxx b/vcl/generic/print/genpspgraphics.cxx
index 1a271a0..7b2ce32 100644
--- a/vcl/generic/print/genpspgraphics.cxx
+++ b/vcl/generic/print/genpspgraphics.cxx
@@ -840,13 +840,13 @@
     // determine which font attributes need to be emulated
     bool bArtItalic = false;
     bool bArtBold = false;
-    if( pEntry->meItalic == ITALIC_OBLIQUE || pEntry->meItalic == ITALIC_NORMAL )
+    if( pEntry->GetSlant() == ITALIC_OBLIQUE || pEntry->GetSlant() == ITALIC_NORMAL )
     {
         FontItalic eItalic = m_pPrinterGfx->GetFontMgr().getFontItalic( nID );
         if( eItalic != ITALIC_NORMAL && eItalic != ITALIC_OBLIQUE )
             bArtItalic = true;
     }
-    int nWeight = (int)pEntry->meWeight;
+    int nWeight = (int)pEntry->GetWeight();
     int nRealWeight = (int)m_pPrinterGfx->GetFontMgr().getFontWeight( nID );
     if( nRealWeight <= (int)WEIGHT_MEDIUM && nWeight > (int)WEIGHT_MEDIUM )
     {
@@ -1124,14 +1124,14 @@
 ImplDevFontAttributes GenPspGraphics::Info2DevFontAttributes( const psp::FastPrintFontInfo& rInfo )
 {
     ImplDevFontAttributes aDFA;
-    aDFA.maName         = rInfo.m_aFamilyName;
-    aDFA.maStyleName    = rInfo.m_aStyleName;
-    aDFA.meFamily       = rInfo.m_eFamilyStyle;
-    aDFA.meWeight       = rInfo.m_eWeight;
-    aDFA.meItalic       = rInfo.m_eItalic;
-    aDFA.meWidthType    = rInfo.m_eWidth;
-    aDFA.mePitch        = rInfo.m_ePitch;
-    aDFA.mbSymbolFlag   = (rInfo.m_aEncoding == RTL_TEXTENCODING_SYMBOL);
+    aDFA.SetFamilyName( rInfo.m_aFamilyName );
+    aDFA.SetStyleName( rInfo.m_aStyleName );
+    aDFA.SetFamilyType( rInfo.m_eFamilyStyle );
+    aDFA.SetWeight( rInfo.m_eWeight );
+    aDFA.SetItalic( rInfo.m_eItalic );
+    aDFA.SetWidthType( rInfo.m_eWidth );
+    aDFA.SetPitch( rInfo.m_ePitch );
+    aDFA.SetSymbolFlag( (rInfo.m_aEncoding == RTL_TEXTENCODING_SYMBOL) );
     aDFA.mbSubsettable  = rInfo.m_bSubsettable;
     aDFA.mbEmbeddable   = rInfo.m_bEmbeddable;
 
diff --git a/vcl/inc/outfont.hxx b/vcl/inc/outfont.hxx
index 0fe3cb6..741d6be 100644
--- a/vcl/inc/outfont.hxx
+++ b/vcl/inc/outfont.hxx
@@ -61,14 +61,21 @@
     FontPitch       GetPitch() const        { return mePitch; }
     FontWidth       GetWidthType() const    { return meWidthType; }
     bool            IsSymbolFont() const    { return mbSymbolFlag; }
-
+    void            SetFamilyName(const String sFamilyName)    { maName = sFamilyName; }
+    void            SetStyleName( const String sStyleName)           { maStyleName = sStyleName; }
+    void            SetFamilyType(const FontFamily eFontFamily)    { meFamily = eFontFamily; }
+    void            SetPitch(const FontPitch ePitch )          { mePitch = ePitch; }
+    void            SetItalic(const FontItalic eItalic )       { meItalic = eItalic; }
+    void            SetWeight(const FontWeight eWeight )       { meWeight = eWeight; }
+    void            SetWidthType(const FontWidth eWidthType)   { meWidthType = eWidthType; }
+    void            SetSymbolFlag(const bool bSymbolFlag )     { mbSymbolFlag = bSymbolFlag; }
     bool operator==(const ImplFontAttributes& rOther) const;
     bool operator!=(const ImplFontAttributes& rOther) const
     {
         return !(*this == rOther);
     }
 
-public: // TODO: hide members behind accessor methods
+private:
     String          maName;         // Font Family Name
     String          maStyleName;    // Font Style Name
     FontWeight      meWeight;       // Weight Type
diff --git a/vcl/source/gdi/font.cxx b/vcl/source/gdi/font.cxx
index 11897d0..d3110a7 100644
--- a/vcl/source/gdi/font.cxx
+++ b/vcl/source/gdi/font.cxx
@@ -751,14 +751,22 @@
 void Font::GetFontAttributes( ImplFontAttributes& rAttrs ) const
 {
     // #i56788# Use members directly, don't risc config access.
-    rAttrs.maName = mpImplFont->maFamilyName;
-    rAttrs.maStyleName = mpImplFont->maStyleName;
-    rAttrs.meFamily = mpImplFont->meFamily;
-    rAttrs.mePitch = mpImplFont->mePitch;
-    rAttrs.meItalic = mpImplFont->meItalic;
-    rAttrs.meWeight = mpImplFont->meWeight;
-    rAttrs.meWidthType = WIDTH_DONTKNOW;
-    rAttrs.mbSymbolFlag= (mpImplFont->meCharSet == RTL_TEXTENCODING_SYMBOL);
+//    rAttrs.maName = mpImplFont->maFamilyName;
+    rAttrs.SetFamilyName( mpImplFont->maFamilyName );
+//    rAttrs.maStyleName = mpImplFont->maStyleName;
+    rAttrs.SetStyleName( mpImplFont->maStyleName );
+//    rAttrs.meFamily = mpImplFont->meFamily;
+    rAttrs.SetFamilyType( mpImplFont->meFamily );
+//    rAttrs.mePitch = mpImplFont->mePitch;
+    rAttrs.SetPitch( mpImplFont->mePitch );
+//    rAttrs.meItalic = mpImplFont->meItalic;
+    rAttrs.SetItalic( mpImplFont->meItalic );
+//    rAttrs.meWeight = mpImplFont->meWeight;
+    rAttrs.SetWeight( mpImplFont->meWeight );
+//    rAttrs.meWidthType = WIDTH_DONTKNOW;
+    rAttrs.SetWidthType( WIDTH_DONTKNOW );
+//    rAttrs.mbSymbolFlag= (mpImplFont->meCharSet == RTL_TEXTENCODING_SYMBOL);
+    rAttrs.SetSymbolFlag( mpImplFont->meCharSet == RTL_TEXTENCODING_SYMBOL );
 }
 
 
diff --git a/vcl/source/gdi/outdev3.cxx b/vcl/source/gdi/outdev3.cxx
index db9cb8d..6d3b02e 100644
--- a/vcl/source/gdi/outdev3.cxx
+++ b/vcl/source/gdi/outdev3.cxx
@@ -614,7 +614,7 @@
                     if (pEntry)
                     {
                         if( pEntry->maFontSelData.mpFontData )
-                            aFont.SetName( pEntry->maFontSelData.mpFontData->maName );
+                            aFont.SetName( pEntry->maFontSelData.mpFontData->GetFamilyName() );
                         else
                             aFont.SetName( pEntry->maFontSelData.maTargetName );
                     }
@@ -745,10 +745,10 @@
     mpNext( NULL )
 {
     // StarSymbol is a unicode font, but it still deserves the symbol flag
-    if( !mbSymbolFlag )
+    if( !IsSymbolFont() )
         if( 0 == GetFamilyName().CompareIgnoreCaseToAscii( "starsymbol", 10)
         ||  0 == GetFamilyName().CompareIgnoreCaseToAscii( "opensymbol", 10) )
-            mbSymbolFlag = true;
+            SetSymbolFlag( true );
 }
 
 // -----------------------------------------------------------------------
@@ -756,22 +756,22 @@
 StringCompare PhysicalFontFace::CompareIgnoreSize( const PhysicalFontFace& rOther ) const
 {
     // compare their width, weight, italic and style name
-    if( meWidthType < rOther.meWidthType )
+    if( GetWidthType() < rOther.GetWidthType() )
         return COMPARE_LESS;
-    else if( meWidthType > rOther.meWidthType )
+    else if( GetWidthType() > rOther.GetWidthType() )
         return COMPARE_GREATER;
 
-    if( meWeight < rOther.meWeight )
+    if( GetWeight() < rOther.GetWeight() )
         return COMPARE_LESS;
-    else if( meWeight > rOther.meWeight )
+    else if( GetWeight() > rOther.GetWeight() )
         return COMPARE_GREATER;
 
-    if( meItalic < rOther.meItalic )
+    if( GetSlant() < rOther.GetSlant() )
         return COMPARE_LESS;
-    else if( meItalic > rOther.meItalic )
+    else if( GetSlant() > rOther.GetSlant() )
         return COMPARE_GREATER;
 
-    StringCompare eCompare = maName.CompareTo( rOther.maName );
+    StringCompare eCompare = GetFamilyName().CompareTo( rOther.GetFamilyName() );
     return eCompare;
 }
 
@@ -812,34 +812,34 @@
     int nMatch = 0;
 
     const String& rFontName = rFSD.maTargetName;
-    if( (rFontName == maName) || rFontName.EqualsIgnoreCaseAscii( maName ) )
+    if( (rFontName == GetFamilyName()) || rFontName.EqualsIgnoreCaseAscii( GetFamilyName() ) )
         nMatch += 240000;
 
     if( rStatus.mpTargetStyleName
-    &&  maStyleName.EqualsIgnoreCaseAscii( rStatus.mpTargetStyleName ) )
+    &&  GetStyleName().EqualsIgnoreCaseAscii( rStatus.mpTargetStyleName ) )
         nMatch += 120000;
 
-    if( (rFSD.mePitch != PITCH_DONTKNOW) && (rFSD.mePitch == mePitch) )
+    if( (rFSD.GetPitch() != PITCH_DONTKNOW) && (rFSD.GetPitch() == GetPitch()) )
         nMatch += 20000;
 
     // prefer NORMAL font width
     // TODO: change when the upper layers can tell their width preference
-    if( meWidthType == WIDTH_NORMAL )
+    if( GetWidthType() == WIDTH_NORMAL )
         nMatch += 400;
-    else if( (meWidthType == WIDTH_SEMI_EXPANDED) || (meWidthType == WIDTH_SEMI_CONDENSED) )
+    else if( (GetWidthType() == WIDTH_SEMI_EXPANDED) || (GetWidthType() == WIDTH_SEMI_CONDENSED) )
         nMatch += 300;
 
-    if( rFSD.meWeight != WEIGHT_DONTKNOW )
+    if( rFSD.GetWeight() != WEIGHT_DONTKNOW )
     {
         // if not bold or requiring emboldening prefer light fonts to bold fonts
-        FontWeight ePatternWeight = rFSD.mbEmbolden ? WEIGHT_NORMAL : rFSD.meWeight;
+        FontWeight ePatternWeight = rFSD.mbEmbolden ? WEIGHT_NORMAL : rFSD.GetWeight();
 
         int nReqWeight = (int)ePatternWeight;
         if ( ePatternWeight > WEIGHT_MEDIUM )
             nReqWeight += 100;
 
-        int nGivenWeight = (int)meWeight;
-        if( meWeight > WEIGHT_MEDIUM )
+        int nGivenWeight = (int)GetWeight();
+        if( GetWeight() > WEIGHT_MEDIUM )
             nGivenWeight += 100;
 
         int nWeightDiff = nReqWeight - nGivenWeight;
@@ -855,29 +855,29 @@
     {
         // prefer NORMAL font weight
         // TODO: change when the upper layers can tell their weight preference
-        if( meWeight == WEIGHT_NORMAL )
+        if( GetWeight() == WEIGHT_NORMAL )
             nMatch += 450;
-        else if( meWeight == WEIGHT_MEDIUM )
+        else if( GetWeight() == WEIGHT_MEDIUM )
             nMatch += 350;
-        else if( (meWeight == WEIGHT_SEMILIGHT) || (meWeight == WEIGHT_SEMIBOLD) )
+        else if( (GetWeight() == WEIGHT_SEMILIGHT) || (GetWeight() == WEIGHT_SEMIBOLD) )
             nMatch += 200;
-        else if( meWeight == WEIGHT_LIGHT )
+        else if( GetWeight() == WEIGHT_LIGHT )
             nMatch += 150;
     }
 
     // if requiring custom matrix to fake italic, prefer upright font
-    FontItalic ePatternItalic = rFSD.maItalicMatrix != ItalicMatrix() ? ITALIC_NONE : rFSD.meItalic;
+    FontItalic ePatternItalic = rFSD.maItalicMatrix != ItalicMatrix() ? ITALIC_NONE : rFSD.GetSlant();
 
     if ( ePatternItalic == ITALIC_NONE )
     {
-        if( meItalic == ITALIC_NONE )
+        if( GetSlant() == ITALIC_NONE )
             nMatch += 900;
     }
     else
     {
-        if( ePatternItalic == meItalic )
+        if( ePatternItalic == GetSlant() )
             nMatch += 900;
-        else if( meItalic != ITALIC_NONE )
+        else if( GetSlant() != ITALIC_NONE )
             nMatch += 600;
     }
 
@@ -1051,18 +1051,18 @@
 
     if( !mpFirst )
     {
-        maName         = pNewData->maName;
+        maName         = pNewData->GetFamilyName();
         maMapNames     = pNewData->maMapNames;
-        meFamily       = pNewData->meFamily;
-        mePitch        = pNewData->mePitch;
+        meFamily       = pNewData->GetFamilyType();
+        mePitch        = pNewData->GetPitch();
         mnMinQuality   = pNewData->mnQuality;
     }
     else
     {
         if( meFamily == FAMILY_DONTKNOW )
-            meFamily = pNewData->meFamily;
+            meFamily = pNewData->GetFamilyType();
         if( mePitch == PITCH_DONTKNOW )
-            mePitch = pNewData->mePitch;
+            mePitch = pNewData->GetPitch();
         if( mnMinQuality > pNewData->mnQuality )
             mnMinQuality = pNewData->mnQuality;
     }
@@ -1076,20 +1076,20 @@
     else
         mnTypeFaces |= IMPL_DEVFONT_NONESYMBOL;
 
-    if( pNewData->meWeight != WEIGHT_DONTKNOW )
+    if( pNewData->GetWeight() != WEIGHT_DONTKNOW )
     {
-        if( pNewData->meWeight >= WEIGHT_SEMIBOLD )
+        if( pNewData->GetWeight() >= WEIGHT_SEMIBOLD )
             mnTypeFaces |= IMPL_DEVFONT_BOLD;
-        else if( pNewData->meWeight <= WEIGHT_SEMILIGHT )
+        else if( pNewData->GetWeight() <= WEIGHT_SEMILIGHT )
             mnTypeFaces |= IMPL_DEVFONT_LIGHT;
         else
             mnTypeFaces |= IMPL_DEVFONT_NORMAL;
     }
 
-    if( pNewData->meItalic == ITALIC_NONE )
+    if( pNewData->GetSlant() == ITALIC_NONE )
         mnTypeFaces |= IMPL_DEVFONT_NONEITALIC;
-    else if( (pNewData->meItalic == ITALIC_NORMAL)
-         ||  (pNewData->meItalic == ITALIC_OBLIQUE) )
+    else if( (pNewData->GetSlant() == ITALIC_NORMAL)
+         ||  (pNewData->GetSlant() == ITALIC_OBLIQUE) )
         mnTypeFaces |= IMPL_DEVFONT_ITALIC;
 
     if( (meMatchWeight == WEIGHT_DONTKNOW)
@@ -1106,8 +1106,8 @@
     }
 
     // reassign name (sharing saves memory)
-    if( pNewData->maName == maName )
-        pNewData->maName = maName;
+    if( pNewData->GetFamilyName() == GetFamilyName() )
+        pNewData->SetFamilyName( GetFamilyName() );
 
     // insert new physical font face into linked list
     // TODO: get rid of linear search?
@@ -1480,7 +1480,7 @@
 
 void ImplDevFontList::Add( PhysicalFontFace* pNewData )
 {
-    String aSearchName = pNewData->maName;
+    String aSearchName = pNewData->GetFamilyName();
     GetEnglishSearchFontName( aSearchName );
 
     DevFontList::const_iterator it = maDevFontList.find( aSearchName );
@@ -2129,7 +2129,7 @@
     , mbNonAntialiased( false )
     , mbEmbolden( false )
 {
-    maTargetName = maName;
+    maTargetName = GetFamilyName();
 
     rFont.GetFontAttributes( *this );
 
@@ -2170,7 +2170,7 @@
     , mbNonAntialiased( false )
     , mbEmbolden( false )
 {
-    maTargetName = maSearchName = maName;
+    maTargetName = maSearchName = GetFamilyName();
     // NOTE: no normalization for width/height/orientation
 }
 
@@ -2208,8 +2208,8 @@
     }
 #endif
     nHash += 11 * mnHeight;
-    nHash += 19 * meWeight;
-    nHash += 29 * meItalic;
+    nHash += 19 * GetWeight();
+    nHash += 29 * GetSlant();
     nHash += 37 * mnOrientation;
     nHash += 41 * meLanguage;
     if( mbVertical )
@@ -2278,14 +2278,14 @@
         return false;
 
     // check font face attributes
-    if( (rA.meWeight       != rB.meWeight)
-    ||  (rA.meItalic       != rB.meItalic)
+    if( (rA.GetWeight()       != rB.GetWeight())
+    ||  (rA.GetSlant()       != rB.GetSlant())
 //    ||  (rA.meFamily       != rB.meFamily) // TODO: remove this mostly obsolete member
-    ||  (rA.mePitch        != rB.mePitch) )
+    ||  (rA.GetPitch()     != rB.GetPitch()) )
         return false;
 
     // check style name
-    if( rA.maStyleName != rB.maStyleName)
+    if( rA.GetStyleName() != rB.GetStyleName() )
         return false;
 
     // Symbol fonts may recode from one type to another So they are only
@@ -2504,7 +2504,7 @@
     String& aSearchName = rFSD.maSearchName; // TODO: get rid of reference
     for(;;)
     {
-        rFSD.maTargetName = GetNextFontToken( rFSD.maName, nTokenPos );
+        rFSD.maTargetName = GetNextFontToken( rFSD.GetFamilyName(), nTokenPos );
         aSearchName = rFSD.maTargetName;
 
 #ifdef ENABLE_GRAPHITE
@@ -2526,7 +2526,7 @@
         ImplFontSubstitute( aSearchName, nSubstFlags, pDevSpecific );
         // #114999# special emboldening for Ricoh fonts
         // TODO: smarter check for special cases by using PreMatch infrastructure?
-        if( (rFSD.meWeight > WEIGHT_MEDIUM)
+        if( (rFSD.GetWeight() > WEIGHT_MEDIUM)
         &&  aSearchName.EqualsAscii( "hg", 0, 2) )
         {
             String aBoldName;
@@ -2548,7 +2548,7 @@
                 // the other font is available => use it
                 aSearchName = aBoldName;
                 // prevent synthetic emboldening of bold version
-                rFSD.meWeight = WEIGHT_DONTKNOW;
+                rFSD.SetWeight(WEIGHT_DONTKNOW);
             }
         }
 
@@ -2613,7 +2613,7 @@
     {
         if( bMultiToken )
         {
-            rFSD.maTargetName = GetNextFontToken( rFSD.maName, nTokenPos );
+            rFSD.maTargetName = GetNextFontToken( rFSD.GetFamilyName(), nTokenPos );
             aSearchName = rFSD.maTargetName;
             GetEnglishSearchFontName( aSearchName );
         }
@@ -2633,15 +2633,15 @@
     if ( bMultiToken )
     {
         nTokenPos = 0;
-        rFSD.maTargetName = GetNextFontToken( rFSD.maName, nTokenPos );
+        rFSD.maTargetName = GetNextFontToken( rFSD.GetFamilyName(), nTokenPos );
         aSearchName = rFSD.maTargetName;
         GetEnglishSearchFontName( aSearchName );
     }
 
     String      aSearchShortName;
     String      aSearchFamilyName;
-    FontWeight  eSearchWeight   = rFSD.meWeight;
-    FontWidth   eSearchWidth    = rFSD.meWidthType;
+    FontWeight  eSearchWeight   = rFSD.GetWeight();
+    FontWidth   eSearchWidth    = rFSD.GetWidthType();
     sal_uLong       nSearchType     = 0;
     FontSubstConfiguration::getMapName( aSearchName, aSearchShortName, aSearchFamilyName,
                                         eSearchWeight, eSearchWidth, nSearchType );
@@ -2705,7 +2705,7 @@
     // now try the other font name tokens
     while( nTokenPos != STRING_NOTFOUND )
     {
-        rFSD.maTargetName = GetNextFontToken( rFSD.maName, nTokenPos );
+        rFSD.maTargetName = GetNextFontToken( rFSD.GetFamilyName(), nTokenPos );
         if( !rFSD.maTargetName.Len() )
             continue;
 
@@ -2715,7 +2715,7 @@
         String      aTempShortName;
         String      aTempFamilyName;
         sal_uLong       nTempType   = 0;
-        FontWeight  eTempWeight = rFSD.meWeight;
+        FontWeight  eTempWeight = rFSD.GetWeight();
         FontWidth   eTempWidth  = WIDTH_DONTKNOW;
         FontSubstConfiguration::getMapName( aSearchName, aTempShortName, aTempFamilyName,
                                             eTempWeight, eTempWidth, nTempType );
@@ -2769,32 +2769,32 @@
         nSearchType |= IMPL_FONT_ATTR_CJK | IMPL_FONT_ATTR_CJK_JP;
     else
     {
-        nSearchType |= ImplIsCJKFont( rFSD.maName );
+        nSearchType |= ImplIsCJKFont( rFSD.GetFamilyName() );
         if( rFSD.IsSymbolFont() )
             nSearchType |= IMPL_FONT_ATTR_SYMBOL;
     }
 
-    ImplCalcType( nSearchType, eSearchWeight, eSearchWidth, rFSD.meFamily, pFontAttr );
+    ImplCalcType( nSearchType, eSearchWeight, eSearchWidth, rFSD.GetFamilyType(), pFontAttr );
     ImplDevFontListData* pFoundData = ImplFindByAttributes( nSearchType,
-        eSearchWeight, eSearchWidth, rFSD.meItalic, aSearchFamilyName );
+        eSearchWeight, eSearchWidth, rFSD.GetSlant(), aSearchFamilyName );
 
     if( pFoundData )
     {
         // overwrite font selection attributes using info from the typeface flags
         if( (eSearchWeight >= WEIGHT_BOLD)
-        &&  (eSearchWeight > rFSD.meWeight)
+        &&  (eSearchWeight > rFSD.GetWeight())
         &&  (pFoundData->mnTypeFaces & IMPL_DEVFONT_BOLD) )
-            rFSD.meWeight = eSearchWeight;
+            rFSD.SetWeight( eSearchWeight );
         else if( (eSearchWeight < WEIGHT_NORMAL)
-        &&  (eSearchWeight < rFSD.meWeight)
+        &&  (eSearchWeight < rFSD.GetWeight())
         &&  (eSearchWeight != WEIGHT_DONTKNOW)
         &&  (pFoundData->mnTypeFaces & IMPL_DEVFONT_LIGHT) )
-            rFSD.meWeight = eSearchWeight;
+            rFSD.SetWeight( eSearchWeight );
 
         if( (nSearchType & IMPL_FONT_ATTR_ITALIC)
-        &&  ((rFSD.meItalic == ITALIC_DONTKNOW) || (rFSD.meItalic == ITALIC_NONE))
+        &&  ((rFSD.GetSlant() == ITALIC_DONTKNOW) || (rFSD.GetSlant() == ITALIC_NONE))
         &&  (pFoundData->mnTypeFaces & IMPL_DEVFONT_ITALIC) )
-            rFSD.meItalic = ITALIC_NORMAL;
+            rFSD.SetItalic( ITALIC_NORMAL );
     }
     else
     {
@@ -2833,7 +2833,7 @@
         if( !pFallbackData  )
             return NULL;
         // override the font name
-        rFontSelData.maName = pFallbackData->GetFamilyName();
+        rFontSelData.SetFamilyName( pFallbackData->GetFamilyName() );
         // clear the cached normalized name
         rFontSelData.maSearchName = String();
     }
@@ -3399,16 +3399,16 @@
     // intialize the used font name
     if( rFontSelData.mpFontData )
     {
-        maName     = rFontSelData.mpFontData->maName;
-        maStyleName= rFontSelData.mpFontData->maStyleName;
+        SetFamilyName( rFontSelData.mpFontData->GetFamilyName() );
+        SetStyleName( rFontSelData.mpFontData->GetStyleName() );
         mbDevice   = rFontSelData.mpFontData->mbDevice;
         mbKernableFont = true;
     }
     else
     {
         xub_StrLen nTokenPos = 0;
-        maName     = GetNextFontToken( rFontSelData.maName, nTokenPos );
-        maStyleName= rFontSelData.maStyleName;
+        SetFamilyName( GetNextFontToken( rFontSelData.GetFamilyName(), nTokenPos ) );
+        SetStyleName( rFontSelData.GetStyleName() );
         mbDevice   = false;
         mbKernableFont = false;
     }
@@ -7304,14 +7304,14 @@
     if( nDevFontIndex < nCount )
     {
         const PhysicalFontFace& rData = *mpGetDevFontList->Get( nDevFontIndex );
-        aFontInfo.SetName( rData.maName );
-        aFontInfo.SetStyleName( rData.maStyleName );
-        aFontInfo.SetCharSet( rData.mbSymbolFlag ? RTL_TEXTENCODING_SYMBOL : RTL_TEXTENCODING_UNICODE );
-        aFontInfo.SetFamily( rData.meFamily );
-        aFontInfo.SetPitch( rData.mePitch );
-        aFontInfo.SetWeight( rData.meWeight );
-        aFontInfo.SetItalic( rData.meItalic );
-        aFontInfo.SetWidthType( rData.meWidthType );
+        aFontInfo.SetName( rData.GetFamilyName() );
+        aFontInfo.SetStyleName( rData.GetStyleName() );
+        aFontInfo.SetCharSet( rData.IsSymbolFont() ? RTL_TEXTENCODING_SYMBOL : RTL_TEXTENCODING_UNICODE );
+        aFontInfo.SetFamily( rData.GetFamilyType() );
+        aFontInfo.SetPitch( rData.GetPitch() );
+        aFontInfo.SetWeight( rData.GetWeight() );
+        aFontInfo.SetItalic( rData.GetSlant() );
+        aFontInfo.SetWidthType( rData.GetWidthType() );
         if( rData.IsScalable() )
             aFontInfo.mpImplMetric->mnMiscFlags |= ImplFontMetric::SCALABLE_FLAG;
         if( rData.mbDevice )
@@ -7423,14 +7423,14 @@
 
     // set aMetric with info from font
     aMetric.SetName( maFont.GetName() );
-    aMetric.SetStyleName( pMetric->maStyleName );
+    aMetric.SetStyleName( pMetric->GetStyleName() );
     aMetric.SetSize( PixelToLogic( Size( pMetric->mnWidth, pMetric->mnAscent+pMetric->mnDescent-pMetric->mnIntLeading ) ) );
-    aMetric.SetCharSet( pMetric->mbSymbolFlag ? RTL_TEXTENCODING_SYMBOL : RTL_TEXTENCODING_UNICODE );
-    aMetric.SetFamily( pMetric->meFamily );
-    aMetric.SetPitch( pMetric->mePitch );
-    aMetric.SetWeight( pMetric->meWeight );
-    aMetric.SetItalic( pMetric->meItalic );
-    aMetric.SetWidthType( pMetric->meWidthType );
+    aMetric.SetCharSet( pMetric->IsSymbolFont() ? RTL_TEXTENCODING_SYMBOL : RTL_TEXTENCODING_UNICODE );
+    aMetric.SetFamily( pMetric->GetFamilyType() );
+    aMetric.SetPitch( pMetric->GetPitch() );
+    aMetric.SetWeight( pMetric->GetWeight() );
+    aMetric.SetItalic( pMetric->GetSlant() );
+    aMetric.SetWidthType( pMetric->GetWidthType() );
     if ( pEntry->mnOwnOrientation )
         aMetric.SetOrientation( pEntry->mnOwnOrientation );
     else
diff --git a/vcl/source/gdi/pdfwriter_impl.cxx b/vcl/source/gdi/pdfwriter_impl.cxx
index b017daa..83f5753 100644
--- a/vcl/source/gdi/pdfwriter_impl.cxx
+++ b/vcl/source/gdi/pdfwriter_impl.cxx
@@ -2292,14 +2292,14 @@
 static ImplDevFontAttributes GetDevFontAttributes( const PDFWriterImpl::BuiltinFont& rBuiltin )
 {
     ImplDevFontAttributes aDFA;
-    aDFA.maName         = rtl::OUString::createFromAscii( rBuiltin.m_pName );
-    aDFA.maStyleName    = rtl::OUString::createFromAscii( rBuiltin.m_pStyleName );
-    aDFA.meFamily       = rBuiltin.m_eFamily;
-    aDFA.mbSymbolFlag   = (rBuiltin.m_eCharSet != RTL_TEXTENCODING_MS_1252 );
-    aDFA.mePitch        = rBuiltin.m_ePitch;
-    aDFA.meWeight       = rBuiltin.m_eWeight;
-    aDFA.meItalic       = rBuiltin.m_eItalic;
-    aDFA.meWidthType    = rBuiltin.m_eWidthType;
+    aDFA.SetFamilyName( rtl::OUString::createFromAscii( rBuiltin.m_pName ) );
+    aDFA.SetStyleName( rtl::OUString::createFromAscii( rBuiltin.m_pStyleName ) );
+    aDFA.SetFamilyType( rBuiltin.m_eFamily );
+    aDFA.SetSymbolFlag( rBuiltin.m_eCharSet != RTL_TEXTENCODING_MS_1252 );
+    aDFA.SetPitch( rBuiltin.m_ePitch );
+    aDFA.SetWeight( rBuiltin.m_eWeight );
+    aDFA.SetItalic( rBuiltin.m_eItalic );
+    aDFA.SetWidthType( rBuiltin.m_eWidthType );
 
     aDFA.mbOrientation  = true;
     aDFA.mbDevice       = true;
@@ -2349,11 +2349,11 @@
     const BuiltinFont* pBuiltinFont = pFD->GetBuiltinFont();
 
     pMetric->mnOrientation  = sal::static_int_cast<short>(pSelect->mnOrientation);
-    pMetric->meFamily       = pBuiltinFont->m_eFamily;
-    pMetric->mePitch        = pBuiltinFont->m_ePitch;
-    pMetric->meWeight       = pBuiltinFont->m_eWeight;
-    pMetric->meItalic       = pBuiltinFont->m_eItalic;
-    pMetric->mbSymbolFlag   = pFD->IsSymbolFont();
+    pMetric->SetFamilyType( pBuiltinFont->m_eFamily );
+    pMetric->SetPitch( pBuiltinFont->m_ePitch );
+    pMetric->SetWeight( pBuiltinFont->m_eWeight );
+    pMetric->SetItalic( pBuiltinFont->m_eItalic );
+    pMetric->SetSymbolFlag( pFD->IsSymbolFont() );
     pMetric->mnWidth        = pSelect->mnHeight;
     pMetric->mnAscent       = ( pSelect->mnHeight * +pBuiltinFont->m_nAscent + 500 ) / 1000;
     pMetric->mnDescent      = ( pSelect->mnHeight * -pBuiltinFont->m_nDescent + 500 ) / 1000;
@@ -3158,7 +3158,7 @@
     aInfo.m_nDescent = 200;
     aInfo.m_nCapHeight = 1000;
     aInfo.m_aFontBBox = Rectangle( Point( -200, -200 ), Size( 1700, 1700 ) );
-    aInfo.m_aPSName = pFont->maName;
+    aInfo.m_aPSName = pFont->GetFamilyName();
     sal_Int32 pWidths[256];
     memset( pWidths, 0, sizeof(pWidths) );
     if( pFont->IsEmbeddable() )
@@ -3231,7 +3231,7 @@
             aLine.append( "/BaseFont/" );
             appendName( aInfo.m_aPSName, aLine );
             aLine.append( "\n" );
-            if( !pFont->mbSymbolFlag )
+            if( !pFont->IsSymbolFont() )
                 aLine.append( "/Encoding/WinAnsiEncoding\n" );
             aLine.append( "/FirstChar 32 /LastChar 255\n"
                           "/Widths[" );
@@ -3728,7 +3728,7 @@
                 "<</Type/Font/Subtype/Type1/BaseFont/" );
             appendName( aInfo.m_aPSName, aLine );
             aLine.append( "\n" );
-            if( !pFont->mbSymbolFlag &&  pEncoding == 0 )
+            if( !pFont->IsSymbolFont() &&  pEncoding == 0 )
                 aLine.append( "/Encoding/WinAnsiEncoding\n" );
             if( nToUnicodeStream )
             {

-- 
To view, visit https://gerrit.libreoffice.org/1551
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I56ff5d7d2420c334c6fc9e4c552f8b5c4715d93f
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: navin patidar <patidar at kacst.edu.sa>



More information about the LibreOffice mailing list