diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-06-18 12:47:11 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-06-18 12:47:11 -0700 |
| commit | 2c28619de8051661c7f66f2192c6fb9ef14ee905 (patch) | |
| tree | 4bd15bf0bb72e4127403548bf2f40230bbdd352b /core/src/fxcrt/fx_arabic.h | |
| parent | 0c1bd7c6ad378ea97a2f54cb0c4761716f6d1057 (diff) | |
| download | pdfium-2c28619de8051661c7f66f2192c6fb9ef14ee905.tar.xz | |
Replace some Release() calls with virtual destructors.
Required fixing xfa-specific code.
Original Review URL: https://codereview.chromium.org/1192013002.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1176413004.
Diffstat (limited to 'core/src/fxcrt/fx_arabic.h')
| -rw-r--r-- | core/src/fxcrt/fx_arabic.h | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/core/src/fxcrt/fx_arabic.h b/core/src/fxcrt/fx_arabic.h index 862aabbd73..16938325ec 100644 --- a/core/src/fxcrt/fx_arabic.h +++ b/core/src/fxcrt/fx_arabic.h @@ -184,21 +184,15 @@ class CFX_BidiChar final : public IFX_BidiChar { public: CFX_BidiChar(); - virtual void Release() override - { - delete this; - } - virtual void SetPolicy(FX_BOOL bSeparateNeutral = TRUE) override - { - m_bSeparateNeutral = bSeparateNeutral; - } - virtual FX_BOOL AppendChar(FX_WCHAR wch) override; - virtual FX_BOOL EndChar() override; - virtual int32_t GetBidiInfo(int32_t &iStart, int32_t &iCount) override; - virtual void Reset() override; + ~CFX_BidiChar() override {} + + void SetPolicy(FX_BOOL bSeparateNeutral = TRUE) override; + FX_BOOL AppendChar(FX_WCHAR wch) override; + FX_BOOL EndChar() override; + int32_t GetBidiInfo(int32_t &iStart, int32_t &iCount) override; + void Reset() override; private: - ~CFX_BidiChar() { } FX_BOOL m_bSeparateNeutral; int32_t m_iCurStart; int32_t m_iCurCount; |