diff options
| author | Tom Sepez <tsepez@chromium.org> | 2016-02-08 13:39:53 -0800 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2016-02-08 13:39:53 -0800 |
| commit | 7a73effb9fe071fbc852b30865d7a332e96fbd56 (patch) | |
| tree | e3a61318b805e6b7108e6889235a2e44e382b3d6 /core/src/fpdfapi | |
| parent | 1d95d654dde9380265be8d412141e03f7ba12da3 (diff) | |
| download | pdfium-7a73effb9fe071fbc852b30865d7a332e96fbd56.tar.xz | |
Tidy fpdfsave.cpp
Remove CFX_PtrArray filelist.
Promote ScopedFileStream to .h file and use it.
Fix _CAPS names, bool returns, and put in anonymous namespace.
FX_CreateMemoryStream() can't return null, so remove checks.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1672153002 .
Diffstat (limited to 'core/src/fpdfapi')
| -rw-r--r-- | core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp | 5 | ||||
| -rw-r--r-- | core/src/fpdfapi/fpdf_parser/fpdf_parser_parser_unittest.cpp | 5 |
2 files changed, 0 insertions, 10 deletions
diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp index 6489b1ed98..f170b20d98 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp @@ -81,11 +81,6 @@ bool CanReadFromBitStream(const CFX_BitStream* hStream, } // namespace -// TODO(thestig) Using unique_ptr with ReleaseDeleter is still not ideal. -// Come up or wait for something better. -using ScopedFileStream = - std::unique_ptr<IFX_FileStream, ReleaseDeleter<IFX_FileStream>>; - bool IsSignatureDict(const CPDF_Dictionary* pDict) { CPDF_Object* pType = pDict->GetElementValue("Type"); if (!pType) diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser_unittest.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser_unittest.cpp index 77780b9dcf..e71b19011e 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser_unittest.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser_unittest.cpp @@ -68,11 +68,6 @@ class CPDF_TestParser : public CPDF_Parser { FRIEND_TEST(fpdf_parser_parser, LoadCrossRefV4); }; -// TODO(thestig) Using unique_ptr with ReleaseDeleter is still not ideal. -// Come up or wait for something better. -using ScopedFileStream = - std::unique_ptr<IFX_FileStream, ReleaseDeleter<IFX_FileStream>>; - TEST(fpdf_parser_parser, ReadHexString) { { // Empty string. |