summaryrefslogtreecommitdiff
path: root/core/fxge/cttfontdesc.cpp
diff options
context:
space:
mode:
Diffstat (limited to 'core/fxge/cttfontdesc.cpp')
-rw-r--r--core/fxge/cttfontdesc.cpp43
1 files changed, 12 insertions, 31 deletions
diff --git a/core/fxge/cttfontdesc.cpp b/core/fxge/cttfontdesc.cpp
index 97a12732e7..3d0b27b159 100644
--- a/core/fxge/cttfontdesc.cpp
+++ b/core/fxge/cttfontdesc.cpp
@@ -6,35 +6,26 @@
#include "core/fxge/cttfontdesc.h"
+#include <utility>
+
#include "core/fxge/fx_freetype.h"
#include "third_party/base/stl_util.h"
-CTTFontDesc::CTTFontDesc(uint8_t* pData, FXFT_Face face)
- : m_bIsTTC(false), m_pFontData(pData), m_SingleFace(face) {}
-
-CTTFontDesc::CTTFontDesc(uint8_t* pData, size_t index, FXFT_Face face)
- : m_bIsTTC(true), m_pFontData(pData) {
+CTTFontDesc::CTTFontDesc(std::unique_ptr<uint8_t, FxFreeDeleter> pData)
+ : m_pFontData(std::move(pData)) {
for (size_t i = 0; i < FX_ArraySize(m_TTCFaces); i++)
m_TTCFaces[i] = nullptr;
- SetTTCFace(index, face);
}
CTTFontDesc::~CTTFontDesc() {
ASSERT(m_RefCount == 0);
- if (m_bIsTTC) {
- for (size_t i = 0; i < FX_ArraySize(m_TTCFaces); i++) {
- if (m_TTCFaces[i])
- FXFT_Done_Face(m_TTCFaces[i]);
- }
- } else {
- if (m_SingleFace)
- FXFT_Done_Face(m_SingleFace);
+ for (size_t i = 0; i < FX_ArraySize(m_TTCFaces); i++) {
+ if (m_TTCFaces[i])
+ FXFT_Done_Face(m_TTCFaces[i]);
}
- FX_Free(m_pFontData);
}
-void CTTFontDesc::SetTTCFace(size_t index, FXFT_Face face) {
- ASSERT(m_bIsTTC);
+void CTTFontDesc::SetFace(size_t index, FXFT_Face face) {
ASSERT(index < FX_ArraySize(m_TTCFaces));
m_TTCFaces[index] = face;
}
@@ -45,24 +36,14 @@ void CTTFontDesc::AddRef() {
}
CTTFontDesc::ReleaseStatus CTTFontDesc::ReleaseFace(FXFT_Face face) {
- if (m_bIsTTC) {
- if (!pdfium::ContainsValue(m_TTCFaces, face))
- return kNotAppropriate;
- } else {
- if (m_SingleFace != face)
- return kNotAppropriate;
- }
+ if (!pdfium::ContainsValue(m_TTCFaces, face))
+ return kNotAppropriate;
+
ASSERT(m_RefCount > 0);
return --m_RefCount == 0 ? kReleased : kNotReleased;
}
-FXFT_Face CTTFontDesc::SingleFace() const {
- ASSERT(!m_bIsTTC);
- return m_SingleFace;
-}
-
-FXFT_Face CTTFontDesc::TTCFace(size_t index) const {
- ASSERT(m_bIsTTC);
+FXFT_Face CTTFontDesc::GetFace(size_t index) const {
ASSERT(index < FX_ArraySize(m_TTCFaces));
return m_TTCFaces[index];
}