CFX_ArabicChar contains only static methods, no need to instantiate.

core/fxcrt/include/fx_arabic.h

Index: core/fxcrt/include/fx_arabic.h
diff --git a/core/fxcrt/include/fx_arabic.h b/core/fxcrt/include/fx_arabic.h
index 96fa7954c3de1f15b2c1d248d5729a0f650949aa..120c4012869e89081f0fb1788ad3f34b446b876d 100644
--- a/core/fxcrt/include/fx_arabic.h
+++ b/core/fxcrt/include/fx_arabic.h
@@ -16,22 +16,18 @@
 
 class CFX_ArabicChar {
  public:
-  void Release() { delete this; }
+  static bool IsArabicChar(FX_WCHAR wch);
+  static bool IsArabicFormChar(FX_WCHAR wch);
 
-  FX_BOOL IsArabicChar(FX_WCHAR wch) const;
-  FX_BOOL IsArabicFormChar(FX_WCHAR wch) const;
+  static FX_WCHAR GetFormChar(FX_WCHAR wch,
+                              FX_WCHAR prev = 0,
+                              FX_WCHAR next = 0);
+  static FX_WCHAR GetFormChar(const CFX_Char* cur,
+                              const CFX_Char* prev,
+                              const CFX_Char* next);
 
-  FX_WCHAR GetFormChar(FX_WCHAR wch,
-                       FX_WCHAR prev = 0,
-                       FX_WCHAR next = 0) const;
-  FX_WCHAR GetFormChar(const CFX_Char* cur,
-                       const CFX_Char* prev,
-                       const CFX_Char* next) const;
-
- protected:
-  const FX_ARBFORMTABLE* ParseChar(const CFX_Char* pTC,
-                                   FX_WCHAR& wChar,
-                                   FX_CHARTYPE& eType) const;
+ private:
+  CFX_ArabicChar() = delete;  // Just a namespace !!!

[Wei Li, 2016/05/06 18:58:28]

Looks like namespace is preferred in this case
https://google.github.io/styleguide/cppguide.html#Nonmember,_Static_Member,_a...

 };
 
 void FX_BidiReverseString(CFX_WideString& wsText,