// option. This file may not be copied, modified, or distributed
// except according to those terms.
-use std::cmp::{min, Ordering};
+use std::cmp::min;
use std::borrow::Cow;
-use std::fmt::Write;
use std::iter::{repeat, ExactSizeIterator};
use syntax::{ast, ptr};
shape,
false,
),
- ast::ExprKind::Lit(ref l) => match l.node {
- ast::LitKind::Str(_, ast::StrStyle::Cooked) => {
- rewrite_string_lit(context, l.span, shape)
- }
- _ => Some(context.snippet(expr.span)),
- },
+ 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());
- let callee_str = try_opt!(callee.rewrite(context, 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),
// 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)
}
}
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 {
Some(format!(
"{}{}",
"do catch ",
- try_opt!(block.rewrite(context, Shape::legacy(budget, shape.indent)))
+ block.rewrite(context, Shape::legacy(budget, shape.indent))?
))
}
}
})
.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
// 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(
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
};
// 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.
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))
}
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,
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 {
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(),
};
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)
// 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))
}
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
+ if !forbid_same_line && !context.config.match_arm_forces_newline()
&& (is_block
|| (!body_str.contains('\n') && body_str.len() <= body_shape.width)) =>
{
} 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);
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(
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)
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)
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')
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
{
- 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);
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);