Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp

Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp

Replace two parallel arrays with single array of pairs.

Committed: https://pdfium.googlesource.com/pdfium/+/ca612c1f9c47fae74dda493291d52ec27eff4a7b

[Tom Sepez, 2016-04-21 00:07:36]

Description was changed from

==========
Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp
==========

to

==========
Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp

Replace two parallel arrays with single array of pairs.
==========

[Tom Sepez, 2016-04-21 00:07:36]

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

[Tom Sepez, 2016-04-21 00:07:47]

Lei, for review.

[Lei Zhang, 2016-04-21 00:51:58]

lgtm

https://codereview.chromium.org/1911673002/diff/20001/core/fpdfapi/fpdf_parse...
File core/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp (right):

https://codereview.chromium.org/1911673002/diff/20001/core/fpdfapi/fpdf_parse...
core/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp:355: pParams ?
pParamsArray->GetDictAt(i) : nullptr});
Use |pParamsArray| instead of |pParams| and don't bother with lines 350/351?

[Tom Sepez, 2016-04-21 17:48:07]

https://codereview.chromium.org/1911673002/diff/20001/core/fpdfapi/fpdf_parse...
File core/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp (right):

https://codereview.chromium.org/1911673002/diff/20001/core/fpdfapi/fpdf_parse...
core/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp:355: pParams ?
pParamsArray->GetDictAt(i) : nullptr});
On 2016/04/21 00:51:58, Lei Zhang wrote:
> Use |pParamsArray| instead of |pParams| and don't bother with lines 350/351?

Done.

[Tom Sepez, 2016-04-21 17:48:59]

The CQ bit was checked by tsepez@chromium.org

[Tom Sepez, 2016-04-21 17:48:59]

The patchset sent to the CQ was uploaded after l-g-t-m from thestig@chromium.org
Link to the patchset: https://codereview.chromium.org/1911673002/#ps40001
(title: "tidy")

[commit-bot: I haz the power, 2016-04-21 17:49:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1911673002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1911673002/40001

[commit-bot: I haz the power, 2016-04-21 18:28:06]

Description was changed from

==========
Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp

Replace two parallel arrays with single array of pairs.
==========

to

==========
Remove CFX_ArrayTemplate from fpdf_parser_decode.cpp

Replace two parallel arrays with single array of pairs.

Committed:
https://pdfium.googlesource.com/pdfium/+/ca612c1f9c47fae74dda493291d52ec27eff...
==========

[commit-bot: I haz the power, 2016-04-21 18:28:07]

Committed patchset #3 (id:40001) as
https://pdfium.googlesource.com/pdfium/+/ca612c1f9c47fae74dda493291d52ec27eff...