From 72a1b26ce4afd4635acd3d094591305ce1029c76 Mon Sep 17 00:00:00 2001 From: Sami Vaarala Date: Tue, 4 Nov 2014 00:25:44 +0200 Subject: [PATCH] Add deep C stack for dukweb.js By default dukweb.js gets a shallow stack intended for embedded environments. This results in C recursion limits being hit for large expression that work fine on the command line, see e.g. GH-67. --- Makefile | 2 +- RELEASES.rst | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 17b69bba..83508bf7 100644 --- a/Makefile +++ b/Makefile @@ -559,7 +559,7 @@ emscriptenduktest: emscripten dist # and providing an eval() facility from both sides. This is a placeholder now # and doesn't do anything useful yet. EMCCOPTS_DUKWEB_EXPORT=-s EXPORTED_FUNCTIONS='["_dukweb_is_open", "_dukweb_open","_dukweb_close","_dukweb_eval"]' -EMCCOPTS_DUKWEB_DEFINES=-DDUK_OPT_ASSERTIONS -DDUK_OPT_SELF_TESTS '-DDUK_OPT_DECLARE=extern void dukweb_panic_handler(int code, const char *msg);' '-DDUK_OPT_PANIC_HANDLER(code,msg)={dukweb_panic_handler((code),(msg));abort();}' +EMCCOPTS_DUKWEB_DEFINES=-DDUK_OPT_ASSERTIONS -DDUK_OPT_SELF_TESTS -DDUK_OPT_DEEP_C_STACK '-DDUK_OPT_DECLARE=extern void dukweb_panic_handler(int code, const char *msg);' '-DDUK_OPT_PANIC_HANDLER(code,msg)={dukweb_panic_handler((code),(msg));abort();}' # FIXME: need to be able to declare dukweb_panic_handler to avoid warnings dukweb.js: emscripten dist diff --git a/RELEASES.rst b/RELEASES.rst index 2105edfe..a73521e0 100644 --- a/RELEASES.rst +++ b/RELEASES.rst @@ -623,6 +623,9 @@ Planned previously fileName would always be duk_bi_global.c which is misleading (see GH-58) +* Use deep C stack for dukweb.js to remove some compiler recursion limit + limitations (see GH-67) + 1.2.0 (2015-XX-XX) ------------------