diff options
| author |  Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com>
| 2023-04-26 22:32:48 -0700 |
|---|---|---|
| committer | Jarred Sumner <709451+Jarred-Sumner@users.noreply.github.com>
| 2023-04-26 22:32:48 -0700 |
| commit | 6142f6eb61ff95821363c44c709e3a038e61dd54 (patch) | |
| tree | 8719b6ba24198067189e8b30e8739c6007bd69eb | |
| parent | 316a75005b393be4054a2be0acf9d14a4871ba22 (diff) | |
| download | bun-6142f6eb61ff95821363c44c709e3a038e61dd54.tar.gz bun-6142f6eb61ff95821363c44c709e3a038e61dd54.tar.zst bun-6142f6eb61ff95821363c44c709e3a038e61dd54.zip | |
Wrap some things in minify_syntax flag
| -rw-r--r-- | src/js_parser.zig | 65 |
1 files changed, 34 insertions, 31 deletions
diff --git a/src/js_parser.zig b/src/js_parser.zig index f49247f9c..6b34dbd84 100644 --- a/src/js_parser.zig +++ b/src/js_parser.zig @@ -15607,11 +15607,6 @@ fn NewParser_( return e_.right; } } - - // TODO: - // "(1 && fn)()" => "fn()" - // "(1 && this.fn)" => "this.fn" - // "(1 && this.fn)()" => "(0, this.fn)()" }, .bin_add => { if (p.should_fold_typescript_constant_expressions) { @@ -15891,7 +15886,10 @@ fn NewParser_( // though this is a run-time error, we make it a compile-time error when // bundling because scope hoisting means these will no longer be run-time // errors. - if ((in.assign_target != .none or is_delete_target) and @as(Expr.Tag, e_.target.data) == .e_identifier and p.symbols.items[e_.target.data.e_identifier.ref.innerIndex()].kind == .import) { + if ((in.assign_target != .none or is_delete_target) and + @as(Expr.Tag, e_.target.data) == .e_identifier and + p.symbols.items[e_.target.data.e_identifier.ref.innerIndex()].kind == .import) + { const r = js_lexer.rangeOfIdentifier(p.source, e_.target.loc); p.log.addRangeErrorFmt( p.source, @@ -15935,15 +15933,18 @@ fn NewParser_( switch (e_.op) { .un_not => { - e_.value = SideEffects.simplifyBoolean(p, e_.value); + if (p.options.features.minify_syntax) + e_.value = SideEffects.simplifyBoolean(p, e_.value); const side_effects = SideEffects.toBoolean(e_.value.data); if (side_effects.ok) { return p.newExpr(E.Boolean{ .value = !side_effects.value }, expr.loc); } - if (e_.value.maybeSimplifyNot(p.allocator)) |exp| { - return exp; + if (p.options.features.minify_syntax) { + if (e_.value.maybeSimplifyNot(p.allocator)) |exp| { + return exp; + } } }, .un_void => { @@ -15979,28 +15980,30 @@ fn NewParser_( else => {}, } - // "-(a, b)" => "a, -b" - if (switch (e_.op) { - .un_delete, .un_typeof => false, - else => true, - }) { - switch (e_.value.data) { - .e_binary => |comma| { - if (comma.op == .bin_comma) { - return Expr.joinWithComma( - comma.left, - p.newExpr( - E.Unary{ - .op = e_.op, - .value = comma.right, - }, - comma.right.loc, - ), - p.allocator, - ); - } - }, - else => {}, + if (p.options.features.minify_syntax) { + // "-(a, b)" => "a, -b" + if (switch (e_.op) { + .un_delete, .un_typeof => false, + else => true, + }) { + switch (e_.value.data) { + .e_binary => |comma| { + if (comma.op == .bin_comma) { + return Expr.joinWithComma( + comma.left, + p.newExpr( + E.Unary{ + .op = e_.op, + .value = comma.right, + }, + comma.right.loc, + ), + p.allocator, + ); + } + }, + else => {}, + } } } }, |