implement --line-length

mishoo · Jul 5, 2022 · 8788eeb · 8788eeb
1 parent aa2a9fb
commit 8788eeb
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -118,6 +118,7 @@ a double dash to prevent input files being used as option arguments:
     --keep-fargs                Do not mangle/drop function arguments.
     --keep-fnames               Do not mangle/drop function names.  Useful for
                                 code relying on Function.prototype.name.
+    -l, --line-length <value>   Maximum line length for output code.
     --module                    Process input as ES module (implies --toplevel)
     --name-cache <file>         File to hold mangled name mappings.
     --self                      Build UglifyJS as a library (implies --wrap UglifyJS)

diff --git a/bin/uglifyjs b/bin/uglifyjs
@@ -21,6 +21,7 @@ var short_forms = {
     d: "define",
     e: "enclose",
     h: "help",
+    l: "line-length",
     m: "mangle",
     o: "output",
     O: "output-opts",
@@ -107,6 +108,7 @@ function process_option(name, no_value) {
                 "  --ie                                     Support non-standard Internet Explorer.",
                 "  --keep-fargs                             Do not mangle/drop function arguments.",
                 "  --keep-fnames                            Do not mangle/drop function names. Useful for code relying on Function.prototype.name.",
+                "  -l, --line-length <value>                Maximum line length for output code.",
                 "  --module                                 Process input as ES module (implies --toplevel)",
                 "  --name-cache <file>                      File to hold mangled name mappings.",
                 "  --rename                                 Force symbol expansion.",
@@ -153,6 +155,9 @@ function process_option(name, no_value) {
       case "annotations":
       case "ie":
       case "ie8":
+      case "line-length":
+        options.max_line_len = read_value(true);
+        break;
       case "module":
       case "timings":
       case "toplevel":

diff --git a/lib/minify.js b/lib/minify.js
@@ -80,6 +80,7 @@ function minify(files, options) {
             ie8: false,
             keep_fargs: false,
             keep_fnames: false,
+            max_line_len: undefined,
             mangle: {},
             module: false,
             nameCache: null,
@@ -214,6 +215,7 @@ function minify(files, options) {
         var output = defaults(options.output, {
             ast: false,
             code: true,
+            max_line_len: options.max_line_len,
         });
         if (output.ast) result.ast = toplevel;
         if (output.code) {

diff --git a/test/mocha/cli.js b/test/mocha/cli.js
@@ -964,4 +964,20 @@ describe("bin/uglifyjs", function() {
             done();
         }).stdin.end(code);
     });
+    it("Should fail with empty --line-length", function(done) {
+        exec(uglifyjscmd + " -l", function(err, stdout, stderr) {
+            assert.ok(err);
+            assert.strictEqual(stdout, "");
+            assert.strictEqual(stderr, "ERROR: missing option argument for --line-length\n");
+            done();
+        })
+    });
+    it("Should work with --line-length", function(done) {
+        exec(uglifyjscmd + " --line-length 20 test/input/reduce/label.js", function(err, stdout, stderr) {
+            if (err) throw err
+            assert.strictEqual(stdout, "UNUSED:{console.log(\n0-.1-.1-.1)}\n");
+            assert.strictEqual(stderr, "");
+            done();
+        })
+    });
 });