squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Split a long chain with a single child

Browse source
This commit is contained in:
topecongiro 2017-05-03 11:22:36 +09:00
parent 3f703fd33d
commit 2c1d896f60
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/chains.rs
View file

@ -176,9 +176,15 @@ pub fn rewrite_chain(expr: &ast::Expr, context: &RewriteContext, shape: Shape) -
.fold(0, |a, b| a + first_line_width(b)) + parent_rewrite.len();
let one_line_len = rewrites.iter().fold(0, |a, r| a + r.len()) + parent_rewrite.len();
let veto_single_line = if one_line_len > context.config.chain_one_line_max - 1 &&
rewrites.len() > 1 {
true
let veto_single_line = if one_line_len > context.config.chain_one_line_max - 1 {
if rewrites.len() > 1 {
true
} else if rewrites.len() == 1 {
let one_line_len = parent_rewrite.len() + first_line_width(&rewrites[0]);
one_line_len > shape.width
} else {
false
}
} else if context.config.take_source_hints && subexpr_list.len() > 1 {
// Look at the source code. Unless all chain elements start on the same
// line, we won't consider putting them on a single line either.