diff options
| author | Tom Sepez <tsepez@chromium.org> | 2014-08-13 17:12:28 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2014-08-13 17:12:28 -0700 |
| commit | 368ed462dd6e5e63977ca2b2e56a7e83b3a3bdda (patch) | |
| tree | d6e9f37c824646a245d2753fe93a8404da52dbff /core/src/fpdfapi/fpdf_font | |
| parent | 2fd7b0be4473f4c9877fed2c2ade403c71ded395 (diff) | |
| download | pdfium-368ed462dd6e5e63977ca2b2e56a7e83b3a3bdda.tar.xz | |
Add FX_OVERRIDE and use it for virtual functions of FX_FINAL classes.
Should there be cases where this fails to compile, it indicates a mistake,
either an incorrectly declared overrriden virtual method, or a method that
should be declared non-virtual.
The only issues were with CPDF_CustomAccess::GetBlock(), CPDF_CustomAccess::GetByte(),
and CPDF_CustomAccess::GetFullPath(). These don't appear to be used anywhere,
and are removed. Two members are removed that are no longer needed once those
methods are removed.
R=jam@chromium.org, jun_fang@foxitsoftware.com
Review URL: https://codereview.chromium.org/454983003
Diffstat (limited to 'core/src/fpdfapi/fpdf_font')
| -rw-r--r-- | core/src/fpdfapi/fpdf_font/ttgsubtable.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/core/src/fpdfapi/fpdf_font/ttgsubtable.h b/core/src/fpdfapi/fpdf_font/ttgsubtable.h index cc7f8c8752..515b9f9242 100644 --- a/core/src/fpdfapi/fpdf_font/ttgsubtable.h +++ b/core/src/fpdfapi/fpdf_font/ttgsubtable.h @@ -409,11 +409,11 @@ private: class CFX_GSUBTable FX_FINAL : public IFX_GSUBTable, public CFX_Object { public: - virtual void Release() + virtual void Release() FX_OVERRIDE { delete this; } - virtual FX_BOOL GetVerticalGlyph(FX_DWORD glyphnum, FX_DWORD* vglyphnum); + virtual FX_BOOL GetVerticalGlyph(FX_DWORD glyphnum, FX_DWORD* vglyphnum) FX_OVERRIDE; CFX_CTTGSUBTable m_GsubImp; }; #endif |