diff options
Diffstat (limited to 'src/jsifier.js')
| -rw-r--r-- | src/jsifier.js | 18 |
1 files changed, 15 insertions, 3 deletions
diff --git a/src/jsifier.js b/src/jsifier.js index 846cb4c1..7bff588c 100644 --- a/src/jsifier.js +++ b/src/jsifier.js @@ -247,7 +247,8 @@ function JSify(data, functionsOnly, givenFunctions) { substrate.addActor('GlobalVariable', { processItem: function(item) { function needsPostSet(value) { - return value[0] in UNDERSCORE_OPENPARENS || value.substr(0, 14) === 'CHECK_OVERFLOW'; + return value[0] in UNDERSCORE_OPENPARENS || value.substr(0, 14) === 'CHECK_OVERFLOW' + || value.substr(0, 13) === 'STRING_TABLE.'; } item.intertype = 'GlobalVariableStub'; @@ -264,7 +265,9 @@ function JSify(data, functionsOnly, givenFunctions) { // they would shadow similarly-named globals in the parent. item.JS = ''; } else { - item.JS = 'var ' + item.ident + ';'; + if (!(item.ident in Variables.globals) || !Variables.globals[item.ident].isString) { + item.JS = 'var ' + item.ident + ';'; + } } var constant = null; if (item.external) { @@ -313,7 +316,16 @@ function JSify(data, functionsOnly, givenFunctions) { // allocations in a shared library. constant = makePointer(constant, null, BUILD_AS_SHARED_LIB ? 'ALLOC_NORMAL' : 'ALLOC_STATIC', item.type); - var js = item.ident + '=' + constant + ';'; + var js; + + // Strings are held in STRING_TABLE, to not clutter up the main namespace (in some cases we have + // many many strings, possibly exceeding the js engine limit on global vars). + if (Variables.globals[item.ident].isString) { + js = 'STRING_TABLE.' + item.ident + '=' + constant + ';'; + } else { + js = item.ident + '=' + constant + ';'; + } + // Special case: class vtables. We make sure they are null-terminated, to allow easy runtime operations if (item.ident.substr(0, 5) == '__ZTV') { js += '\n' + makePointer('[0]', null, BUILD_AS_SHARED_LIB ? 'ALLOC_NORMAL' : 'ALLOC_STATIC', ['void*']) + ';'; |