Fix Error.stackTraceLimit = Infinity (#861)

Author: ammarahm-ed

quickjs.c

@@ -38051,10 +38051,16 @@ static JSValue js_error_set_stackTraceLimit(JSContext *ctx, JSValue this_val, JS
 {
     if (JS_IsUndefined(this_val) || JS_IsNull(this_val))
         return JS_ThrowTypeErrorNotAnObject(ctx);
-    int limit;
-    if (JS_ToInt32(ctx, &limit, value) < 0)
+    double limit = 0;
+    if (JS_ToFloat64(ctx, &limit, value) < 0)
         return JS_EXCEPTION;
-    ctx->error_stack_trace_limit = limit;
+    if (isfinite(limit)) {
+        ctx->error_stack_trace_limit = limit;
+    } else if (isnan(limit) || limit == -INFINITY) {
+        ctx->error_stack_trace_limit = 0;
+    } else {
+        ctx->error_stack_trace_limit = INT32_MAX;
+    }
     return JS_UNDEFINED;
 }
 

tests/bug858.js

@@ -0,0 +1,10 @@
+import {assert} from "./assert.js";
+
+Error.stackTraceLimit = Infinity;
+assert(new Error("error").stack.includes("bug858.js")); // stack should not be empty
+
+Error.stackTraceLimit = -Infinity;
+assert(!new Error("error").stack.includes("bug858.js")); // stack should be empty
+
+Error.stackTraceLimit = NaN;
+assert(!new Error("error").stack.includes("bug858.js")); // stack should be empty