fix(codegen): minify large numbers

oxc-project · Aug 14, 2024 · f7acbd6 · f7acbd6
1 parent 4d28d03
commit f7acbd6
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 23 deletions.
diff --git a/crates/oxc_codegen/src/gen.rs b/crates/oxc_codegen/src/gen.rs
@@ -7,7 +7,6 @@ use oxc_span::GetSpan;
 use oxc_syntax::{
     identifier::{LS, PS},
     keyword::is_reserved_keyword_or_global_object,
-    number::NumberBase,
     operator::{BinaryOperator, LogicalOperator, UnaryOperator},
     precedence::{GetPrecedence, Precedence},
 };
@@ -1153,26 +1152,7 @@ impl<'a, const MINIFY: bool> Gen<MINIFY> for NumericLiteral<'a> {
                 p.print_str("-");
             }
 
-            let result = if self.base == NumberBase::Float {
-                print_non_negative_float(abs_value, p)
-            } else {
-                let value = abs_value as u64;
-                // If integers less than 1000, we know that exponential notation will always be longer than
-                // the integer representation. This is not the case for 1000 which is "1e3".
-                if value < 1000 {
-                    format!("{value}")
-                } else if (1_000_000_000_000..=0xFFFF_FFFF_FFFF_F800).contains(&value) {
-                    let hex = format!("{value:#x}");
-                    let result = print_non_negative_float(abs_value, p);
-                    if hex.len() < result.len() {
-                        hex
-                    } else {
-                        result
-                    }
-                } else {
-                    print_non_negative_float(abs_value, p)
-                }
-            };
+            let result = print_non_negative_float(abs_value, p);
             let bytes = result.as_str();
             p.print_str(bytes);
             need_space_before_dot(bytes, p);
@@ -1190,13 +1170,15 @@ impl<'a, const MINIFY: bool> Gen<MINIFY> for NumericLiteral<'a> {
 // <https://github.com/evanw/esbuild/blob/360d47230813e67d0312ad754cad2b6ee09b151b/internal/js_printer/js_printer.go#L3472>
 fn print_non_negative_float<const MINIFY: bool>(value: f64, _p: &Codegen<{ MINIFY }>) -> String {
     use oxc_syntax::number::ToJsString;
-    let mut result = value.to_js_string();
+    if value < 1000.0 && value.fract() == 0.0 {
+        return value.to_js_string();
+    }
+    let mut result = format!("{value}");
     let chars = result.as_bytes();
     let len = chars.len();
     let dot = chars.iter().position(|&c| c == b'.');
     let u8_to_string = |num: &[u8]| {
         // SAFETY: criteria of `from_utf8_unchecked`.are met.
-
         unsafe { String::from_utf8_unchecked(num.to_vec()) }
     };
 
@@ -1243,6 +1225,15 @@ fn print_non_negative_float<const MINIFY: bool>(value: f64, _p: &Codegen<{ MINIF
             result = u8_to_string(chars);
         }
     }
+    if MINIFY && value.fract() == 0.0 {
+        let value = value as u64;
+        if (1_000_000_000_000..=0xFFFF_FFFF_FFFF_F800).contains(&value) {
+            let hex = format!("{value:#x}");
+            if hex.len() < result.len() {
+                result = hex;
+            }
+        }
+    }
 
     result
 }

diff --git a/crates/oxc_codegen/tests/integration/unit.rs b/crates/oxc_codegen/tests/integration/unit.rs
@@ -18,6 +18,9 @@ fn expr() {
         "class Foo {\n\t#test;\n\tbar() {\n\t\tif (!(#test in Foo)) {}\n\t}\n}\n",
     );
     test_minify("x in new Error()", "x in new Error();");
+
+    test("1000000000000000128.0.toFixed(0)", "1000000000000000128.0.toFixed(0);\n");
+    test_minify("1000000000000000128.0.toFixed(0)", "0xde0b6b3a7640080.toFixed(0);");
 }
 
 #[test]