Add CFX_FontFaceInfo constructor.

core/src/fxge/ge/text_int.h

Index: core/src/fxge/ge/text_int.h
diff --git a/core/src/fxge/ge/text_int.h b/core/src/fxge/ge/text_int.h
index a05907162cb8505e9da10d00d9ed75c3968a0783..29e76d41b1339c4182fa314c3a3b99ee548f798a 100644
--- a/core/src/fxge/ge/text_int.h
+++ b/core/src/fxge/ge/text_int.h
@@ -64,20 +64,34 @@ class CFX_UnicodeEncoding : public IFX_FontEncoding {
 #define CHARSET_FLAG_BIG5 8
 #define CHARSET_FLAG_GB 16
 #define CHARSET_FLAG_KOREAN 32
-class CFontFaceInfo {
+
+class CFX_FontFaceInfo {
  public:
-  CFX_ByteString m_FilePath;
-  CFX_ByteString m_FaceName;
+  CFX_FontFaceInfo(CFX_ByteString filePath, CFX_ByteString faceName,
+                   CFX_ByteString fontTables, FX_DWORD fontOffset,
+                   FX_DWORD fileSize)
+      : m_FilePath(filePath),
+        m_FaceName(faceName),
+        m_FontTables(fontTables),
+        m_Styles(0),
+        m_Charsets(0),
+        m_FontOffset(fontOffset),
+        m_FileSize(fileSize) {}
+
+  const CFX_ByteString m_FilePath;
+  const CFX_ByteString m_FaceName;
+  const CFX_ByteString m_FontTables;
   FX_DWORD m_Styles;
   FX_DWORD m_Charsets;
-  FX_DWORD m_FontOffset;
-  FX_DWORD m_FileSize;
-  CFX_ByteString m_FontTables;
+  const FX_DWORD m_FontOffset;

[Lei Zhang, 2015/08/06 18:29:44]

maybe put all the constants together?

[Tom Sepez, 2015/08/06 18:40:42]

Sure.  But I'm more interested in putting like-types together, since they may
pack better (the lost art of struct packing).

+  const FX_DWORD m_FileSize;
 };
+
 class CFontFileFaceInfo {
  public:
   CFontFileFaceInfo();
   ~CFontFileFaceInfo();
+
   IFX_FileStream* m_pFile;
   FXFT_Face m_Face;
   CFX_ByteString m_FaceName;