Cleanup IFX_BidiChar

core/src/fxcrt/fx_arabic.h

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #ifndef CORE_SRC_FXCRT_FX_ARABIC_H_
 #define CORE_SRC_FXCRT_FX_ARABIC_H_
 
 #include "../../include/fxcrt/fx_arb.h"
 
 class CFX_BidiChar final : public IFX_BidiChar
 {
 public:
     CFX_BidiChar();
-    ~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;
+    ~CFX_BidiChar() override;
 
 private:
-    FX_BOOL     m_bSeparateNeutral;
-    int32_t     m_iCurStart;
-    int32_t     m_iCurCount;
-    int32_t     m_iCurBidi;
-    int32_t     m_iLastBidi;
-    int32_t     m_iLastStart;
-    int32_t     m_iLastCount;
+    // IFX_BidiChar

[Tom Sepez, 2015/06/19 20:30:32]

nit: maybe colon after IFX_BidiChar:

+    bool AppendChar(FX_WCHAR wch) override;
+    bool EndChar() override;
+    Direction GetBidiInfo(int32_t* iStart, int32_t* iCount) override;
+
+    void SaveCurrentStateToLastState();
+
+    // Position of the current segment.
+    int32_t m_iCurStart;
+
+    // Number of characters in the current segment.
+    int32_t m_iCurCount;
+
+    // Direction of the current segment.
+    Direction m_CurBidi;
+
+    // Number of characters in the last segment.
+    int32_t m_iLastStart;
+
+    // Number of characters in the last segment.
+    int32_t m_iLastCount;
+
+    // Direction of the last segment.
+    Direction m_LastBidi;
 };
 
 #endif  // CORE_SRC_FXCRT_FX_ARABIC_H_