Commit e54b433
util: use ES2015+ Object.is to check negative zero
Use `Object.is` to check whether the value is negative zero or not.
Ref: b3e4fc6
PR-URL: #11332
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent e7b7d72 commit e54b433
1 file changed
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -598,9 +598,8 @@ function formatValue(ctx, value, recurseTimes) { | |||
| 598 | 598 | ||
| 599 | 599 | ||
| 600 | 600 | function formatNumber(ctx, value) { | |
| 601 | - // Format -0 as '-0'. Strict equality won't distinguish 0 from -0, | ||
| 602 | - // so instead we use the fact that 1 / -0 < 0 whereas 1 / 0 > 0 . | ||
| 603 | - if (value === 0 && 1 / value < 0) | ||
| 601 | + // Format -0 as '-0'. Strict equality won't distinguish 0 from -0. | ||
| 602 | + if (Object.is(value, -0)) | ||
| 604 | 603 | return ctx.stylize('-0', 'number'); | |
| 605 | 604 | return ctx.stylize('' + value, 'number'); | |
| 606 | 605 | } | |
0 commit comments