refactor: apply heuristic config changes in lib

Author: calebcartwright

rustfmt-core/rustfmt-lib/src/attr.rs

@@ -231,7 +231,7 @@ impl Rewrite for ast::MetaItem {
                     // 1 = "]"
                     shape.sub_width(1)?,
                     self.span,
-                    context.config.width_heuristics().attr_fn_like_width,
+                    context.config.attr_fn_like_width(),
                     Some(if has_trailing_comma {
                         SeparatorTactic::Always
                     } else {

rustfmt-core/rustfmt-lib/src/chains.rs

@@ -558,7 +558,7 @@ impl<'a> ChainFormatterShared<'a> {
         let one_line_budget = if self.child_count == 1 {
             shape.width
         } else {
-            min(shape.width, context.config.width_heuristics().chain_width)
+            min(shape.width, context.config.chain_width())
         }
         .saturating_sub(almost_total);
 

rustfmt-core/rustfmt-lib/src/expr.rs

@@ -943,22 +943,11 @@ impl<'a> ControlFlow<'a> {
                 || last_line_offsetted(shape.used_width(), &pat_expr_string));
 
         // Try to format if-else on single line.
-        if self.allow_single_line
-            && context
-                .config
-                .width_heuristics()
-                .single_line_if_else_max_width
-                > 0
-        {
+        if self.allow_single_line && context.config.single_line_if_else_max_width() > 0 {
             let trial = self.rewrite_single_line(&pat_expr_string, context, shape.width);
 
             if let Some(cond_str) = trial {
-                if cond_str.len()
-                    <= context
-                        .config
-                        .width_heuristics()
-                        .single_line_if_else_max_width
-                {
+                if cond_str.len() <= context.config.single_line_if_else_max_width() {
                     return Some((cond_str, 0));
                 }
             }
@@ -1295,7 +1284,7 @@ pub(crate) fn rewrite_call(
         args.iter(),
         shape,
         span,
-        context.config.width_heuristics().fn_call_width,
+        context.config.fn_call_width(),
         choose_separator_tactic(context, span),
     )
 }
@@ -1831,7 +1820,7 @@ pub(crate) fn rewrite_tuple<'a, T: 'a + IntoOverflowableItem<'a>>(
             items,
             shape,
             span,
-            context.config.width_heuristics().fn_call_width,
+            context.config.fn_call_width(),
             force_tactic,
         )
     } else {

rustfmt-core/rustfmt-lib/src/items.rs

@@ -566,8 +566,8 @@ impl<'a> FmtVisitor<'a> {
             )
             .collect()
         };
-        let mut items: Vec<_> =
-            itemize_list_with(self.config.width_heuristics().struct_variant_width);
+        let mut items: Vec<_> = itemize_list_with(self.config.struct_variant_width());
+
         // If one of the variants use multiple lines, use multi-lined formatting for all variants.
         let has_multiline_variant = items.iter().any(|item| item.inner_as_ref().contains('\n'));
         let has_single_line_variant = items.iter().any(|item| !item.inner_as_ref().contains('\n'));
@@ -1527,7 +1527,7 @@ fn format_tuple_struct(
             fields.iter(),
             shape,
             span,
-            context.config.width_heuristics().fn_call_width,
+            context.config.fn_call_width(),
             None,
         )?;
     }

rustfmt-core/rustfmt-lib/src/lists.rs

@@ -863,7 +863,7 @@ pub(crate) fn struct_lit_shape(
     };
     let shape_width = shape.width.checked_sub(prefix_width + suffix_width);
     if let Some(w) = shape_width {
-        let shape_width = cmp::min(w, context.config.width_heuristics().struct_lit_width);
+        let shape_width = cmp::min(w, context.config.struct_lit_width());
         Some((Some(Shape::legacy(shape_width, shape.indent)), v_shape))
     } else {
         Some((None, v_shape))

rustfmt-core/rustfmt-lib/src/macros.rs

@@ -368,7 +368,7 @@ fn rewrite_macro_inner(
                     arg_vec.iter(),
                     shape,
                     mac.span(),
-                    context.config.width_heuristics().fn_call_width,
+                    context.config.fn_call_width(),
                     if trailing_comma {
                         Some(SeparatorTactic::Always)
                     } else {

rustfmt-core/rustfmt-lib/src/overflow.rs

@@ -317,7 +317,7 @@ pub(crate) fn rewrite_with_square_brackets<'a, T: 'a + IntoOverflowableItem<'a>>
         span,
         lhs,
         rhs,
-        context.config.width_heuristics().array_width,
+        context.config.array_width(),
         force_separator_tactic,
         Some(("[", "]")),
     )

rustfmt-core/rustfmt-lib/tests/source/chains.rs

@@ -1,4 +1,4 @@
-// rustfmt-use_small_heuristics: Off
+// rustfmt-width_heuristics: Off
 // Test chain formatting.
 
 fn main() {

rustfmt-core/rustfmt-lib/tests/source/configs/use_small_heuristics/max.rs renamed to rustfmt-core/rustfmt-lib/tests/source/configs/width_heuristics/max.rs

@@ -1,4 +1,4 @@
-// rustfmt-use_small_heuristics: Max
+// rustfmt-width_heuristics: Max
 
 enum Lorem {
     Ipsum,

rustfmt-core/rustfmt-lib/tests/target/chains.rs

@@ -1,4 +1,4 @@
-// rustfmt-use_small_heuristics: Off
+// rustfmt-width_heuristics: Off
 // Test chain formatting.
 
 fn main() {

rustfmt-core/rustfmt-lib/tests/target/configs/use_small_heuristics/max.rs renamed to rustfmt-core/rustfmt-lib/tests/target/configs/width_heuristics/max.rs

@@ -1,4 +1,4 @@
-// rustfmt-use_small_heuristics: Max
+// rustfmt-width_heuristics: Max
 
 enum Lorem {
     Ipsum,

rustfmt-core/rustfmt-lib/tests/target/issue_4049.rs

@@ -1,5 +1,5 @@
 // rustfmt-max_width: 110
-// rustfmt-use_small_heuristics: Max
+// rustfmt-width_heuristics: Max
 // rustfmt-hard_tabs: true
 // rustfmt-use_field_init_shorthand: true
 // rustfmt-overflow_delimited_expr: true