Fix max_line_len not working for JSON files
authormicschro <micschro@users.noreply.github.com>
Wed, 17 Dec 2014 15:31:03 +0000 (16:31 +0100)
committermicschro <micschro@users.noreply.github.com>
Wed, 17 Dec 2014 15:31:03 +0000 (16:31 +0100)
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.

lib/output.js

index 7fe61af..b713be8 100644 (file)
@@ -221,7 +221,7 @@ function OutputStream(options) {
 
     var newline = options.beautify ? function() {
         print("\n");
-    } : noop;
+    } : maybe_newline;
 
     var semicolon = options.beautify ? function() {
         print(";");