Replace CPDF_SortObjNumArray with a std::set.

Replace CPDF_SortObjNumArray with a std::set.

R=tsepez@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/6caae4d3c6815c5a1e12340734412a3bc7548652

[Lei Zhang, 2016-01-06 00:32:14]

thestig@chromium.org changed reviewers:
+ tsepez@chromium.org

[Tom Sepez, 2016-01-06 00:49:56]

lgtm

https://codereview.chromium.org/1566553002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (right):

https://codereview.chromium.org/1566553002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:2820: std::set<FX_DWORD>
m_objnum_set;
nit: the naming convention for a class that is directly a member seems to be
m_ObjectSet (with caps but no hungarian prefix character), but this is
inconsistent (see m_parser above for example).

[Lei Zhang, 2016-01-06 00:52:26]

https://codereview.chromium.org/1566553002/diff/1/core/src/fpdfapi/fpdf_parse...
File core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp (right):

https://codereview.chromium.org/1566553002/diff/1/core/src/fpdfapi/fpdf_parse...
core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp:2820: std::set<FX_DWORD>
m_objnum_set;
On 2016/01/06 00:49:56, Tom Sepez wrote:
> nit: the naming convention for a class that is directly a member seems to be
> m_ObjectSet (with caps but no hungarian prefix character), but this is
> inconsistent (see m_parser above for example).

m_ObjectSet it is.

[Lei Zhang, 2016-01-06 00:55:10]

Description was changed from

==========
Replace CPDF_SortObjNumArray with a std::set.
==========

to

==========
Replace CPDF_SortObjNumArray with a std::set.

R=tsepez@chromium.org

Committed:
https://pdfium.googlesource.com/pdfium/+/6caae4d3c6815c5a1e12340734412a3bc754...
==========

[Lei Zhang, 2016-01-06 00:55:11]

Committed patchset #2 (id:20001) manually as
6caae4d3c6815c5a1e12340734412a3bc7548652 (presubmit successful).