perf: use Intl.NumberFormat instead of toLocaleString (#13949)

vitejs · Jul 25, 2023 · a48bf88 · a48bf88
1 parent d4f13bd
commit a48bf88
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/packages/vite/src/node/plugins/reporter.ts b/packages/vite/src/node/plugins/reporter.ts
@@ -26,6 +26,14 @@ export function buildReporterPlugin(config: ResolvedConfig): Plugin {
   const compress = promisify(gzip)
   const chunkLimit = config.build.chunkSizeWarningLimit
 
+  const numberFormatter = new Intl.NumberFormat('en', {
+    maximumFractionDigits: 2,
+    minimumFractionDigits: 2,
+  })
+  const displaySize = (bytes: number) => {
+    return `${numberFormatter.format(bytes / 1000)} kB`
+  }
+
   const tty = process.stdout.isTTY && !process.env.CI
   const shouldLogInfo = LogLevels[config.logLevel || 'info'] >= LogLevels.info
   let hasTransformed = false
@@ -322,13 +330,6 @@ function throttle(fn: Function) {
   }
 }
 
-function displaySize(bytes: number) {
-  return `${(bytes / 1000).toLocaleString('en', {
-    maximumFractionDigits: 2,
-    minimumFractionDigits: 2,
-  })} kB`
-}
-
 function displayTime(time: number) {
   // display: {X}ms
   if (time < 1000) {