I was actually going to post on #5219 to ask you to open a separate issue, so thanks for doing this! I'm definitely open to supporting this with a config option, provided the default is kept the same and someone is willing to submit a PR. It should be a relatively easy one for anyone that's interested, some relevant code sections below:

• Add a new config option (feel free to proposal any name)

  • Probably add to the misc block, along with a test entry
    

    rustfmt/src/config/mod.rs

    Lines 106 to 108 in 5df8c8f

    	// Misc.
    	remove_nested_parens: bool, true, true, "Remove nested parens";
    	combine_control_expr: bool, true, false, "Combine control expressions with function calls";

  rustfmt/src/config/mod.rs

  Lines 590 to 591 in 5df8c8f

  	remove_nested_parens = true
  	combine_control_expr = true

  • Add an entry in the Configuration docs

• Drop the existing constant and replace it's usage with the new config option
  

  rustfmt/src/overflow.rs

  Line 29 in 0b8ffac

  const SHORT_ITEM_THRESHOLD: usize = 10;

  
  

  rustfmt/src/overflow.rs

  Lines 758 to 761 in 0b8ffac

  	fn no_long_items(list: &[ListItem]) -> bool {
  	list.iter()
  	.all(|item| item.inner_as_ref().len() <= SHORT_ITEM_THRESHOLD)
  	}

@calebcartwright I'm new to Rust and would like to help out. Can you assign this to me?

@123vivekr feel free to comment @rustbot claim to have the issue assigned to you!

@rustbot claim