From 39d8880f2cd17c9b96bf73a3bcd7c27a89cf919e Mon Sep 17 00:00:00 2001 From: micschro Date: Wed, 17 Dec 2014 16:31:03 +0100 Subject: [PATCH] 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. --- lib/output.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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(";"); -- 2.34.1