diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-01-11 08:58:35 -0500 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-01-11 14:08:42 +0000 |
| commit | 76a44dea318041f8229d80e70ca3568a435611eb (patch) | |
| tree | 63928177f2a5fe2ea339cf20b10189f621c2acaf /xfa/fxfa/fm2js/xfa_fm2jscontext.h | |
| parent | 98822430907b603d671f8ade5f20723a0f8ecc03 (diff) | |
| download | pdfium-76a44dea318041f8229d80e70ca3568a435611eb.tar.xz | |
Cleaning up memory allocation in CXFA_FM2JSContext - IV
This CL removes the use of FX_Alloc and any remaining new'd CFXJSE_Value objects
from CXFA_FM2JSContext and replaces them with unique_ptrs and vectors.
Change-Id: I30ba697d65ee326d2faa895c3217bdc407419298
Reviewed-on: https://pdfium-review.googlesource.com/2157
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'xfa/fxfa/fm2js/xfa_fm2jscontext.h')
| -rw-r--r-- | xfa/fxfa/fm2js/xfa_fm2jscontext.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/xfa/fxfa/fm2js/xfa_fm2jscontext.h b/xfa/fxfa/fm2js/xfa_fm2jscontext.h index 66e5dcce8b..0f802fe65e 100644 --- a/xfa/fxfa/fm2js/xfa_fm2jscontext.h +++ b/xfa/fxfa/fm2js/xfa_fm2jscontext.h @@ -401,11 +401,11 @@ class CXFA_FM2JSContext : public CFXJSE_HostObject { static bool simpleValueCompare(CFXJSE_Value* pThis, CFXJSE_Value* firstValue, CFXJSE_Value* secondValue); - static void unfoldArgs(CFXJSE_Value* pThis, - CFXJSE_Arguments& args, - CFXJSE_Value**& resultValues, - int32_t& iCount, - int32_t iStart = 0); + static void unfoldArgs( + CFXJSE_Value* pThis, + CFXJSE_Arguments& args, + std::vector<std::unique_ptr<CFXJSE_Value>>* resultValues, + int32_t iStart = 0); static void GetObjectDefaultValue(CFXJSE_Value* pObjectValue, CFXJSE_Value* pDefaultValue); static bool SetObjectDefaultValue(CFXJSE_Value* pObjectValue, |