timers: document ref option for scheduler.wait

ShogunPanda · targos · commit b6ed97c66d8c · 2024-10-02T07:57:48.000+02:00
PR-URL: #54605 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
diff --git a/doc/api/timers.md b/doc/api/timers.md
@@ -521,6 +521,9 @@ added:
 * `delay` {number} The number of milliseconds to wait before resolving the
   promise.
 * `options` {Object}
+  * `ref` {boolean} Set to `false` to indicate that the scheduled `Timeout`
+    should not require the Node.js event loop to remain active.
+    **Default:** `true`.
   * `signal` {AbortSignal} An optional `AbortSignal` that can be used to
     cancel waiting.
 * Returns: {Promise}
diff --git a/lib/timers/promises.js b/lib/timers/promises.js
@@ -131,22 +131,18 @@ function setImmediate(value, options = kEmptyObject) {
 }
 
 async function* setInterval(after, value, options = kEmptyObject) {
-  try {
-    if (typeof after !== 'undefined') {
-      validateNumber(after, 'delay');
-    }
+  if (typeof after !== 'undefined') {
+    validateNumber(after, 'delay');
+  }
 
-    validateObject(options, 'options');
+  validateObject(options, 'options');
 
-    if (typeof options?.signal !== 'undefined') {
-      validateAbortSignal(options.signal, 'options.signal');
-    }
+  if (typeof options?.signal !== 'undefined') {
+    validateAbortSignal(options.signal, 'options.signal');
+  }
 
-    if (typeof options?.ref !== 'undefined') {
-      validateBoolean(options.ref, 'options.ref');
-    }
-  } catch (err) {
-    return PromiseReject(err);
+  if (typeof options?.ref !== 'undefined') {
+    validateBoolean(options.ref, 'options.ref');
   }
 
   const { signal, ref = true } = options;
