feat: remove max-width

src/builder.rs

@@ -296,6 +296,7 @@ pub fn fits_in_single_line(
     build_ctx: &crate::builder::BuildCtx,
     node: rnix::SyntaxElement,
 ) -> bool {
+    let line = build_ctx.pos_new.line;
     let maybe_green_node = build(
         &build_ctx.config.with_layout(crate::config::Layout::Wide),
         node,
@@ -304,10 +305,7 @@ pub fn fits_in_single_line(
     );
 
     match maybe_green_node {
-        Some(finished) => {
-            build_ctx.pos_new.column + finished.to_string().chars().count()
-                <= build_ctx.config.max_width()
-        }
+        Some(_) => build_ctx.pos_new.line == line,
         None => false,
     }
 }

src/cli.rs

@@ -8,14 +8,6 @@ pub fn parse(args: Vec<String>) -> clap::ArgMatches {
                 .short('d')
                 .takes_value(false),
         )
-        .arg(
-            clap::Arg::new("max-width")
-                .default_value("80")
-                .help("How many characters per line to allow.")
-                .long("max-width")
-                .takes_value(true)
-                .value_name("CHARS"),
-        )
         .arg(
             clap::Arg::new("paths")
                 .help("Files or directories, or none to format stdin.")

src/config.rs

@@ -6,14 +6,13 @@ pub enum Layout {
 
 #[derive(Clone)]
 pub struct Config {
-    debug:     bool,
-    layout:    Layout,
-    max_width: usize,
+    debug:  bool,
+    layout: Layout,
 }
 
 impl Config {
     pub fn new() -> Config {
-        Config { debug: false, layout: Layout::Tall, max_width: 80 }
+        Config { debug: false, layout: Layout::Tall }
     }
 
     pub fn debug(&self) -> bool {
@@ -24,19 +23,11 @@ impl Config {
         &self.layout
     }
 
-    pub fn max_width(&self) -> usize {
-        self.max_width
-    }
-
     pub fn with_debug(&self, debug: bool) -> Config {
-        Config { debug, layout: self.layout.clone(), max_width: self.max_width }
+        Config { debug, layout: self.layout.clone() }
     }
 
     pub fn with_layout(&self, layout: Layout) -> Config {
-        Config { debug: self.debug, layout, max_width: self.max_width }
-    }
-
-    pub fn with_max_width(&self, max_width: usize) -> Config {
-        Config { debug: self.debug, layout: self.layout.clone(), max_width }
+        Config { debug: self.debug, layout }
     }
 }

src/main.rs

@@ -5,12 +5,7 @@ fn main() -> std::io::Result<()> {
     let matches = alejandra::cli::parse(std::env::args().collect());
 
     let debug: bool = matches.is_present("debug");
-    let max_width: usize =
-        matches.value_of("max-width").unwrap().parse().unwrap();
-
-    let config = alejandra::config::Config::new()
-        .with_debug(debug)
-        .with_max_width(max_width);
+    let config = alejandra::config::Config::new().with_debug(debug);
 
     match matches.values_of("paths") {
         Some(paths) => {

tests/cases/assert/in

@@ -4,5 +4,6 @@
   (assert /*a*/ b;       c)
   (assert /*a*/ b; /*b*/ c)
   ( assert b; cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
-  ( assert b; cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
+  ( assert b;
+  cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
 ]

tests/cases/attr_set/in

@@ -19,7 +19,12 @@
   rec /*a*/ { /*b*/ c=1;       }
   rec /*a*/ { /*b*/ c=1; /*d*/ }
 
-  {a=rec {a={a=rec {a={a=rec {a={a=rec {a={a=rec {a={};};};};};};};};};};}
+  {
+    a=rec {
+    a={
+    a=rec {
+    a={
+    a=rec {a={a=rec {a={a=rec {a={};};};};};};};};};};}
 
   rec {
 

tests/cases/bin_op/in

@@ -5,5 +5,6 @@
   (1/**/+/**/1)
   (1/**/+/**/(1/**/+/**/(1/**/+/**/1)))
   ( 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 )
-  ( 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1)
+  ( 1
+  + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1 + 1)
 ]

tests/cases/inherit/in

@@ -1,6 +1,8 @@
 [
-  { inherit aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa; }
-  { inherit aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa; }
+  {
+    inherit aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa; }
+  { inherit
+  aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa; }
   { inherit       b       d       ; }
   { inherit       b       d /*e*/ ; }
   { inherit       b /*c*/ d       ; }

tests/cases/lambda/in

@@ -9,6 +9,7 @@
   (
     aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
   )
-  ( aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa )
+  (
+    aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa: aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa )
   ({ pkgs ? import ./.. { }, locationsXml }: null)
 ]

tests/cases/or_default/in

@@ -5,5 +5,6 @@
   (a/**/?/**/a)
   (a/**/?/**/(a/**/?/**/(a/**/?/**/a)))
   ( a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a )
-  ( a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a)
+  ( a ? a
+      ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a ? a)
 ]

tests/cases/select/in

@@ -4,5 +4,6 @@
   (a/**/.    a)
   (a/**/./**/a)
   ( a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a )
-  ( a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a )
+  ( a.a
+     .a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a.a )
 ]

tests/cases/string/out

@@ -58,9 +58,7 @@
     [${mkSectionName sectName}]
   ''
   ###
-  ''-couch_ini ${cfg.package}/etc/default.ini ${configFile} ${
-    pkgs.writeText "couchdb-extra.ini" cfg.extraConfig
-  } ${cfg.configFile}''
+  ''-couch_ini ${cfg.package}/etc/default.ini ${configFile} ${pkgs.writeText "couchdb-extra.ini" cfg.extraConfig} ${cfg.configFile}''
   ###
   ''exec i3-input -F "mark %s" -l 1 -P 'Mark: ' ''
   ###

tests/cases/with/in

@@ -4,5 +4,6 @@
   (with /*a*/ b;       c)
   (with /*a*/ b; /*b*/ c)
   ( with b; cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
-  ( with b; cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
+  ( with b;
+  cccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc )
 ]