Commit f86ca89
util: Fix number format for pad
`pad` is now using `toString(10)`, actually we don't need to do this. As for https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/toString, `toString(N)` is a radix converter, which isn't proper here for time conversion.
PR-URL: #21906
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Ruby <rubys@intertwingly.net>1 parent e8e014a commit f86ca89
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1301,7 +1301,7 @@ function isPrimitive(arg) { | |||
| 1301 | 1301 | } | |
| 1302 | 1302 | ||
| 1303 | 1303 | function pad(n) { | |
| 1304 | - return n < 10 ? `0${n.toString(10)}` : n.toString(10); | ||
| 1304 | + return n.toString().padStart(2, '0'); | ||
| 1305 | 1305 | } | |
| 1306 | 1306 | ||
| 1307 | 1307 | const months = ['Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', | |
0 commit comments