// option. This file may not be copied, modified, or distributed
// except according to those terms.
-use std::cmp::{min, Ordering};
-use std::fmt::Write;
+use std::cmp::min;
+use std::borrow::Cow;
use std::iter::{repeat, ExactSizeIterator};
use syntax::{ast, ptr};
use syntax::codemap::{BytePos, CodeMap, Span};
use syntax::parse::classify;
-use {Indent, Shape, Spanned};
+use spanned::Spanned;
use chains::rewrite_chain;
use codemap::{LineRangeUtils, SpanUtils};
use comment::{combine_strs_with_missing_comments, contains_comment, recover_comment_removed,
use macros::{rewrite_macro, MacroArg, MacroPosition};
use patterns::{can_be_overflowed_pat, TuplePatField};
use rewrite::{Rewrite, RewriteContext};
+use shape::{Indent, Shape};
use string::{rewrite_string, StringFormat};
use types::{can_be_overflowed_type, rewrite_path, PathContext};
-use utils::{binary_search, colon_spaces, contains_skip, extra_offset, first_line_width,
- inner_attributes, last_line_extendable, last_line_width, left_most_sub_expr, mk_sp,
- outer_attributes, paren_overhead, semicolon_for_stmt, stmt_expr,
- trimmed_last_line_width, wrap_str};
+use utils::{colon_spaces, contains_skip, extra_offset, first_line_width, inner_attributes,
+ last_line_extendable, last_line_width, left_most_sub_expr, mk_sp, outer_attributes,
+ paren_overhead, ptr_vec_to_ref_vec, semicolon_for_stmt, stmt_expr,
+ trimmed_last_line_width};
use vertical::rewrite_with_alignment;
use visitor::FmtVisitor;
shape,
false,
),
- ast::ExprKind::Lit(ref l) => match l.node {
- ast::LitKind::Str(_, ast::StrStyle::Cooked) => {
- rewrite_string_lit(context, l.span, shape)
- }
- _ => wrap_str(
- context.snippet(expr.span),
- context.config.max_width(),
- shape,
- ),
- },
+ ast::ExprKind::Lit(ref l) => rewrite_literal(context, l, shape),
ast::ExprKind::Call(ref callee, ref args) => {
let inner_span = mk_sp(callee.span.hi(), expr.span.hi());
- rewrite_call_with_binary_search(
- context,
- &**callee,
- &args.iter().map(|x| &**x).collect::<Vec<_>>()[..],
- inner_span,
- shape,
- )
+ let callee_str = callee.rewrite(context, shape)?;
+ rewrite_call(context, &callee_str, &args, inner_span, shape)
}
ast::ExprKind::Paren(ref subexpr) => rewrite_paren(context, subexpr, shape),
ast::ExprKind::Binary(ref op, ref lhs, ref rhs) => {
"",
context,
shape,
- context.config.binop_sep(),
+ context.config.binop_separator(),
)
}
ast::ExprKind::Unary(ref op, ref subexpr) => rewrite_unary_op(context, op, subexpr, shape),
expr.span,
shape,
),
- ast::ExprKind::Tup(ref items) => rewrite_tuple(
- context,
- &items.iter().map(|x| &**x).collect::<Vec<_>>()[..],
- expr.span,
- shape,
- ),
+ ast::ExprKind::Tup(ref items) => {
+ rewrite_tuple(context, &ptr_vec_to_ref_vec(&items), expr.span, shape)
+ }
ast::ExprKind::If(..) |
ast::ExprKind::IfLet(..) |
ast::ExprKind::ForLoop(..) |
// Rewrite block without trying to put it in a single line.
rw
} else {
- let prefix = try_opt!(block_prefix(context, block, shape));
+ let prefix = block_prefix(context, block, shape)?;
rewrite_block_with_visitor(context, &prefix, block, shape)
}
}
Some(ident) => format!(" {}", ident.node),
None => String::new(),
};
- wrap_str(
- format!("continue{}", id_str),
- context.config.max_width(),
- shape,
- )
+ Some(format!("continue{}", id_str))
}
ast::ExprKind::Break(ref opt_ident, ref opt_expr) => {
let id_str = match *opt_ident {
if let Some(ref expr) = *opt_expr {
rewrite_unary_prefix(context, &format!("break{} ", id_str), &**expr, shape)
} else {
- wrap_str(
- format!("break{}", id_str),
- context.config.max_width(),
- shape,
- )
+ Some(format!("break{}", id_str))
}
}
ast::ExprKind::Yield(ref opt_expr) => if let Some(ref expr) = *opt_expr {
rewrite_unary_prefix(context, "yield ", &**expr, shape)
} else {
- wrap_str("yield".to_string(), context.config.max_width(), shape)
+ Some("yield".to_string())
},
ast::ExprKind::Closure(capture, ref fn_decl, ref body, _) => {
rewrite_closure(capture, fn_decl, body, expr.span, context, shape)
ast::ExprKind::Mac(ref mac) => {
// Failure to rewrite a marco should not imply failure to
// rewrite the expression.
- rewrite_macro(mac, None, context, shape, MacroPosition::Expression).or_else(|| {
- wrap_str(
- context.snippet(expr.span),
- context.config.max_width(),
- shape,
- )
- })
- }
- ast::ExprKind::Ret(None) => {
- wrap_str("return".to_owned(), context.config.max_width(), shape)
+ rewrite_macro(mac, None, context, shape, MacroPosition::Expression)
+ .or_else(|| Some(context.snippet(expr.span)))
}
+ ast::ExprKind::Ret(None) => Some("return".to_owned()),
ast::ExprKind::Ret(Some(ref expr)) => {
rewrite_unary_prefix(context, "return ", &**expr, shape)
}
"",
context,
shape,
- SeparatorPlace::Front,
+ SeparatorPlace::Back,
),
ast::ExprKind::Index(ref expr, ref index) => {
rewrite_index(&**expr, &**index, context, shape)
ast::ExprKind::Range(ref lhs, ref rhs, limits) => {
let delim = match limits {
ast::RangeLimits::HalfOpen => "..",
- ast::RangeLimits::Closed => "...",
+ ast::RangeLimits::Closed => "..=",
};
fn needs_space_before_range(context: &RewriteContext, lhs: &ast::Expr) -> bool {
};
rewrite_unary_suffix(context, &sp_delim, &*lhs, shape)
}
- (None, None) => wrap_str(delim.into(), context.config.max_width(), shape),
+ (None, None) => Some(delim.into()),
}
}
// We do not format these expressions yet, but they should still
// satisfy our width restrictions.
- ast::ExprKind::InPlace(..) | ast::ExprKind::InlineAsm(..) => wrap_str(
- context.snippet(expr.span),
- context.config.max_width(),
- shape,
- ),
+ ast::ExprKind::InPlace(..) | ast::ExprKind::InlineAsm(..) => {
+ Some(context.snippet(expr.span))
+ }
ast::ExprKind::Catch(ref block) => {
if let rw @ Some(_) = rewrite_single_line_block(context, "do catch ", block, shape) {
rw
Some(format!(
"{}{}",
"do catch ",
- try_opt!(block.rewrite(&context, Shape::legacy(budget, shape.indent)))
+ block.rewrite(context, Shape::legacy(budget, shape.indent))?
))
}
}
expr_rw
.and_then(|expr_str| {
- recover_comment_removed(expr_str, expr.span, context, shape)
+ recover_comment_removed(expr_str, expr.span, context)
})
.and_then(|expr_str| {
let attrs = outer_attributes(&expr.attrs);
- let attrs_str = try_opt!(attrs.rewrite(context, shape));
+ let attrs_str = attrs.rewrite(context, shape)?;
let span = mk_sp(
attrs.last().map_or(expr.span.lo(), |attr| attr.span.hi()),
expr.span.lo(),
width: context.budget(lhs_overhead),
..shape
};
- let lhs_result = try_opt!(
- lhs.rewrite(context, lhs_shape)
- .map(|lhs_str| format!("{}{}", prefix, lhs_str))
- );
+ let lhs_result = lhs.rewrite(context, lhs_shape)
+ .map(|lhs_str| format!("{}{}", prefix, lhs_str))?;
// Try to the both lhs and rhs on the same line.
let rhs_orig_result = shape
.map(|first_line| first_line.ends_with('{'))
.unwrap_or(false);
if !rhs_result.contains('\n') || allow_same_line {
- return Some(format!("{}{}{}{}", lhs_result, infix, rhs_result, suffix));
+ let one_line_width = last_line_width(&lhs_result) + infix.len()
+ + first_line_width(&rhs_result) + suffix.len();
+ if one_line_width <= shape.width {
+ return Some(format!("{}{}{}{}", lhs_result, infix, rhs_result, suffix));
+ }
}
}
// We have to use multiple lines.
// Re-evaluate the rhs because we have more space now:
- let mut rhs_shape = try_opt!(match context.config.control_style() {
+ let mut rhs_shape = match context.config.control_style() {
Style::Legacy => shape
- .sub_width(suffix.len() + prefix.len())
- .map(|s| s.visual_indent(prefix.len())),
+ .sub_width(suffix.len() + prefix.len())?
+ .visual_indent(prefix.len()),
Style::Rfc => {
// Try to calculate the initial constraint on the right hand side.
let rhs_overhead = shape.rhs_overhead(context.config);
Shape::indented(shape.indent.block_indent(context.config), context.config)
- .sub_width(rhs_overhead)
+ .sub_width(rhs_overhead)?
}
- });
+ };
let infix = match separator_place {
SeparatorPlace::Back => infix.trim_right(),
SeparatorPlace::Front => infix.trim_left(),
};
if separator_place == SeparatorPlace::Front {
- rhs_shape = try_opt!(rhs_shape.offset_left(infix.len()));
+ rhs_shape = rhs_shape.offset_left(infix.len())?;
}
- let rhs_result = try_opt!(rhs.rewrite(context, rhs_shape));
+ let rhs_result = rhs.rewrite(context, rhs_shape)?;
match separator_place {
SeparatorPlace::Back => Some(format!(
"{}{}\n{}{}{}",
};
let nested_shape = match context.config.array_layout() {
- IndentStyle::Block => try_opt!(
- shape
- .block()
- .block_indent(context.config.tab_spaces())
- .with_max_width(context.config)
- .sub_width(1)
- ),
- IndentStyle::Visual => try_opt!(
- shape
- .visual_indent(bracket_size)
- .sub_width(bracket_size * 2)
- ),
+ IndentStyle::Block => shape
+ .block()
+ .block_indent(context.config.tab_spaces())
+ .with_max_width(context.config)
+ .sub_width(1)?,
+ IndentStyle::Visual => shape
+ .visual_indent(bracket_size)
+ .sub_width(bracket_size * 2)?,
};
let items = itemize_list(
},
};
let ends_with_newline = tactic.ends_with_newline(context.config.array_layout());
- if context.config.array_horizontal_layout_threshold() > 0 &&
- items.len() > context.config.array_horizontal_layout_threshold()
+ if context.config.array_horizontal_layout_threshold() > 0
+ && items.len() > context.config.array_horizontal_layout_threshold()
{
tactic = DefinitiveListTactic::Mixed;
}
preserve_newline: false,
config: context.config,
};
- let list_str = try_opt!(write_list(&items, &fmt));
+ let list_str = write_list(&items, &fmt)?;
- let result = if context.config.array_layout() == IndentStyle::Visual ||
- tactic == DefinitiveListTactic::Horizontal
+ let result = if context.config.array_layout() == IndentStyle::Visual
+ || tactic == DefinitiveListTactic::Horizontal
{
if context.config.spaces_within_square_brackets() && !list_str.is_empty() {
format!("[ {} ]", list_str)
};
// 4 = "|| {".len(), which is overconservative when the closure consists of
// a single expression.
- let nested_shape = try_opt!(try_opt!(shape.shrink_left(mover.len())).sub_width(4));
+ let nested_shape = shape.shrink_left(mover.len())?.sub_width(4)?;
// 1 = |
let argument_offset = nested_shape.indent + 1;
- let arg_shape = try_opt!(nested_shape.offset_left(1)).visual_indent(0);
- let ret_str = try_opt!(fn_decl.output.rewrite(context, arg_shape));
+ let arg_shape = nested_shape.offset_left(1)?.visual_indent(0);
+ let ret_str = fn_decl.output.rewrite(context, arg_shape)?;
let arg_items = itemize_list(
context.codemap,
horizontal_budget,
);
let arg_shape = match tactic {
- DefinitiveListTactic::Horizontal => try_opt!(arg_shape.sub_width(ret_str.len() + 1)),
+ DefinitiveListTactic::Horizontal => arg_shape.sub_width(ret_str.len() + 1)?,
_ => arg_shape,
};
preserve_newline: true,
config: context.config,
};
- let list_str = try_opt!(write_list(&item_vec, &fmt));
+ let list_str = write_list(&item_vec, &fmt)?;
let mut prefix = format!("{}|{}|", mover, list_str);
- // 1 = space between `|...|` and body.
- let extra_offset = extra_offset(&prefix, shape) + 1;
if !ret_str.is_empty() {
if prefix.contains('\n') {
}
prefix.push_str(&ret_str);
}
+ // 1 = space between `|...|` and body.
+ let extra_offset = last_line_width(&prefix) + 1;
Some((prefix, extra_offset))
}
context: &RewriteContext,
shape: Shape,
) -> Option<String> {
- let (prefix, extra_offset) = try_opt!(rewrite_closure_fn_decl(
- capture,
- fn_decl,
- body,
- span,
- context,
- shape,
- ));
+ let (prefix, extra_offset) =
+ rewrite_closure_fn_decl(capture, fn_decl, body, span, context, shape)?;
// 1 = space between `|...|` and body.
- let body_shape = try_opt!(shape.offset_left(extra_offset));
+ let body_shape = shape.offset_left(extra_offset)?;
if let ast::ExprKind::Block(ref block) = body.node {
// The body of the closure is an empty block.
}
// Figure out if the block is necessary.
- let needs_block = block.rules != ast::BlockCheckMode::Default || block.stmts.len() > 1 ||
- context.inside_macro ||
- block_contains_comment(block, context.codemap) ||
- prefix.contains('\n');
+ let needs_block = block.rules != ast::BlockCheckMode::Default || block.stmts.len() > 1
+ || context.inside_macro
+ || block_contains_comment(block, context.codemap)
+ || prefix.contains('\n');
let no_return_type = if let ast::FunctionRetTy::Default(_) = fn_decl.output {
true
let block_threshold = context.config.closure_block_indent_threshold();
if block_threshold >= 0 {
if let Some(block_str) = block.rewrite(context, shape) {
- if block_str.matches('\n').count() <= block_threshold as usize &&
- !need_block_indent(&block_str, shape)
+ if block_str.matches('\n').count() <= block_threshold as usize
+ && !need_block_indent(&block_str, shape)
{
- if let Some(block_str) = block_str.rewrite(context, shape) {
- return Some(format!("{} {}", prefix, block_str));
- }
+ return Some(format!("{} {}", prefix, block_str));
}
}
}
// The body of the closure is big enough to be block indented, that
// means we must re-format.
let block_shape = shape.block();
- let block_str = try_opt!(block.rewrite(context, block_shape));
+ let block_str = block.rewrite(context, block_shape)?;
Some(format!("{} {}", prefix, block_str))
}
fn nop_block_collapse(block_str: Option<String>, budget: usize) -> Option<String> {
debug!("nop_block_collapse {:?} {}", block_str, budget);
block_str.map(|block_str| {
- if block_str.starts_with('{') && budget >= 2 &&
- (block_str[1..].find(|c: char| !c.is_whitespace()).unwrap() == block_str.len() - 2)
+ if block_str.starts_with('{') && budget >= 2
+ && (block_str[1..].find(|c: char| !c.is_whitespace()).unwrap() == block_str.len() - 2)
{
"{}".to_owned()
} else {
let user_str = user_str.trim();
if user_str.starts_with('{') && user_str.ends_with('}') {
let comment_str = user_str[1..user_str.len() - 1].trim();
- if block.stmts.is_empty() && !comment_str.contains('\n') &&
- !comment_str.starts_with("//") && comment_str.len() + 4 <= shape.width
+ if block.stmts.is_empty() && !comment_str.contains('\n') && !comment_str.starts_with("//")
+ && comment_str.len() + 4 <= shape.width
{
return Some(format!("{{ {} }}", comment_str));
}
Some(match block.rules {
ast::BlockCheckMode::Unsafe(..) => {
let snippet = context.snippet(block.span);
- let open_pos = try_opt!(snippet.find_uncommented("{"));
+ let open_pos = snippet.find_uncommented("{")?;
// Extract comment between unsafe and block start.
let trimmed = &snippet[6..open_pos].trim();
if !trimmed.is_empty() {
// 9 = "unsafe {".len(), 7 = "unsafe ".len()
- let budget = try_opt!(shape.width.checked_sub(9));
+ let budget = shape.width.checked_sub(9)?;
format!(
"unsafe {} ",
- try_opt!(rewrite_comment(
+ rewrite_comment(
trimmed,
true,
Shape::legacy(budget, shape.indent + 7),
context.config,
- ))
+ )?
)
} else {
"unsafe ".to_owned()
) -> Option<String> {
if is_simple_block(block, context.codemap) {
let expr_shape = Shape::legacy(shape.width - prefix.len(), shape.indent);
- let expr_str = try_opt!(block.stmts[0].rewrite(context, expr_shape));
+ let expr_str = block.stmts[0].rewrite(context, expr_shape)?;
let result = format!("{}{{ {} }}", prefix, expr_str);
if result.len() <= shape.width && !result.contains('\n') {
return Some(result);
match block.rules {
ast::BlockCheckMode::Unsafe(..) => {
let snippet = context.snippet(block.span);
- let open_pos = try_opt!(snippet.find_uncommented("{"));
+ let open_pos = snippet.find_uncommented("{")?;
visitor.last_pos = block.span.lo() + BytePos(open_pos as u32)
}
ast::BlockCheckMode::Default => visitor.last_pos = block.span.lo(),
return rw;
}
- let prefix = try_opt!(block_prefix(context, self, shape));
+ let prefix = block_prefix(context, self, shape)?;
let result = rewrite_block_with_visitor(context, &prefix, self, shape);
if let Some(ref result_str) = result {
""
};
- let shape = try_opt!(shape.sub_width(suffix.len()));
+ let shape = shape.sub_width(suffix.len())?;
format_expr(ex, ExprType::Statement, context, shape).map(|s| s + suffix)
}
ast::StmtKind::Mac(..) | ast::StmtKind::Item(..) => None,
};
- result.and_then(|res| {
- recover_comment_removed(res, self.span(), context, shape)
- })
+ result.and_then(|res| recover_comment_removed(res, self.span(), context))
}
}
ast::ExprKind::Match(ref cond, _) => {
// `match `cond` {`
let cond_shape = match context.config.control_style() {
- Style::Legacy => try_opt!(shape.shrink_left(6).and_then(|s| s.sub_width(2))),
- Style::Rfc => try_opt!(shape.offset_left(8)),
+ Style::Legacy => shape.shrink_left(6).and_then(|s| s.sub_width(2))?,
+ Style::Rfc => shape.offset_left(8)?,
};
cond.rewrite(context, cond_shape)
}
width: usize,
) -> Option<String> {
assert!(self.allow_single_line);
- let else_block = try_opt!(self.else_block);
+ let else_block = self.else_block?;
let fixed_cost = self.keyword.len() + " { } else { }".len();
if let ast::ExprKind::Block(ref else_node) = else_block.node {
- if !is_simple_block(self.block, context.codemap) ||
- !is_simple_block(else_node, context.codemap) ||
- pat_expr_str.contains('\n')
+ if !is_simple_block(self.block, context.codemap)
+ || !is_simple_block(else_node, context.codemap)
+ || pat_expr_str.contains('\n')
{
return None;
}
- let new_width = try_opt!(width.checked_sub(pat_expr_str.len() + fixed_cost));
+ let new_width = width.checked_sub(pat_expr_str.len() + fixed_cost)?;
let expr = &self.block.stmts[0];
- let if_str = try_opt!(expr.rewrite(context, Shape::legacy(new_width, Indent::empty())));
+ let if_str = expr.rewrite(context, Shape::legacy(new_width, Indent::empty()))?;
- let new_width = try_opt!(new_width.checked_sub(if_str.len()));
+ let new_width = new_width.checked_sub(if_str.len())?;
let else_expr = &else_node.stmts[0];
- let else_str =
- try_opt!(else_expr.rewrite(context, Shape::legacy(new_width, Indent::empty())));
+ let else_str = else_expr.rewrite(context, Shape::legacy(new_width, Indent::empty()))?;
if if_str.contains('\n') || else_str.contains('\n') {
return None;
let constr_shape = if self.nested_if {
// We are part of an if-elseif-else chain. Our constraints are tightened.
// 7 = "} else " .len()
- try_opt!(fresh_shape.offset_left(7))
+ fresh_shape.offset_left(7)?
} else {
fresh_shape
};
let pat_expr_string = match self.cond {
Some(cond) => {
let cond_shape = match context.config.control_style() {
- Style::Legacy => try_opt!(constr_shape.shrink_left(offset)),
- Style::Rfc => try_opt!(constr_shape.offset_left(offset)),
+ Style::Legacy => constr_shape.shrink_left(offset)?,
+ Style::Rfc => constr_shape.offset_left(offset)?,
};
- try_opt!(rewrite_pat_expr(
+ rewrite_pat_expr(
context,
self.pat,
cond,
self.connector,
self.keyword,
cond_shape,
- ))
+ )?
}
None => String::new(),
};
.max_width()
.checked_sub(constr_shape.used_width() + offset + brace_overhead)
.unwrap_or(0);
- let force_newline_brace = context.config.control_style() == Style::Rfc &&
- (pat_expr_string.contains('\n') || pat_expr_string.len() > one_line_budget) &&
- !last_line_extendable(&pat_expr_string);
+ let force_newline_brace = context.config.control_style() == Style::Rfc
+ && (pat_expr_string.contains('\n') || pat_expr_string.len() > one_line_budget)
+ && !last_line_extendable(&pat_expr_string);
// Try to format if-else on single line.
if self.allow_single_line && context.config.single_line_if_else_max_width() > 0 {
let block_sep = if self.cond.is_none() && between_kwd_cond_comment.is_some() {
""
- } else if context.config.control_brace_style() == ControlBraceStyle::AlwaysNextLine ||
- force_newline_brace
+ } else if context.config.control_brace_style() == ControlBraceStyle::AlwaysNextLine
+ || force_newline_brace
{
alt_block_sep
} else {
let alt_block_sep =
String::from("\n") + &shape.indent.block_only().to_string(context.config);
- let (cond_str, used_width) = try_opt!(self.rewrite_cond(context, shape, &alt_block_sep));
+ let (cond_str, used_width) = self.rewrite_cond(context, shape, &alt_block_sep)?;
// If `used_width` is 0, it indicates that whole control flow is written in a single line.
if used_width == 0 {
return Some(cond_str);
};
let mut block_context = context.clone();
block_context.is_if_else_block = self.else_block.is_some();
- let block_str = try_opt!(rewrite_block_with_visitor(
- &block_context,
- "",
- self.block,
- block_shape,
- ));
+ let block_str = rewrite_block_with_visitor(&block_context, "", self.block, block_shape)?;
let mut result = format!("{}{}", cond_str, block_str);
ControlBraceStyle::AlwaysNextLine if last_in_chain => &*alt_block_sep,
_ => " ",
};
- try_opt!(
- write!(
- &mut result,
- "{}else{}",
- between_kwd_else_block_comment
- .as_ref()
- .map_or(between_sep, |s| &**s),
- after_else_comment.as_ref().map_or(after_sep, |s| &**s)
- ).ok()
- );
- result.push_str(&try_opt!(rewrite));
+
+ result.push_str(&format!(
+ "{}else{}",
+ between_kwd_else_block_comment
+ .as_ref()
+ .map_or(between_sep, |s| &**s),
+ after_else_comment.as_ref().map_or(after_sep, |s| &**s),
+ ));
+ result.push_str(&rewrite?);
}
Some(result)
}
}
-fn rewrite_label(label: Option<ast::SpannedIdent>) -> String {
+fn rewrite_label(label: Option<ast::SpannedIdent>) -> Cow<'static, str> {
match label {
- Some(ident) => format!("{}: ", ident.node),
- None => "".to_owned(),
+ Some(ident) => Cow::from(format!("{}: ", ident.node)),
+ None => Cow::from(""),
}
}
// FIXME: incorrectly returns false when comment is contained completely within
// the expression.
pub fn is_simple_block(block: &ast::Block, codemap: &CodeMap) -> bool {
- (block.stmts.len() == 1 && stmt_is_expr(&block.stmts[0]) &&
- !block_contains_comment(block, codemap))
+ (block.stmts.len() == 1 && stmt_is_expr(&block.stmts[0])
+ && !block_contains_comment(block, codemap))
}
/// Checks whether a block contains at most one statement or expression, and no comments.
// Do not take the rhs overhead from the upper expressions into account
// when rewriting match condition.
- let new_width = try_opt!(context.config.max_width().checked_sub(shape.used_width()));
+ let new_width = context.config.max_width().checked_sub(shape.used_width())?;
let cond_shape = Shape {
width: new_width,
..shape
};
// 6 = `match `
let cond_shape = match context.config.control_style() {
- Style::Legacy => try_opt!(cond_shape.shrink_left(6)),
- Style::Rfc => try_opt!(cond_shape.offset_left(6)),
+ Style::Legacy => cond_shape.shrink_left(6)?,
+ Style::Rfc => cond_shape.offset_left(6)?,
};
- let cond_str = try_opt!(cond.rewrite(context, cond_shape));
+ let cond_str = cond.rewrite(context, cond_shape)?;
let alt_block_sep = String::from("\n") + &shape.indent.block_only().to_string(context.config);
let block_sep = match context.config.control_brace_style() {
ControlBraceStyle::AlwaysNextLine => &alt_block_sep,
let inner_attrs_str = if inner_attrs.is_empty() {
String::new()
} else {
- try_opt!(
- inner_attrs
- .rewrite(context, shape)
- .map(|s| format!("{}{}\n", nested_indent_str, s))
- )
+ inner_attrs
+ .rewrite(context, shape)
+ .map(|s| format!("{}{}\n", nested_indent_str, s))?
};
let open_brace_pos = if inner_attrs.is_empty() {
block_sep,
inner_attrs_str,
arm_indent_str,
- try_opt!(rewrite_match_arms(
- context,
- arms,
- shape,
- span,
- open_brace_pos,
- )),
+ rewrite_match_arms(context, arms, shape, span, open_brace_pos,)?,
shape.indent.to_string(context.config),
))
}
arm.attrs[arm.attrs.len() - 1].span.hi(),
arm.pats[0].span.lo(),
),
- try_opt!(arm.attrs.rewrite(context, shape)),
+ arm.attrs.rewrite(context, shape)?,
)
} else {
(mk_sp(arm.span().lo(), arm.span().lo()), String::new())
};
- let pats_str = try_opt!(
+ let pats_str =
rewrite_match_pattern(context, &arm.pats, &arm.guard, shape).and_then(|pats_str| {
combine_strs_with_missing_comments(
context,
shape,
false,
)
- })
- );
+ })?;
rewrite_match_body(
context,
&arm.body,
) -> Option<String> {
// Patterns
// 5 = ` => {`
- let pat_shape = try_opt!(shape.sub_width(5));
+ let pat_shape = shape.sub_width(5)?;
- let pat_strs = try_opt!(
- pats.iter()
- .map(|p| p.rewrite(context, pat_shape))
- .collect::<Option<Vec<_>>>()
- );
+ let pat_strs = pats.iter()
+ .map(|p| p.rewrite(context, pat_shape))
+ .collect::<Option<Vec<_>>>()?;
let items: Vec<_> = pat_strs.into_iter().map(ListItem::from_str).collect();
let tactic = definitive_tactic(
preserve_newline: false,
config: context.config,
};
- let pats_str = try_opt!(write_list(&items, &fmt));
+ let pats_str = write_list(&items, &fmt)?;
// Guard
- let guard_str = try_opt!(rewrite_guard(
- context,
- guard,
- shape,
- trimmed_last_line_width(&pats_str),
- ));
+ let guard_str = rewrite_guard(context, guard, shape, trimmed_last_line_width(&pats_str))?;
Some(format!("{}{}", pats_str, guard_str))
}
{
if let ast::StmtKind::Expr(ref expr) = block.stmts[0].node {
(
- !context.config.multiline_match_arm_forces_block() &&
- expr.can_be_overflowed(context, 1),
+ !context.config.multiline_match_arm_forces_block()
+ && expr.can_be_overflowed(context, 1),
&**expr,
)
} else {
}
}
_ => (
- !context.config.multiline_match_arm_forces_block() &&
- body.can_be_overflowed(context, 1),
+ !context.config.multiline_match_arm_forces_block()
+ && body.can_be_overflowed(context, 1),
&*body,
),
}
is_last: bool,
) -> Option<String> {
let (extend, body) = flatten_arm_body(context, body);
-
- let comma = arm_comma(context.config, body, is_last);
- let alt_block_sep = String::from("\n") + &shape.indent.block_only().to_string(context.config);
- let alt_block_sep = alt_block_sep.as_str();
let (is_block, is_empty_block) = if let ast::ExprKind::Block(ref block) = body.node {
(true, is_empty_block(block, context.codemap))
} else {
(false, false)
};
+ let extend = if context.config.match_arm_forces_newline() {
+ is_block
+ } else {
+ extend
+ };
+
+ let comma = arm_comma(context.config, body, is_last);
+ let alt_block_sep = String::from("\n") + &shape.indent.block_only().to_string(context.config);
+ let alt_block_sep = alt_block_sep.as_str();
let combine_orig_body = |body_str: &str| {
let block_sep = match context.config.control_brace_style() {
let forbid_same_line = has_guard && pats_str.contains('\n') && !is_empty_block;
let next_line_indent = if is_block {
- shape.indent
+ if is_empty_block {
+ shape.indent.block_indent(context.config)
+ } else {
+ shape.indent
+ }
} else {
shape.indent.block_indent(context.config)
};
match rewrite {
Some(ref body_str)
- if !forbid_same_line &&
- (is_block ||
- (!body_str.contains('\n') && body_str.len() <= body_shape.width)) =>
+ if !forbid_same_line && !context.config.match_arm_forces_newline()
+ && (is_block
+ || (!body_str.contains('\n') && body_str.len() <= body_shape.width)) =>
{
return combine_orig_body(body_str);
}
} else {
format!("{} ", matcher)
};
- let pat_shape =
- try_opt!(try_opt!(shape.offset_left(matcher.len())).sub_width(connector.len()));
- let pat_string = try_opt!(pat.rewrite(context, pat_shape));
+ let pat_shape = shape.offset_left(matcher.len())?.sub_width(connector.len())?;
+ let pat_string = pat.rewrite(context, pat_shape)?;
let result = format!("{}{}{}", matcher, pat_string, connector);
return rewrite_assign_rhs(context, result, expr, shape);
}
.map(|expr_rw| format!("\n{}{}", nested_indent_str, expr_rw))
}
+pub fn rewrite_literal(context: &RewriteContext, l: &ast::Lit, shape: Shape) -> Option<String> {
+ match l.node {
+ ast::LitKind::Str(_, ast::StrStyle::Cooked) => rewrite_string_lit(context, l.span, shape),
+ _ => Some(context.snippet(l.span)),
+ }
+}
+
fn rewrite_string_lit(context: &RewriteContext, span: Span, shape: Shape) -> Option<String> {
let string_lit = context.snippet(span);
string_lit
.lines()
.map(|line| {
- new_indent.to_string(context.config) + line.trim_left()
+ format!(
+ "{}{}",
+ new_indent.to_string(context.config),
+ line.trim_left()
+ )
})
.collect::<Vec<_>>()
.join("\n")
}
}
- if !context.config.force_format_strings() &&
- !string_requires_rewrite(context, span, &string_lit, shape)
+ if !context.config.force_format_strings()
+ && !string_requires_rewrite(context, span, &string_lit, shape)
{
return Some(string_lit);
}
- let fmt = StringFormat {
- opener: "\"",
- closer: "\"",
- line_start: " ",
- line_end: "\\",
- shape: shape,
- trim_end: false,
- config: context.config,
- };
-
// Remove the quote characters.
let str_lit = &string_lit[1..string_lit.len() - 1];
- rewrite_string(str_lit, &fmt)
+ rewrite_string(str_lit, &StringFormat::new(shape, context.config))
}
fn string_requires_rewrite(
false
}
-pub fn rewrite_call_with_binary_search<R>(
- context: &RewriteContext,
- callee: &R,
- args: &[&ast::Expr],
- span: Span,
- shape: Shape,
-) -> Option<String>
-where
- R: Rewrite,
-{
- let force_trailing_comma = if context.inside_macro {
- span_ends_with_comma(context, span)
- } else {
- false
- };
- let closure = |callee_max_width| {
- // FIXME using byte lens instead of char lens (and probably all over the
- // place too)
- let callee_shape = Shape {
- width: callee_max_width,
- ..shape
- };
- let callee_str = callee
- .rewrite(context, callee_shape)
- .ok_or(Ordering::Greater)?;
-
- rewrite_call_inner(
- context,
- &callee_str,
- args,
- span,
- shape,
- context.config.fn_call_width(),
- force_trailing_comma,
- )
- };
-
- binary_search(1, shape.width, closure)
-}
-
pub fn rewrite_call(
context: &RewriteContext,
callee: &str,
rewrite_call_inner(
context,
callee,
- &args.iter().map(|x| &**x).collect::<Vec<_>>(),
+ &ptr_vec_to_ref_vec(&args),
span,
shape,
context.config.fn_call_width(),
force_trailing_comma,
- ).ok()
+ )
}
pub fn rewrite_call_inner<'a, T>(
shape: Shape,
args_max_width: usize,
force_trailing_comma: bool,
-) -> Result<String, Ordering>
+) -> Option<String>
where
T: Rewrite + Spanned + ToExpr + 'a,
{
1
};
let used_width = extra_offset(callee_str, shape);
- let one_line_width = shape
- .width
- .checked_sub(used_width + 2 * paren_overhead)
- .ok_or(Ordering::Greater)?;
+ let one_line_width = shape.width.checked_sub(used_width + 2 * paren_overhead)?;
let nested_shape = shape_from_fn_call_style(
context,
shape,
used_width + 2 * paren_overhead,
used_width + paren_overhead,
- ).ok_or(Ordering::Greater)?;
+ )?;
let span_lo = context.codemap.span_after(span, "(");
let args_span = mk_sp(span_lo, span.hi());
one_line_width,
args_max_width,
force_trailing_comma,
- ).ok_or(Ordering::Less)?;
+ )?;
if !context.use_block_indent() && need_block_indent(&list_str, nested_shape) && !extendable {
let mut new_context = context.clone();
);
}
- let args_shape = shape
- .sub_width(last_line_width(callee_str))
- .ok_or(Ordering::Less)?;
- Ok(format!(
+ let args_shape = shape.sub_width(last_line_width(callee_str))?;
+ Some(format!(
"{}{}",
callee_str,
wrap_args_with_parens(context, &list_str, extendable, args_shape, nested_shape)
_ if args.len() >= 1 => {
item_vec[args.len() - 1].item = args.last()
.and_then(|last_arg| last_arg.rewrite(context, shape));
- tactic = definitive_tactic(
- &*item_vec,
- ListTactic::LimitedHorizontalVertical(args_max_width),
- Separator::Comma,
- one_line_width,
- );
+ // Use horizontal layout for a function with a single argument as long as
+ // everything fits in a single line.
+ if args.len() == 1
+ && args_max_width != 0 // Vertical layout is forced.
+ && !item_vec[0].has_comment()
+ && !item_vec[0].inner_as_ref().contains('\n')
+ && ::lists::total_item_width(&item_vec[0]) <= one_line_width
+ {
+ tactic = DefinitiveListTactic::Horizontal;
+ } else {
+ tactic = definitive_tactic(
+ &*item_vec,
+ ListTactic::LimitedHorizontalVertical(args_max_width),
+ Separator::Comma,
+ one_line_width,
+ );
+ }
}
_ => (),
}
shape.block().indent
};
Some(Shape {
- width: try_opt!(max_width.checked_sub(overhead)),
+ width: max_width.checked_sub(overhead)?,
indent: arg_indent,
offset: 0,
})
}
_ => body,
};
- let (prefix, extra_offset) = try_opt!(rewrite_closure_fn_decl(
- capture,
- fn_decl,
- body,
- expr.span,
- context,
- shape,
- ));
+ let (prefix, extra_offset) =
+ rewrite_closure_fn_decl(capture, fn_decl, body, expr.span, context, shape)?;
// If the closure goes multi line before its body, do not overflow the closure.
if prefix.contains('\n') {
return None;
}
- let body_shape = try_opt!(shape.offset_left(extra_offset));
+ let body_shape = shape.offset_left(extra_offset)?;
// When overflowing the closure which consists of a single control flow expression,
// force to use block if its condition uses multi line.
let is_multi_lined_cond = rewrite_cond(context, body, body_shape)
ast::ExprKind::Closure(..) => {
// If the argument consists of multiple closures, we do not overflow
// the last closure.
- if args.len() > 1 &&
- args.iter()
+ if args.len() > 1
+ && args.iter()
.rev()
.skip(1)
.filter_map(|arg| arg.to_expr())
pub fn can_be_overflowed_expr(context: &RewriteContext, expr: &ast::Expr, args_len: usize) -> bool {
match expr.node {
ast::ExprKind::Match(..) => {
- (context.use_block_indent() && args_len == 1) ||
- (context.config.fn_call_style() == IndentStyle::Visual && args_len > 1)
+ (context.use_block_indent() && args_len == 1)
+ || (context.config.fn_call_style() == IndentStyle::Visual && args_len > 1)
}
ast::ExprKind::If(..) |
ast::ExprKind::IfLet(..) |
context.config.combine_control_expr() && context.use_block_indent() && args_len == 1
}
ast::ExprKind::Block(..) | ast::ExprKind::Closure(..) => {
- context.use_block_indent() ||
- context.config.fn_call_style() == IndentStyle::Visual && args_len > 1
+ context.use_block_indent()
+ || context.config.fn_call_style() == IndentStyle::Visual && args_len > 1
}
ast::ExprKind::Array(..) |
ast::ExprKind::Call(..) |
shape: Shape,
nested_shape: Shape,
) -> String {
- if !context.use_block_indent() ||
- (context.inside_macro && !args_str.contains('\n') &&
- args_str.len() + paren_overhead(context) <= shape.width) || is_extendable
+ if !context.use_block_indent()
+ || (context.inside_macro && !args_str.contains('\n')
+ && args_str.len() + paren_overhead(context) <= shape.width) || is_extendable
{
if context.config.spaces_within_parens() && !args_str.is_empty() {
format!("( {} )", args_str)
debug!("rewrite_paren, shape: {:?}", shape);
let total_paren_overhead = paren_overhead(context);
let paren_overhead = total_paren_overhead / 2;
- let sub_shape = try_opt!(
- shape
- .offset_left(paren_overhead)
- .and_then(|s| s.sub_width(paren_overhead))
- );
+ let sub_shape = shape
+ .offset_left(paren_overhead)
+ .and_then(|s| s.sub_width(paren_overhead))?;
let paren_wrapper = |s: &str| if context.config.spaces_within_parens() && !s.is_empty() {
format!("( {} )", s)
format!("({})", s)
};
- let subexpr_str = try_opt!(subexpr.rewrite(context, sub_shape));
+ let subexpr_str = subexpr.rewrite(context, sub_shape)?;
debug!("rewrite_paren, subexpr_str: `{:?}`", subexpr_str);
- if subexpr_str.contains('\n') ||
- first_line_width(&subexpr_str) + total_paren_overhead <= shape.width
+ if subexpr_str.contains('\n')
+ || first_line_width(&subexpr_str) + total_paren_overhead <= shape.width
{
Some(paren_wrapper(&subexpr_str))
} else {
context: &RewriteContext,
shape: Shape,
) -> Option<String> {
- let expr_str = try_opt!(expr.rewrite(context, shape));
+ let expr_str = expr.rewrite(context, shape)?;
let (lbr, rbr) = if context.config.spaces_within_square_brackets() {
("[ ", " ]")
// Try putting index on the next line and see if it fits in a single line.
let indent = shape.indent.block_indent(context.config);
- let index_shape = try_opt!(Shape::indented(indent, context.config).offset_left(lbr.len()));
- let index_shape = try_opt!(index_shape.sub_width(rbr.len() + rhs_overhead));
+ let index_shape = Shape::indented(indent, context.config).offset_left(lbr.len())?;
+ let index_shape = index_shape.sub_width(rbr.len() + rhs_overhead)?;
let new_index_rw = index.rewrite(context, index_shape);
match (orig_index_rw, new_index_rw) {
(_, Some(ref new_index_str)) if !new_index_str.contains('\n') => Some(format!(
}
// 2 = " {".len()
- let path_shape = try_opt!(shape.sub_width(2));
- let path_str = try_opt!(rewrite_path(
- context,
- PathContext::Expr,
- None,
- path,
- path_shape,
- ));
+ let path_shape = shape.sub_width(2)?;
+ let path_str = rewrite_path(context, PathContext::Expr, None, path, path_shape)?;
if fields.is_empty() && base.is_none() {
return Some(format!("{} {{}}", path_str));
}
// Foo { a: Foo } - indent is +3, width is -5.
- let (h_shape, v_shape) = try_opt!(struct_lit_shape(shape, context, path_str.len() + 3, 2));
+ let (h_shape, v_shape) = struct_lit_shape(shape, context, path_str.len() + 3, 2)?;
let one_line_width = h_shape.map_or(0, |shape| shape.width);
let body_lo = context.codemap.span_after(span, "{");
- let fields_str = if struct_lit_can_be_aligned(fields, &base) &&
- context.config.struct_field_align_threshold() > 0
+ let fields_str = if struct_lit_can_be_aligned(fields, &base)
+ && context.config.struct_field_align_threshold() > 0
{
- try_opt!(rewrite_with_alignment(
+ rewrite_with_alignment(
fields,
context,
shape,
mk_sp(body_lo, span.hi()),
one_line_width,
- ))
+ )?
} else {
let field_iter = fields
.into_iter()
let rewrite = |item: &StructLitField| match *item {
StructLitField::Regular(field) => {
// The 1 taken from the v_budget is for the comma.
- rewrite_field(context, field, try_opt!(v_shape.sub_width(1)), 0)
+ rewrite_field(context, field, v_shape.sub_width(1)?, 0)
}
StructLitField::Base(expr) => {
// 2 = ..
- expr.rewrite(context, try_opt!(v_shape.offset_left(2)))
+ expr.rewrite(context, v_shape.offset_left(2)?)
.map(|s| format!("..{}", s))
}
};
let nested_shape = shape_for_tactic(tactic, h_shape, v_shape);
let fmt = struct_lit_formatting(nested_shape, tactic, context, base.is_some());
- try_opt!(write_list(&item_vec, &fmt))
+ write_list(&item_vec, &fmt)?
};
let fields_str = wrap_struct_field(context, &fields_str, shape, v_shape, one_line_width);
nested_shape: Shape,
one_line_width: usize,
) -> String {
- if context.config.struct_lit_style() == IndentStyle::Block &&
- (fields_str.contains('\n') ||
- context.config.struct_lit_multiline_style() == MultilineStyle::ForceMulti ||
- fields_str.len() > one_line_width)
+ if context.config.struct_lit_style() == IndentStyle::Block
+ && (fields_str.contains('\n')
+ || context.config.struct_lit_multiline_style() == MultilineStyle::ForceMulti
+ || fields_str.len() > one_line_width)
{
format!(
"\n{}{}\n{}",
prefix_max_width: usize,
) -> Option<String> {
if contains_skip(&field.attrs) {
- return wrap_str(
- context.snippet(field.span()),
- context.config.max_width(),
- shape,
- );
+ return Some(context.snippet(field.span()));
}
let name = &field.ident.node.to_string();
if field.is_shorthand {
separator.push(' ');
}
let overhead = name.len() + separator.len();
- let expr_shape = try_opt!(shape.offset_left(overhead));
+ let expr_shape = shape.offset_left(overhead)?;
let expr = field.expr.rewrite(context, expr_shape);
- let mut attrs_str = try_opt!(field.attrs.rewrite(context, shape));
+ let mut attrs_str = field.attrs.rewrite(context, shape)?;
if !attrs_str.is_empty() {
attrs_str.push_str(&format!("\n{}", shape.indent.to_string(context.config)));
};
debug!("rewrite_tuple_in_visual_indent_style {:?}", shape);
if items.len() == 1 {
// 3 = "(" + ",)"
- let nested_shape = try_opt!(shape.sub_width(3)).visual_indent(1);
+ let nested_shape = shape.sub_width(3)?.visual_indent(1);
return items.next().unwrap().rewrite(context, nested_shape).map(
|s| if context.config.spaces_within_parens() {
format!("( {}, )", s)
}
let list_lo = context.codemap.span_after(span, "(");
- let nested_shape = try_opt!(shape.sub_width(2)).visual_indent(1);
+ let nested_shape = shape.sub_width(2)?.visual_indent(1);
let items = itemize_list(
context.codemap,
items,
preserve_newline: false,
config: context.config,
};
- let list_str = try_opt!(write_list(&item_vec, &fmt));
+ let list_str = write_list(&item_vec, &fmt)?;
if context.config.spaces_within_parens() && !list_str.is_empty() {
Some(format!("( {} )", list_str))
shape,
context.config.fn_call_width(),
force_trailing_comma,
- ).ok()
+ )
} else {
rewrite_tuple_in_visual_indent_style(context, items, span, shape)
}
shape: Shape,
) -> Option<String> {
rewrite
- .rewrite(context, try_opt!(shape.offset_left(prefix.len())))
+ .rewrite(context, shape.offset_left(prefix.len())?)
.map(|r| format!("{}{}", prefix, r))
}
shape: Shape,
) -> Option<String> {
rewrite
- .rewrite(context, try_opt!(shape.sub_width(suffix.len())))
+ .rewrite(context, shape.sub_width(suffix.len())?)
.map(|mut r| {
r.push_str(suffix);
r
};
// 1 = space between lhs and operator.
- let lhs_shape = try_opt!(shape.sub_width(operator_str.len() + 1));
- let lhs_str = format!(
- "{} {}",
- try_opt!(lhs.rewrite(context, lhs_shape)),
- operator_str
- );
+ let lhs_shape = shape.sub_width(operator_str.len() + 1)?;
+ let lhs_str = format!("{} {}", lhs.rewrite(context, lhs_shape)?, operator_str);
rewrite_assign_rhs(context, lhs_str, rhs, shape)
}
0
};
// 1 = space between operator and rhs.
- let orig_shape = try_opt!(shape.offset_left(last_line_width + 1));
- let rhs = try_opt!(choose_rhs(
- context,
- ex,
- shape,
- ex.rewrite(context, orig_shape)
- ));
+ let orig_shape = shape.offset_left(last_line_width + 1)?;
+ let rhs = choose_rhs(context, ex, orig_shape, ex.rewrite(context, orig_shape))?;
Some(lhs + &rhs)
}
orig_rhs: Option<String>,
) -> Option<String> {
match orig_rhs {
- Some(ref new_str) if !new_str.contains('\n') => Some(format!(" {}", new_str)),
+ Some(ref new_str) if !new_str.contains('\n') && new_str.len() <= shape.width => {
+ Some(format!(" {}", new_str))
+ }
_ => {
// Expression did not fit on the same line as the identifier.
// Try splitting the line and see if that works better.
- let new_shape = try_opt!(
- Shape::indented(
- shape.block().indent.block_indent(context.config),
- context.config,
- ).sub_width(shape.rhs_overhead(context.config))
- );
+ let new_shape = Shape::indented(
+ shape.block().indent.block_indent(context.config),
+ context.config,
+ ).sub_width(shape.rhs_overhead(context.config))?;
let new_rhs = expr.rewrite(context, new_shape);
let new_indent_str = &new_shape.indent.to_string(context.config);
src.chars().filter(|&x| x == '\n').count()
}
- !next_line_rhs.contains('\n') ||
- count_line_breaks(orig_rhs) > count_line_breaks(next_line_rhs) + 1
+ !next_line_rhs.contains('\n')
+ || count_line_breaks(orig_rhs) > count_line_breaks(next_line_rhs) + 1
}
fn rewrite_expr_addrof(
impl<'a> ToExpr for MacroArg {
fn to_expr(&self) -> Option<&ast::Expr> {
- match self {
- &MacroArg::Expr(ref expr) => Some(expr),
+ match *self {
+ MacroArg::Expr(ref expr) => Some(expr),
_ => None,
}
}
fn can_be_overflowed(&self, context: &RewriteContext, len: usize) -> bool {
- match self {
- &MacroArg::Expr(ref expr) => can_be_overflowed_expr(context, expr, len),
- &MacroArg::Ty(ref ty) => can_be_overflowed_type(context, ty, len),
- &MacroArg::Pat(..) => false,
+ match *self {
+ MacroArg::Expr(ref expr) => can_be_overflowed_expr(context, expr, len),
+ MacroArg::Ty(ref ty) => can_be_overflowed_type(context, ty, len),
+ MacroArg::Pat(..) => false,
}
}
}
projects / rust.git / blobdiff