diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-12-12 01:53:28 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-12-12 01:53:28 +0000 |
| commit | 909fa2daa49e5439c595e3d17a76f0e05d7934b0 (patch) | |
| tree | 7dac1ee220b8e0968ff42cedf53775b1ca5a62ac /fxjs/cjs_position.cpp | |
| parent | 5e99d62f4d92531833daeb062073d805bf363df3 (diff) | |
| download | pdfium-909fa2daa49e5439c595e3d17a76f0e05d7934b0.tar.xz | |
[js] Convert to using size instead of sentinels
This CL changes the DefineMethod, DefineProps and DefineConsts methods
to pass a size instead of depending on a sentinel value in the
definition arrays.
Change-Id: Ie054544124290c0833a8b21af175a203ca99591a
Reviewed-on: https://pdfium-review.googlesource.com/20551
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fxjs/cjs_position.cpp')
| -rw-r--r-- | fxjs/cjs_position.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fxjs/cjs_position.cpp b/fxjs/cjs_position.cpp index ba1f3d947c..16e4ab093b 100644 --- a/fxjs/cjs_position.cpp +++ b/fxjs/cjs_position.cpp @@ -13,8 +13,7 @@ const JSConstSpec CJS_Position::ConstSpecs[] = { {"textIconV", JSConstSpec::Number, 3, 0}, {"iconTextH", JSConstSpec::Number, 4, 0}, {"textIconH", JSConstSpec::Number, 5, 0}, - {"overlay", JSConstSpec::Number, 6, 0}, - {0, JSConstSpec::Number, 0, 0}}; + {"overlay", JSConstSpec::Number, 6, 0}}; int CJS_Position::ObjDefnID = -1; @@ -22,5 +21,5 @@ int CJS_Position::ObjDefnID = -1; void CJS_Position::DefineJSObjects(CFXJS_Engine* pEngine) { ObjDefnID = pEngine->DefineObj("position", FXJSOBJTYPE_STATIC, nullptr, nullptr); - DefineConsts(pEngine, ObjDefnID, ConstSpecs); + DefineConsts(pEngine, ObjDefnID, ConstSpecs, FX_ArraySize(ConstSpecs)); } |