From: Mihai Bazon <mihai@bazon.net>
Date: Mon, 3 Sep 2012 08:05:59 +0000 (+0300)
Subject: switch branches must be declared `required` so that the compressor doesn't
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=66c869c8f6d4392efb56a15c6663a1f93b74da88;p=UglifyJS.git

switch branches must be declared `required` so that the compressor doesn't
replace nodes with a single statement.

looks stable for now, though mess begins to sink in.  need to review the AST
hierarchy.
---

diff --git a/lib/ast.js b/lib/ast.js
index bfafac6c..d0acbb41 100644
--- a/lib/ast.js
+++ b/lib/ast.js
@@ -304,11 +304,17 @@ var AST_Switch = DEFNODE("Switch", "expression", {
 }, AST_Statement);
 
 var AST_SwitchBlock = DEFNODE("SwitchBlock", null, {
-    $documentation: "The switch block is somewhat special, hence a special node for it"
+    $documentation: "The switch block is somewhat special, hence a special node for it",
+    initialize: function() {
+        this.required = true;
+    },
 }, AST_BlockStatement);
 
 var AST_SwitchBranch = DEFNODE("SwitchBranch", null, {
-    $documentation: "Base class for `switch` branches"
+    $documentation: "Base class for `switch` branches",
+    initialize: function() {
+        this.required = true;
+    },
 }, AST_BlockStatement);
 
 var AST_Default = DEFNODE("Default", null, {
diff --git a/lib/output.js b/lib/output.js
index 05bd9136..6d86a087 100644
--- a/lib/output.js
+++ b/lib/output.js
@@ -151,7 +151,6 @@ function OutputStream(options) {
                 current_pos++;
             }
             might_need_space = false;
-            maybe_newline();
         }
         var a = str.split(/\r?\n/), n = a.length - 1;
         current_line += n;
@@ -466,7 +465,6 @@ function OutputStream(options) {
     });
     DEFPRINT(AST_BlockStatement, function(self, output){
         var body = self.body;
-        //if (!(body instanceof Array)) body = [ body ];
         if (body.length > 0) output.with_block(function(){
             display_body(body, false, output);
         });