From e7738fa44ef19b502e9e0cf4eed4980138a0942b Mon Sep 17 00:00:00 2001
From: "Rob Swindell (on Windows 11)" <rob@synchro.net>
Date: Mon, 26 Feb 2024 20:00:52 -0800
Subject: [PATCH] Fix commit 78dc9470 - wasn't complete
For some reason, I committed test code that commented-out the actual call
to js_IsTerminated() which avoided the unnecessary error log message. I guess
I was testing to confirm the change actually worked. <shrug>
---
src/sbbs3/js_global.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/sbbs3/js_global.c b/src/sbbs3/js_global.c
index e50393ce4e..bb220cb786 100644
--- a/src/sbbs3/js_global.c
+++ b/src/sbbs3/js_global.c
@@ -729,7 +729,7 @@ js_require(JSContext *cx, uintN argc, jsval *arglist)
if (!JS_IsExceptionPending(cx)) {
if (!JS_HasProperty(cx, exec_obj, property, &found) || !found) {
- if (TRUE) { //!js_IsTerminated(cx, exec_obj)) {
+ if (js_IsTerminated(cx, exec_obj)) {
JSVALUE_TO_MSTRING(cx, argv[fnarg], filename, NULL);
JS_ReportError(cx,"symbol '%s' not defined by script '%s'", property, filename);
}
--
GitLab