| Index: xfa/fgas/font/fgas_gefont.cpp
|
| diff --git a/xfa/fgas/font/fgas_gefont.cpp b/xfa/fgas/font/fgas_gefont.cpp
|
| index 7863ef715be04e480080680718c634d0d52adb2d..a7d8c49cbfdd36a51c1e4e3d626c3ef911f5ea11 100644
|
| --- a/xfa/fgas/font/fgas_gefont.cpp
|
| +++ b/xfa/fgas/font/fgas_gefont.cpp
|
| @@ -16,15 +16,15 @@ CFGAS_GEFont* CFGAS_GEFont::LoadFont(const FX_WCHAR* pszFontFamily,
|
| uint16_t wCodePage,
|
| IFGAS_FontMgr* pFontMgr) {
|
| #if _FXM_PLATFORM_ != _FXM_PLATFORM_WINDOWS_
|
| - if (NULL != pFontMgr) {
|
| + if (pFontMgr) {
|
| return pFontMgr->GetFontByCodePage(wCodePage, dwFontStyles, pszFontFamily);
|
| }
|
| - return NULL;
|
| + return nullptr;
|
| #else
|
| CFGAS_GEFont* pFont = new CFGAS_GEFont(pFontMgr);
|
| if (!pFont->LoadFontInternal(pszFontFamily, dwFontStyles, wCodePage)) {
|
| pFont->Release();
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| return pFont;
|
| #endif
|
| @@ -36,7 +36,7 @@ CFGAS_GEFont* CFGAS_GEFont::LoadFont(CFX_Font* pExtFont,
|
| CFGAS_GEFont* pFont = new CFGAS_GEFont(pFontMgr);
|
| if (!pFont->LoadFontInternal(pExtFont)) {
|
| pFont->Release();
|
| - return NULL;
|
| + return nullptr;
|
| }
|
| return pFont;
|
| }
|
| @@ -73,17 +73,17 @@ CFGAS_GEFont::CFGAS_GEFont(IFGAS_FontMgr* pFontMgr)
|
| m_bUseLogFontStyle(FALSE),
|
| m_dwLogFontStyle(0),
|
| #endif
|
| - m_pFont(NULL),
|
| + m_pFont(nullptr),
|
| m_pFontMgr(pFontMgr),
|
| m_iRefCount(1),
|
| m_bExtFont(FALSE),
|
| - m_pStream(NULL),
|
| - m_pFileRead(NULL),
|
| - m_pFontEncoding(NULL),
|
| - m_pCharWidthMap(NULL),
|
| - m_pRectArray(NULL),
|
| - m_pBBoxMap(NULL),
|
| - m_pProvider(NULL) {
|
| + m_pStream(nullptr),
|
| + m_pFileRead(nullptr),
|
| + m_pFontEncoding(nullptr),
|
| + m_pCharWidthMap(nullptr),
|
| + m_pRectArray(nullptr),
|
| + m_pBBoxMap(nullptr),
|
| + m_pProvider(nullptr) {
|
| }
|
|
|
| CFGAS_GEFont::CFGAS_GEFont(const CFGAS_GEFont& src, uint32_t dwFontStyles)
|
| @@ -92,17 +92,17 @@ CFGAS_GEFont::CFGAS_GEFont(const CFGAS_GEFont& src, uint32_t dwFontStyles)
|
| m_bUseLogFontStyle(FALSE),
|
| m_dwLogFontStyle(0),
|
| #endif
|
| - m_pFont(NULL),
|
| + m_pFont(nullptr),
|
| m_pFontMgr(src.m_pFontMgr),
|
| m_iRefCount(1),
|
| m_bExtFont(FALSE),
|
| - m_pStream(NULL),
|
| - m_pFileRead(NULL),
|
| - m_pFontEncoding(NULL),
|
| - m_pCharWidthMap(NULL),
|
| - m_pRectArray(NULL),
|
| - m_pBBoxMap(NULL),
|
| - m_pProvider(NULL) {
|
| + m_pStream(nullptr),
|
| + m_pFileRead(nullptr),
|
| + m_pFontEncoding(nullptr),
|
| + m_pCharWidthMap(nullptr),
|
| + m_pRectArray(nullptr),
|
| + m_pBBoxMap(nullptr),
|
| + m_pProvider(nullptr) {
|
| ASSERT(src.m_pFont);
|
| m_pFont = new CFX_Font;
|
| m_pFont->LoadClone(src.m_pFont);
|
| @@ -141,7 +141,7 @@ CFGAS_GEFont::~CFGAS_GEFont() {
|
|
|
| void CFGAS_GEFont::Release() {
|
| if (--m_iRefCount < 1) {
|
| - if (m_pFontMgr != NULL) {
|
| + if (m_pFontMgr) {
|
| m_pFontMgr->RemoveFont(this);
|
| }
|
| delete this;
|
| @@ -160,7 +160,7 @@ FX_BOOL CFGAS_GEFont::LoadFontInternal(const FX_WCHAR* pszFontFamily,
|
| return FALSE;
|
| }
|
| CFX_ByteString csFontFamily;
|
| - if (pszFontFamily != NULL) {
|
| + if (pszFontFamily) {
|
| csFontFamily = CFX_ByteString::FromUnicode(pszFontFamily);
|
| }
|
| uint32_t dwFlags = 0;
|
| @@ -276,7 +276,7 @@ void CFGAS_GEFont::GetFamilyName(CFX_WideString& wsFamily) const {
|
| }
|
|
|
| uint32_t CFGAS_GEFont::GetFontStyles() const {
|
| - ASSERT(m_pFont != NULL);
|
| + ASSERT(m_pFont);
|
| #if _FXM_PLATFORM_ != _FXM_PLATFORM_WINDOWS_
|
| if (m_bUseLogFontStyle) {
|
| return m_dwLogFontStyle;
|
| @@ -309,14 +309,14 @@ FX_BOOL CFGAS_GEFont::GetCharWidthInternal(FX_WCHAR wUnicode,
|
| int32_t& iWidth,
|
| FX_BOOL bRecursive,
|
| FX_BOOL bCharCode) {
|
| - ASSERT(m_pCharWidthMap != NULL);
|
| + ASSERT(m_pCharWidthMap);
|
| iWidth = m_pCharWidthMap->GetAt(wUnicode, 0);
|
| if (iWidth < 1) {
|
| if (!m_pProvider ||
|
| !m_pProvider->GetCharWidth(this, wUnicode, iWidth, bCharCode)) {
|
| - CFGAS_GEFont* pFont = NULL;
|
| + CFGAS_GEFont* pFont = nullptr;
|
| int32_t iGlyph = GetGlyphIndex(wUnicode, TRUE, &pFont, bCharCode);
|
| - if (iGlyph != 0xFFFF && pFont != NULL) {
|
| + if (iGlyph != 0xFFFF && pFont) {
|
| if (pFont == this) {
|
| iWidth = m_pFont->GetGlyphWidth(iGlyph);
|
| if (iWidth < 0) {
|
| @@ -345,13 +345,13 @@ FX_BOOL CFGAS_GEFont::GetCharBBoxInternal(FX_WCHAR wUnicode,
|
| CFX_Rect& bbox,
|
| FX_BOOL bRecursive,
|
| FX_BOOL bCharCode) {
|
| - ASSERT(m_pRectArray != NULL);
|
| - ASSERT(m_pBBoxMap != NULL);
|
| - void* pRect = NULL;
|
| + ASSERT(m_pRectArray);
|
| + ASSERT(m_pBBoxMap);
|
| + void* pRect = nullptr;
|
| if (!m_pBBoxMap->Lookup((void*)(uintptr_t)wUnicode, pRect)) {
|
| - CFGAS_GEFont* pFont = NULL;
|
| + CFGAS_GEFont* pFont = nullptr;
|
| int32_t iGlyph = GetGlyphIndex(wUnicode, TRUE, &pFont, bCharCode);
|
| - if (iGlyph != 0xFFFF && pFont != NULL) {
|
| + if (iGlyph != 0xFFFF && pFont) {
|
| if (pFont == this) {
|
| FX_RECT rtBBox;
|
| if (m_pFont->GetGlyphBBox(iGlyph, rtBBox)) {
|
| @@ -390,22 +390,22 @@ int32_t CFGAS_GEFont::GetItalicAngle() const {
|
| return m_pFont->GetSubstFont()->m_ItalicAngle;
|
| }
|
| int32_t CFGAS_GEFont::GetGlyphIndex(FX_WCHAR wUnicode, FX_BOOL bCharCode) {
|
| - return GetGlyphIndex(wUnicode, TRUE, NULL, bCharCode);
|
| + return GetGlyphIndex(wUnicode, TRUE, nullptr, bCharCode);
|
| }
|
| int32_t CFGAS_GEFont::GetGlyphIndex(FX_WCHAR wUnicode,
|
| FX_BOOL bRecursive,
|
| CFGAS_GEFont** ppFont,
|
| FX_BOOL bCharCode) {
|
| - ASSERT(m_pFontEncoding != NULL);
|
| + ASSERT(m_pFontEncoding);
|
| int32_t iGlyphIndex = m_pFontEncoding->GlyphFromCharCode(wUnicode);
|
| if (iGlyphIndex > 0) {
|
| - if (ppFont != NULL) {
|
| + if (ppFont) {
|
| *ppFont = this;
|
| }
|
| return iGlyphIndex;
|
| }
|
| const FGAS_FONTUSB* pFontUSB = FGAS_GetUnicodeBitField(wUnicode);
|
| - if (pFontUSB == NULL) {
|
| + if (!pFontUSB) {
|
| return 0xFFFF;
|
| }
|
| uint16_t wBitField = pFontUSB->wBitField;
|
| @@ -415,7 +415,7 @@ int32_t CFGAS_GEFont::GetGlyphIndex(FX_WCHAR wUnicode,
|
| auto it = m_FontMapper.find(wUnicode);
|
| CFGAS_GEFont* pFont = it != m_FontMapper.end() ? it->second : nullptr;
|
| if (pFont && pFont != this) {
|
| - iGlyphIndex = pFont->GetGlyphIndex(wUnicode, FALSE, NULL, bCharCode);
|
| + iGlyphIndex = pFont->GetGlyphIndex(wUnicode, FALSE, nullptr, bCharCode);
|
| if (iGlyphIndex != 0xFFFF) {
|
| int32_t i = m_SubstFonts.Find(pFont);
|
| if (i > -1) {
|
| @@ -436,7 +436,7 @@ int32_t CFGAS_GEFont::GetGlyphIndex(FX_WCHAR wUnicode,
|
| CFGAS_GEFont* pFont = m_pFontMgr->GetFontByUnicode(
|
| wUnicode, GetFontStyles(), wsFamily.c_str());
|
| if (!pFont)
|
| - pFont = m_pFontMgr->GetFontByUnicode(wUnicode, GetFontStyles(), NULL);
|
| + pFont = m_pFontMgr->GetFontByUnicode(wUnicode, GetFontStyles(), nullptr);
|
| #endif
|
| if (pFont) {
|
| if (pFont == this) {
|
| @@ -446,7 +446,7 @@ int32_t CFGAS_GEFont::GetGlyphIndex(FX_WCHAR wUnicode,
|
| m_FontMapper[wUnicode] = pFont;
|
| int32_t i = m_SubstFonts.GetSize();
|
| m_SubstFonts.Add(pFont);
|
| - iGlyphIndex = pFont->GetGlyphIndex(wUnicode, FALSE, NULL, bCharCode);
|
| + iGlyphIndex = pFont->GetGlyphIndex(wUnicode, FALSE, nullptr, bCharCode);
|
| if (iGlyphIndex != 0xFFFF) {
|
| iGlyphIndex |= ((i + 1) << 24);
|
| if (ppFont)
|
| @@ -466,13 +466,13 @@ int32_t CFGAS_GEFont::GetDescent() const {
|
| void CFGAS_GEFont::Reset() {
|
| for (int32_t i = 0; i < m_SubstFonts.GetSize(); i++)
|
| m_SubstFonts[i]->Reset();
|
| - if (m_pCharWidthMap != NULL) {
|
| + if (m_pCharWidthMap) {
|
| m_pCharWidthMap->RemoveAll();
|
| }
|
| - if (m_pBBoxMap != NULL) {
|
| + if (m_pBBoxMap) {
|
| m_pBBoxMap->RemoveAll();
|
| }
|
| - if (m_pRectArray != NULL) {
|
| + if (m_pRectArray) {
|
| m_pRectArray->RemoveAll();
|
| }
|
| }
|
|
|