timers: remove redundant unref calls

PR-URL: #38320 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Apr 29, 2021 · b3ca1b3 · b3ca1b3
1 parent 769a210
commit b3ca1b3
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/timers/promises.js b/lib/timers/promises.js
@@ -61,8 +61,7 @@ function setTimeout(after, value, options = {}) {
   }
   let oncancel;
   const ret = new Promise((resolve, reject) => {
-    const timeout = new Timeout(resolve, after, args, false, true);
-    if (!ref) timeout.unref();
+    const timeout = new Timeout(resolve, after, args, false, ref);
     insert(timeout, timeout._idleTimeout);
     if (signal) {
       oncancel = FunctionPrototypeBind(cancelListenerHandler,
@@ -141,8 +140,7 @@ async function* setInterval(after, value, options = {}) {
         callback();
         callback = undefined;
       }
-    }, after, undefined, true, true);
-    if (!ref) interval.unref();
+    }, after, undefined, true, ref);
     insert(interval, interval._idleTimeout);
     if (signal) {
       onCancel = () => {