Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(233)

Side by Side Diff: fpdfsdk/src/javascript/global.cpp

Issue 792043005: Remove run-time calculation of hash constants in pdfium. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Tweaks to the CHash class, and removing zeroes. Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include "../../include/javascript/JavaScript.h" 7 #include "../../include/javascript/JavaScript.h"
8 #include "../../include/javascript/IJavaScript.h" 8 #include "../../include/javascript/IJavaScript.h"
9 #include "../../include/javascript/JS_Define.h" 9 #include "../../include/javascript/JS_Define.h"
10 #include "../../include/javascript/JS_Object.h" 10 #include "../../include/javascript/JS_Object.h"
11 #include "../../include/javascript/JS_Value.h" 11 #include "../../include/javascript/JS_Value.h"
12 #include "../../include/javascript/JS_GlobalData.h" 12 #include "../../include/javascript/JS_GlobalData.h"
13 #include "../../include/javascript/global.h" 13 #include "../../include/javascript/global.h"
14 #include "../../include/javascript/JS_EventHandler.h" 14 #include "../../include/javascript/JS_EventHandler.h"
15 #include "../../include/javascript/JS_Context.h" 15 #include "../../include/javascript/JS_Context.h"
16 16
17 /* ---------------------------- global ---------------------------- */ 17 /* ---------------------------- global ---------------------------- */
18 18
19 extern const unsigned int JSCONST_nStringHash = JS_CalcHash(VALUE_NAME_STRING,wc slen(VALUE_NAME_STRING)); 19 // Helper class for compile-time calculation of hash values in order to
20 extern const unsigned int JSCONST_nNumberHash = JS_CalcHash(VALUE_NAME_NUMBER,wc slen(VALUE_NAME_NUMBER)); 20 // avoid having global object initializers.
21 extern const unsigned int JSCONST_nBoolHash = JS_CalcHash(VALUE_NAME_BOOLEAN,wcs len(VALUE_NAME_BOOLEAN)); 21 template <unsigned ACC, wchar_t... Ns>
22 extern const unsigned int JSCONST_nDateHash = JS_CalcHash(VALUE_NAME_DATE,wcslen (VALUE_NAME_DATE)); 22 struct CHash;
23 extern const unsigned int JSCONST_nObjectHash = JS_CalcHash(VALUE_NAME_OBJECT,wc slen(VALUE_NAME_OBJECT)); 23
24 extern const unsigned int JSCONST_nFXobjHash = JS_CalcHash(VALUE_NAME_FXOBJ,wcsl en(VALUE_NAME_FXOBJ)); 24 // Only needed to hash single-character strings.
25 extern const unsigned int JSCONST_nNullHash = JS_CalcHash(VALUE_NAME_NULL,wcslen (VALUE_NAME_NULL)); 25 template <wchar_t N>
26 extern const unsigned int JSCONST_nUndefHash = JS_CalcHash(VALUE_NAME_UNDEFINED, wcslen(VALUE_NAME_UNDEFINED)); 26 struct CHash<N> {
27 static const unsigned value = N;
28 };
29
30 template <unsigned ACC, wchar_t N>
31 struct CHash<ACC, N> {
32 static const unsigned value = (ACC * 1313LLU + N) & 0xFFFFFFFF;
33 };
34
35 template <unsigned ACC, wchar_t N, wchar_t... Ns>
36 struct CHash<ACC, N, Ns...> {
37 static const unsigned value = CHash<CHash<ACC, N>::value, Ns...>::value;
38 };
39
40 extern const unsigned int JSCONST_nStringHash =
41 CHash<'s','t','r','i','n','g'>::value;
42 extern const unsigned int JSCONST_nNumberHash =
43 CHash<'n','u','m','b','e','r'>::value;
44 extern const unsigned int JSCONST_nBoolHash =
45 CHash<'b','o','o','l','e','a','n'>::value;
46 extern const unsigned int JSCONST_nDateHash =
47 CHash<'d','a','t','e'>::value;
48 extern const unsigned int JSCONST_nObjectHash =
49 CHash<'o','b','j','e','c','t'>::value;
50 extern const unsigned int JSCONST_nFXobjHash =
51 CHash<'f','x','o','b','j'>::value;
52 extern const unsigned int JSCONST_nNullHash =
53 CHash<'n','u','l','l'>::value;
54 extern const unsigned int JSCONST_nUndefHash =
55 CHash<'u','n','d','e','f','i','n','e','d'>::value;
56
57 #ifdef _DEBUG
58 class HashVerify
59 {
60 public:
61 HashVerify();
62 } g_hashVerify;
63
64 HashVerify::HashVerify()
65 {
66 ASSERT(JSCONST_nStringHash ==
67 JS_CalcHash(VALUE_NAME_STRING,wcslen(VALUE_NAME_STRING)));
68 ASSERT(JSCONST_nNumberHash ==
69 JS_CalcHash(VALUE_NAME_NUMBER,wcslen(VALUE_NAME_NUMBER)));
70 ASSERT(JSCONST_nBoolHash ==
71 JS_CalcHash(VALUE_NAME_BOOLEAN,wcslen(VALUE_NAME_BOOLEAN)));
72 ASSERT(JSCONST_nDateHash ==
73 JS_CalcHash(VALUE_NAME_DATE,wcslen(VALUE_NAME_DATE)));
74 ASSERT(JSCONST_nObjectHash ==
75 JS_CalcHash(VALUE_NAME_OBJECT,wcslen(VALUE_NAME_OBJECT)));
76 ASSERT(JSCONST_nFXobjHash ==
77 JS_CalcHash(VALUE_NAME_FXOBJ,wcslen(VALUE_NAME_FXOBJ)));
78 ASSERT(JSCONST_nNullHash ==
79 JS_CalcHash(VALUE_NAME_NULL,wcslen(VALUE_NAME_NULL)));
80 ASSERT(JSCONST_nUndefHash ==
81 JS_CalcHash(VALUE_NAME_UNDEFINED,wcslen(VALUE_NAME_UNDEFINED)));
82 }
83 #endif
84
27 85
28 BEGIN_JS_STATIC_CONST(CJS_Global) 86 BEGIN_JS_STATIC_CONST(CJS_Global)
29 END_JS_STATIC_CONST() 87 END_JS_STATIC_CONST()
30 88
31 BEGIN_JS_STATIC_PROP(CJS_Global) 89 BEGIN_JS_STATIC_PROP(CJS_Global)
32 END_JS_STATIC_PROP() 90 END_JS_STATIC_PROP()
33 91
34 BEGIN_JS_STATIC_METHOD(CJS_Global) 92 BEGIN_JS_STATIC_METHOD(CJS_Global)
35 JS_STATIC_METHOD_ENTRY(setPersistent, 2) 93 JS_STATIC_METHOD_ENTRY(setPersistent, 2)
36 END_JS_STATIC_METHOD() 94 END_JS_STATIC_METHOD()
(...skipping 513 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 } 608 }
551 break; 609 break;
552 default: 610 default:
553 return FALSE; 611 return FALSE;
554 } 612 }
555 613
556 m_mapGlobal.SetAt(propname, (FX_LPVOID)pNewData); 614 m_mapGlobal.SetAt(propname, (FX_LPVOID)pNewData);
557 615
558 return TRUE; 616 return TRUE;
559 } 617 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698