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

Issue 2042843003: xfa_fm2jscontext method cleanup - pt V (Closed)

Created:
4 years, 6 months ago by dsinclair
Modified:
4 years, 6 months ago
Reviewers:
Tom Sepez, Lei Zhang
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

xfa_fm2jscontext method cleanup - pt V Cleans up ::Oneof, ::HasValue, ::Exists, ::Choose, ::Term, ::Rate, ::PV, ::PPmt, ::Pmt, ::NPV, ::IPmt, ::FV, ::CTerm and ::Apr Committed: https://pdfium.googlesource.com/pdfium/+/96a05f40eca8c5e717e38b050997ff6248842e21

Patch Set 1 #

Total comments: 5

Patch Set 2 : Rebase to master #

Patch Set 3 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+429 lines, -499 lines) Patch
M xfa/fxfa/fm2js/xfa_fm2jscontext.cpp View 1 2 11 chunks +429 lines, -499 lines 0 comments Download

Messages

Total messages: 11 (5 generated)
dsinclair
PTAL. https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp File xfa/fxfa/fm2js/xfa_fm2jscontext.cpp (right): https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp#newcode2604 xfa/fxfa/fm2js/xfa_fm2jscontext.cpp:2604: std::vector<FX_DOUBLE> data; Converted this to a vector.
4 years, 6 months ago (2016-06-06 21:04:07 UTC) #2
dsinclair
PTAL.
4 years, 6 months ago (2016-06-06 21:04:07 UTC) #4
Tom Sepez
lgtm https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp File xfa/fxfa/fm2js/xfa_fm2jscontext.cpp (right): https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp#newcode2604 xfa/fxfa/fm2js/xfa_fm2jscontext.cpp:2604: std::vector<FX_DOUBLE> data; probably want data(argc - 1) ctor ...
4 years, 6 months ago (2016-06-06 21:22:59 UTC) #5
dsinclair
https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp File xfa/fxfa/fm2js/xfa_fm2jscontext.cpp (right): https://codereview.chromium.org/2042843003/diff/1/xfa/fxfa/fm2js/xfa_fm2jscontext.cpp#newcode2604 xfa/fxfa/fm2js/xfa_fm2jscontext.cpp:2604: std::vector<FX_DOUBLE> data; On 2016/06/06 21:22:59, Tom Sepez wrote: > ...
4 years, 6 months ago (2016-06-07 13:37:59 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2042843003/40001
4 years, 6 months ago (2016-06-07 13:39:25 UTC) #9
commit-bot: I haz the power
4 years, 6 months ago (2016-06-07 13:48:06 UTC) #11
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://pdfium.googlesource.com/pdfium/+/96a05f40eca8c5e717e38b050997ff624884...

Powered by Google App Engine
This is Rietveld 408576698