diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2015-10-28 14:18:26 -0400 |
|---|---|---|
| committer | Dan Sinclair <dsinclair@chromium.org> | 2015-10-28 14:18:26 -0400 |
| commit | 5a4af03849c8389872cd6908222df4c4e7f99ca8 (patch) | |
| tree | c98a4f7e02fcf477e2eede92877adb4a7689e734 | |
| parent | dd4cd523ef4546b43d68e8ca6ec0a97fbe8954e9 (diff) | |
| download | pdfium-5a4af03849c8389872cd6908222df4c4e7f99ca8.tar.xz | |
Add constants for the path operators.
This CL adds constants for some of the path operators to make it more obvious
what is being checked.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1413783009 .
| -rw-r--r-- | core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp | 38 |
1 files changed, 27 insertions, 11 deletions
diff --git a/core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp b/core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp index 1e435ee958..0e059f3fc5 100644 --- a/core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp +++ b/core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp @@ -10,6 +10,18 @@ #include "pageint.h" #include <limits.h> +namespace { + +const char kPathOperatorSubpath = 'm'; +const char kPathOperatorLine = 'l'; +const char kPathOperatorCubicBezier1 = 'c'; +const char kPathOperatorCubicBezier2 = 'v'; +const char kPathOperatorCubicBezier3 = 'y'; +const char kPathOperatorClosePath = 'h'; +const char kPathOperatorRectangle[] = "re"; + +} // namespace + class CPDF_StreamParserAutoClearer { public: CPDF_StreamParserAutoClearer(CPDF_StreamParser** scoped_variable, @@ -150,33 +162,33 @@ void CPDF_StreamContentParser::ParsePathObject() { int len = m_pSyntax->GetWordSize(); if (len == 1) { switch (m_pSyntax->GetWordBuf()[0]) { - case 'm': + case kPathOperatorSubpath: AddPathPoint(params[0], params[1], FXPT_MOVETO); nParams = 0; break; - case 'l': + case kPathOperatorLine: AddPathPoint(params[0], params[1], FXPT_LINETO); nParams = 0; break; - case 'c': + case kPathOperatorCubicBezier1: AddPathPoint(params[0], params[1], FXPT_BEZIERTO); AddPathPoint(params[2], params[3], FXPT_BEZIERTO); AddPathPoint(params[4], params[5], FXPT_BEZIERTO); nParams = 0; break; - case 'v': + case kPathOperatorCubicBezier2: AddPathPoint(m_PathCurrentX, m_PathCurrentY, FXPT_BEZIERTO); AddPathPoint(params[0], params[1], FXPT_BEZIERTO); AddPathPoint(params[2], params[3], FXPT_BEZIERTO); nParams = 0; break; - case 'y': + case kPathOperatorCubicBezier3: AddPathPoint(params[0], params[1], FXPT_BEZIERTO); AddPathPoint(params[2], params[3], FXPT_BEZIERTO); AddPathPoint(params[2], params[3], FXPT_BEZIERTO); nParams = 0; break; - case 'h': + case kPathOperatorClosePath: Handle_ClosePath(); nParams = 0; break; @@ -185,8 +197,8 @@ void CPDF_StreamContentParser::ParsePathObject() { break; } } else if (len == 2) { - if (m_pSyntax->GetWordBuf()[0] == 'r' && - m_pSyntax->GetWordBuf()[1] == 'e') { + if (m_pSyntax->GetWordBuf()[0] == kPathOperatorRectangle[0] && + m_pSyntax->GetWordBuf()[1] == kPathOperatorRectangle[1]) { AddPathRect(params[0], params[1], params[2], params[3]); nParams = 0; } else { @@ -530,6 +542,7 @@ CPDF_StreamParser::SyntaxType CPDF_StreamParser::ParseNextElement() { } return Keyword; } + void CPDF_StreamParser::SkipPathObject() { FX_DWORD command_startpos = m_Pos; if (!PositionIsInBounds()) @@ -573,13 +586,16 @@ void CPDF_StreamParser::SkipPathObject() { if (m_Pos - op_startpos == 2) { int op = m_pBuf[op_startpos]; - // TODO(dsinclair): Can these be turned into named constants? - if (op == 'm' || op == 'l' || op == 'c' || op == 'v' || op == 'y') { + if (op == kPathOperatorSubpath || op == kPathOperatorLine || + op == kPathOperatorCubicBezier1 || + op == kPathOperatorCubicBezier2 || + op == kPathOperatorCubicBezier3) { command_startpos = m_Pos; break; } } else if (m_Pos - op_startpos == 3) { - if (m_pBuf[op_startpos] == 'r' && m_pBuf[op_startpos + 1] == 'e') { + if (m_pBuf[op_startpos] == kPathOperatorRectangle[0] && + m_pBuf[op_startpos + 1] == kPathOperatorRectangle[1]) { command_startpos = m_Pos; break; } |