diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-04-13 13:25:26 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-04-13 20:51:05 +0000 |
| commit | 7062b2632ffa351903e508003788b67a8c8aba77 (patch) | |
| tree | 047730e000ff7127ec834a55e7a416cb7ff4c929 /core/fpdfapi/parser/cpdf_data_avail.h | |
| parent | 742977f943b30638427d1ef6532acbc3d1c36bff (diff) | |
| download | pdfium-7062b2632ffa351903e508003788b67a8c8aba77.tar.xz | |
Use vector of unique ptr in cpdf_data_avail.cpp.
Remove member m_pAcroForm as it is only used in one method,
and immediately set upon entry. A local will be fine here.
Change-Id: I373f68252249a248061828530098ad759907259a
Reviewed-on: https://pdfium-review.googlesource.com/4190
Reviewed-by: dsinclair <dsinclair@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'core/fpdfapi/parser/cpdf_data_avail.h')
| -rw-r--r-- | core/fpdfapi/parser/cpdf_data_avail.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/core/fpdfapi/parser/cpdf_data_avail.h b/core/fpdfapi/parser/cpdf_data_avail.h index 62c5d49c9d..8edf062590 100644 --- a/core/fpdfapi/parser/cpdf_data_avail.h +++ b/core/fpdfapi/parser/cpdf_data_avail.h @@ -235,8 +235,7 @@ class CPDF_DataAvail final { bool m_bHaveAcroForm; uint32_t m_dwAcroFormObjNum; bool m_bAcroFormLoad; - CPDF_Object* m_pAcroForm; - std::vector<CPDF_Object*> m_arrayAcroforms; + std::vector<std::unique_ptr<CPDF_Object>> m_Acroforms; CPDF_Dictionary* m_pPageDict; CPDF_Object* m_pPageResource; bool m_bNeedDownLoadResource; |