core/fxge/include/fx_font.h - Issue 2083943003: Use FXFONT defines in place of integers. - Code Review

Chromium Code Reviews

chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

(1081)

My Issues | Starred Open | Closed | All

Unified Diff: core/fxge/include/fx_font.h

Issue 2083943003: Use FXFONT defines in place of integers. (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master

Patch Set: build Created 4 years, 6 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

« no previous file with comments | « core/fxge/ge/fx_ge_text.cpp ('k') | fpdfsdk/cfx_systemhandler.h » ('j') | fpdfsdk/fpdf_sysfontinfo.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: core/fxge/include/fx_font.h

diff --git a/core/fxge/include/fx_font.h b/core/fxge/include/fx_font.h

index 78913c6867945d2d5911cadca9e9af0ead98f3a2..f99d4a8af9a97359736da898f5c60513c0f55801 100644

--- a/core/fxge/include/fx_font.h

+++ b/core/fxge/include/fx_font.h

@@ -218,23 +218,16 @@ class CFX_SubstFont {

CFX_SubstFont();

void* m_ExtHandle;

-

CFX_ByteString m_Family;

-

int m_Charset;

-

uint32_t m_SubstFlags;

-

int m_Weight;

-

int m_ItalicAngle;

-

- FX_BOOL m_bSubstOfCJK;

-

+ bool m_bSubstCJK;

int m_WeightCJK;

-

- FX_BOOL m_bItlicCJK;

+ bool m_bItalicCJK;

};

+

#define FX_FONT_FLAG_SERIF 0x01

#define FX_FONT_FLAG_FIXEDPITCH 0x02

#define FX_FONT_FLAG_ITALIC 0x04

« no previous file with comments | « core/fxge/ge/fx_ge_text.cpp ('k') | fpdfsdk/cfx_systemhandler.h » ('j') | fpdfsdk/fpdf_sysfontinfo.cpp » ('J')

Powered by Google App Engine

This is Rietveld 408576698