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

Side by Side Diff: core/include/fpdfapi/fpdf_parser.h

Issue 1580573002: Replace more CFX_MapPtrToPtr and remove it. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: And remove it. Created 4 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #ifndef CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_ 7 #ifndef CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_
8 #define CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_ 8 #define CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_
9 9
10 #include <map> 10 #include <map>
(...skipping 527 matching lines...) Expand 10 before | Expand all | Expand 10 after
538 538
539 FX_BOOL m_bVersionUpdated; 539 FX_BOOL m_bVersionUpdated;
540 540
541 CPDF_Object* m_pLinearized; 541 CPDF_Object* m_pLinearized;
542 542
543 FX_DWORD m_dwFirstPageNo; 543 FX_DWORD m_dwFirstPageNo;
544 544
545 FX_DWORD m_dwXrefStartObjNum; 545 FX_DWORD m_dwXrefStartObjNum;
546 546
547 // A map of object numbers to indirect streams. Map owns the streams. 547 // A map of object numbers to indirect streams. Map owns the streams.
548 CFX_MapPtrToPtr m_ObjectStreamMap; 548 std::map<FX_DWORD, std::unique_ptr<CPDF_StreamAcc>> m_ObjectStreamMap;
Oliver Chang 2016/01/20 21:34:24 awesome, we can do this now?
Tom Sepez 2016/01/20 22:18:30 On 2016/01/20 21:34:24, Oliver Chang wrote: Yes, S
549 549
550 // Mapping of object numbers to offsets. The offsets are relative to the first 550 // Mapping of object numbers to offsets. The offsets are relative to the first
551 // object in the stream. 551 // object in the stream.
552 using StreamObjectCache = std::map<FX_DWORD, FX_DWORD>; 552 using StreamObjectCache = std::map<FX_DWORD, FX_DWORD>;
553 553
554 // Mapping of streams to their object caches. This is valid as long as the 554 // Mapping of streams to their object caches. This is valid as long as the
555 // streams in |m_ObjectStreamMap| are valid. 555 // streams in |m_ObjectStreamMap| are valid.
556 std::map<CPDF_StreamAcc*, StreamObjectCache> m_ObjCache; 556 std::map<CPDF_StreamAcc*, StreamObjectCache> m_ObjCache;
557 557
558 // All indirect object numbers that are being parsed. 558 // All indirect object numbers that are being parsed.
(...skipping 438 matching lines...) Expand 10 before | Expand all | Expand 10 after
997 FX_DWORD src_size, 997 FX_DWORD src_size,
998 const CPDF_Dictionary* pDict, 998 const CPDF_Dictionary* pDict,
999 uint8_t*& dest_buf, 999 uint8_t*& dest_buf,
1000 FX_DWORD& dest_size, 1000 FX_DWORD& dest_size,
1001 CFX_ByteString& ImageEncoding, 1001 CFX_ByteString& ImageEncoding,
1002 CPDF_Dictionary*& pImageParms, 1002 CPDF_Dictionary*& pImageParms,
1003 FX_DWORD estimated_size, 1003 FX_DWORD estimated_size,
1004 FX_BOOL bImageAcc); 1004 FX_BOOL bImageAcc);
1005 1005
1006 #endif // CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_ 1006 #endif // CORE_INCLUDE_FPDFAPI_FPDF_PARSER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698