timers: cleanup no-longer relevant TODOs in timers/promises

PR-URL: #46499 Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Feb 5, 2023 · 06b28fb · 06b28fb
1 parent 03854f6
commit 06b28fb
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/lib/timers/promises.js b/lib/timers/promises.js
@@ -70,10 +70,8 @@ function setTimeout(after, value, options = kEmptyObject) {
         'boolean',
         ref));
   }
-  // TODO(@jasnell): If a decision is made that this cannot be backported
-  // to 12.x, then this can be converted to use optional chaining to
-  // simplify the check.
-  if (signal && signal.aborted) {
+
+  if (signal?.aborted) {
     return PromiseReject(new AbortError(undefined, { cause: signal.reason }));
   }
   let oncancel;
@@ -113,10 +111,8 @@ function setImmediate(value, options = kEmptyObject) {
         'boolean',
         ref));
   }
-  // TODO(@jasnell): If a decision is made that this cannot be backported
-  // to 12.x, then this can be converted to use optional chaining to
-  // simplify the check.
-  if (signal && signal.aborted) {
+
+  if (signal?.aborted) {
     return PromiseReject(new AbortError(undefined, { cause: signal.reason }));
   }
   let oncancel;