commit b4958712e5d7ad9e475ac228880a4d98c4b612a7
author Tom Sepez <tsepez@chromium.org> Tue Oct 13 20:30:43 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Tue Oct 13 20:30:43 2020 +0000
tree 2945557a920b2d3316ae445473cfe5a2c4f47f2f
parent 4affe8bcc1153e23c95b0a94da790c7927fea896

Make CJS_Object's ObjDefnID be one-based.

Then use 0 as the invalid object id instead of -1.
Then stop shipping about 80 bytes of "-1" as mutable data, since
zero-initialized .bss doesn't cost anything size-wise.

Change-Id: I79956d85e0e2114038d6f67e11954451097d7d19
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/74970
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>

fxjs/cjs_border.cpp

diff --git a/fxjs/cjs_border.cpp b/fxjs/cjs_border.cpp
index 35204d7..1e8bb10 100644
--- a/fxjs/cjs_border.cpp
+++ b/fxjs/cjs_border.cpp
@@ -13,7 +13,7 @@
     {"i", JSConstSpec::String, 0, "inset"},
     {"u", JSConstSpec::String, 0, "underline"}};
 
-int CJS_Border::ObjDefnID = -1;
+uint32_t CJS_Border::ObjDefnID = 0;
 
 // static
 void CJS_Border::DefineJSObjects(CFXJS_Engine* pEngine) {