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_display.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_display.cpp')
| -rw-r--r-- | fxjs/cjs_display.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fxjs/cjs_display.cpp b/fxjs/cjs_display.cpp index a52b51d462..6970bfcb51 100644 --- a/fxjs/cjs_display.cpp +++ b/fxjs/cjs_display.cpp @@ -10,8 +10,7 @@ const JSConstSpec CJS_Display::ConstSpecs[] = { {"visible", JSConstSpec::Number, 0, 0}, {"hidden", JSConstSpec::Number, 1, 0}, {"noPrint", JSConstSpec::Number, 2, 0}, - {"noView", JSConstSpec::Number, 3, 0}, - {0, JSConstSpec::Number, 0, 0}}; + {"noView", JSConstSpec::Number, 3, 0}}; int CJS_Display::ObjDefnID = -1; @@ -19,5 +18,5 @@ int CJS_Display::ObjDefnID = -1; void CJS_Display::DefineJSObjects(CFXJS_Engine* pEngine) { ObjDefnID = pEngine->DefineObj("display", FXJSOBJTYPE_STATIC, nullptr, nullptr); - DefineConsts(pEngine, ObjDefnID, ConstSpecs); + DefineConsts(pEngine, ObjDefnID, ConstSpecs, FX_ArraySize(ConstSpecs)); } |