From: micschro <micschro@users.noreply.github.com>
Date: Wed, 17 Dec 2014 15:31:03 +0000 (+0100)
Subject: Fix max_line_len not working for JSON files
X-Git-Url: https://git.ndcode.org/public/gitweb.cgi?a=commitdiff_plain;h=39d8880f2cd17c9b96bf73a3bcd7c27a89cf919e;p=UglifyJS.git

Fix max_line_len not working for JSON files

As `maybe_newline()` is only called when `might_need_semicolon` is `true`, the `max_line_len` option has no effect for files without (or with very few) semicolons (like JSON files). A simple for this problem is to use `maybe_newline()` instead of `noop` as the `newline()` function in non-beautify mode.
---

diff --git a/lib/output.js b/lib/output.js
index 7fe61af3..b713be83 100644
--- a/lib/output.js
+++ b/lib/output.js
@@ -221,7 +221,7 @@ function OutputStream(options) {
 
     var newline = options.beautify ? function() {
         print("\n");
-    } : noop;
+    } : maybe_newline;
 
     var semicolon = options.beautify ? function() {
         print(";");