Chromium Code Reviews| Index: core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp |
| diff --git a/core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp b/core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp |
| index a3281999feba156fa79427993097ae63e9d50785..dc79de687d20c778f176027f0a15d8fd70999076 100644 |
| --- a/core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp |
| +++ b/core/src/fpdfapi/fpdf_edit/fpdf_edit_doc.cpp |
| @@ -1105,15 +1105,15 @@ CPDF_Dictionary* CPDF_Document::CreateNewPage(int iPage) { |
| } |
| return pDict; |
| } |
| -int _PDF_GetStandardFontName(CFX_ByteString& name); |
| + |
| CPDF_Font* CPDF_Document::AddStandardFont(const FX_CHAR* font, |
| CPDF_FontEncoding* pEncoding) { |
| CFX_ByteString name(font, -1); |
|
Tom Sepez
2015/10/16 21:32:33
nit: the -1 isn't needed, the one-arg form hits st
Lei Zhang
2015/10/16 21:34:52
There's more of them. I'll fix them all in another
|
| - if (_PDF_GetStandardFontName(name) < 0) { |
| - return NULL; |
| - } |
| + if (PDF_GetStandardFontName(&name) < 0) |
| + return nullptr; |
| return GetPageData()->GetStandardFont(name, pEncoding); |
| } |
| + |
| void CPDF_Document::DeletePage(int iPage) { |
| CPDF_Dictionary* pRoot = GetRoot(); |
| if (pRoot == NULL) { |