timers: simplify the compareTimersLists function

gurgunday · targos · commit 0ad450f295c1 · 2025-03-11T19:40:07.000+01:00
PR-URL: #57110 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Jason Zhang <xzha4350@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
diff --git a/lib/internal/timers.js b/lib/internal/timers.js
@@ -442,10 +442,7 @@ function getTimerDuration(msecs, name) {
 function compareTimersLists(a, b) {
 const expiryDiff = a.expiry - b.expiry;
 if (expiryDiff === 0) {
- if (a.id < b.id)
- return -1;
- if (a.id > b.id)
- return 1;
+ return a.id - b.id;
 }
 return expiryDiff;
 }

Original file line number	Diff line number	Diff line change
`@@ -442,10 +442,7 @@ function getTimerDuration(msecs, name) {`
`442`	`442`	`functioncompareTimersLists(a,b){`
`443`	`443`	`constexpiryDiff=a.expiry-b.expiry;`
`444`	`444`	`if(expiryDiff===0){`
`445`		`-if(a.id<b.id)`
`446`		`-return-1;`
`447`		`-if(a.id>b.id)`
`448`		`-return1;`
	`445`	`+returna.id-b.id;`
`449`	`446`	`}`
`450`	`447`	`returnexpiryDiff;`
`451`	`448`	`}`