diff options
| author | tsepez <tsepez@chromium.org> | 2017-01-24 10:50:20 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2017-01-24 10:50:20 -0800 |
| commit | 2763fba8755e3ef4bc15eb5c347ea2f291c6736a (patch) | |
| tree | 086bf247a3342b11694be65d058bda70e6cf4ae7 /core/fpdfapi/page/cpdf_streamcontentparser.h | |
| parent | a60d5b901e90436fefc1012c7f38e83c9b35f3cf (diff) | |
| download | pdfium-2763fba8755e3ef4bc15eb5c347ea2f291c6736a.tar.xz | |
Replace some loose (ptr, len) pairs with CFX_ByteStringC in fpdfapi.
These separate scalars are an anti-pattern given the ability to pass a single entity and later operate on it sensibly.
Review-Url: https://codereview.chromium.org/2652033002
Diffstat (limited to 'core/fpdfapi/page/cpdf_streamcontentparser.h')
| -rw-r--r-- | core/fpdfapi/page/cpdf_streamcontentparser.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/core/fpdfapi/page/cpdf_streamcontentparser.h b/core/fpdfapi/page/cpdf_streamcontentparser.h index 6429b02bbd..e23ae8ea99 100644 --- a/core/fpdfapi/page/cpdf_streamcontentparser.h +++ b/core/fpdfapi/page/cpdf_streamcontentparser.h @@ -77,16 +77,16 @@ class CPDF_StreamContentParser { std::unordered_map<uint32_t, void (CPDF_StreamContentParser::*)()>; static OpCodes InitializeOpCodes(); - void AddNumberParam(const FX_CHAR* str, int len); + void AddNameParam(const CFX_ByteStringC& str); + void AddNumberParam(const CFX_ByteStringC& str); void AddObjectParam(std::unique_ptr<CPDF_Object> pObj); - void AddNameParam(const FX_CHAR* name, int size); int GetNextParamPos(); void ClearAllParams(); CPDF_Object* GetObject(uint32_t index); CFX_ByteString GetString(uint32_t index); FX_FLOAT GetNumber(uint32_t index); int GetInteger(uint32_t index) { return (int32_t)(GetNumber(index)); } - void OnOperator(const FX_CHAR* op); + void OnOperator(const CFX_ByteStringC& op); void AddTextObject(CFX_ByteString* pText, FX_FLOAT fInitKerning, FX_FLOAT* pKerning, |