commit 909fa2daa49e5439c595e3d17a76f0e05d7934b0
author Dan Sinclair <dsinclair@chromium.org> Tue Dec 12 01:53:28 2017 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Dec 12 01:53:28 2017 +0000
tree 7dac1ee220b8e0968ff42cedf53775b1ca5a62ac
parent 5e99d62f4d92531833daeb062073d805bf363df3

[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>

fxjs/cjs_global.cpp

diff --git a/fxjs/cjs_global.cpp b/fxjs/cjs_global.cpp
index 95124ff..2376354 100644
--- a/fxjs/cjs_global.cpp
+++ b/fxjs/cjs_global.cpp
@@ -174,8 +174,7 @@
 }  // namespace
 
 const JSMethodSpec CJS_Global::MethodSpecs[] = {
-    {"setPersistent", setPersistent_static},
-    {0, 0}};
+    {"setPersistent", setPersistent_static}};
 
 int CJS_Global::ObjDefnID = -1;
 
@@ -227,7 +226,7 @@
   ObjDefnID = pEngine->DefineObj("global", FXJSOBJTYPE_STATIC,
                                  JSConstructor<CJS_Global, JSGlobalAlternate>,
                                  JSDestructor<CJS_Global>);
-  DefineMethods(pEngine, ObjDefnID, MethodSpecs);
+  DefineMethods(pEngine, ObjDefnID, MethodSpecs, FX_ArraySize(MethodSpecs));
   DefineAllProperties(pEngine);
 }