fn expr(&mut self, expr: &hir::Expr, pred: CFGIndex) -> CFGIndex {
match expr.node {
- hir::ExprBlock(ref blk) => {
+ hir::ExprBlock(ref blk, _) => {
let blk_exit = self.block(&blk, pred);
self.add_ast_node(expr.hir_id.local_id, &[blk_exit])
}
expression.span,
expression.id)
}
- ExprBlock(ref block) => visitor.visit_block(block),
+ ExprBlock(ref block, ref opt_label) => {
+ walk_list!(visitor, visit_label, opt_label);
+ visitor.visit_block(block);
+ }
ExprAssign(ref left_hand_expression, ref right_hand_expression) => {
visitor.visit_expr(right_hand_expression);
visitor.visit_expr(left_hand_expression)
);
block.expr = Some(this.wrap_in_try_constructor(
"from_ok", tail, unstable_span));
- hir::ExprBlock(P(block))
+ hir::ExprBlock(P(block), None)
})
}
ExprKind::Match(ref expr, ref arms) => hir::ExprMatch(
})
})
}
- ExprKind::Block(ref blk) => hir::ExprBlock(self.lower_block(blk, false)),
+ ExprKind::Block(ref blk, opt_label) => {
+ hir::ExprBlock(self.lower_block(blk, false), self.lower_label(opt_label))
+ }
ExprKind::Assign(ref el, ref er) => {
hir::ExprAssign(P(self.lower_expr(el)), P(self.lower_expr(er)))
}
}
fn expr_block(&mut self, b: P<hir::Block>, attrs: ThinVec<Attribute>) -> hir::Expr {
- self.expr(b.span, hir::ExprBlock(b), attrs)
+ self.expr(b.span, hir::ExprBlock(b, None), attrs)
}
fn expr_tuple(&mut self, sp: Span, exprs: hir::HirVec<hir::Expr>) -> P<hir::Expr> {
/// This may also be a generator literal, indicated by the final boolean,
/// in that case there is an GeneratorClause.
ExprClosure(CaptureClause, P<FnDecl>, BodyId, Span, Option<GeneratorMovability>),
- /// A block (`{ ... }`)
- ExprBlock(P<Block>),
+ /// A block (`'label: { ... }`)
+ ExprBlock(P<Block>, Option<Label>),
/// An assignment (`a = foo()`)
ExprAssign(P<Expr>, P<Expr>),
self.print_else(e.as_ref().map(|e| &**e))
}
// "final else"
- hir::ExprBlock(ref b) => {
+ hir::ExprBlock(ref b, _) => {
self.cbox(indent_unit - 1)?;
self.ibox(0)?;
self.s.word(" else ")?;
// empty box to satisfy the close.
self.ibox(0)?;
}
- hir::ExprBlock(ref blk) => {
+ hir::ExprBlock(ref blk, opt_label) => {
+ if let Some(label) = opt_label {
+ self.print_name(label.name)?;
+ self.word_space(":")?;
+ }
// containing cbox, will be closed by print-block at }
self.cbox(indent_unit)?;
// head-box, will be closed by print-block after {
self.word_space("=>")?;
match arm.body.node {
- hir::ExprBlock(ref blk) => {
+ hir::ExprBlock(ref blk, opt_label) => {
+ if let Some(label) = opt_label {
+ self.print_name(label.name)?;
+ self.word_space(":")?;
+ }
// the block will close the pattern's ibox
self.print_block_unclosed_indent(&blk, indent_unit)?;
match e.node {
hir::ExprIf(..) |
hir::ExprMatch(..) |
- hir::ExprBlock(_) |
+ hir::ExprBlock(..) |
hir::ExprWhile(..) |
hir::ExprLoop(..) => false,
_ => true,
ExprLoop(body, label, loop_src),
ExprMatch(matchee, arms, match_src),
ExprClosure(capture_clause, decl, body_id, span, gen),
- ExprBlock(blk),
+ ExprBlock(blk, label),
ExprAssign(lhs, rhs),
ExprAssignOp(op, lhs, rhs),
ExprField(owner, field_name),
self.consume_expr(&rhs);
}
- hir::ExprBlock(ref blk) => {
+ hir::ExprBlock(ref blk, _) => {
self.walk_block(&blk);
}
succ
}
- hir::ExprBlock(ref blk) => {
+ // Note that labels have been resolved, so we don't need to look
+ // at the label ident
+ hir::ExprBlock(ref blk, _) => {
self.propagate_through_block(&blk, succ)
}
}
hir::ExprCast(ref subexpr, _) => {
record_rvalue_scope_if_borrow_expr(visitor, &subexpr, blk_id)
}
- hir::ExprBlock(ref block) => {
+ hir::ExprBlock(ref block, _) => {
if let Some(ref subexpr) = block.expr {
record_rvalue_scope_if_borrow_expr(
visitor, &subexpr, blk_id);
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for UnsafeCode {
fn check_expr(&mut self, cx: &LateContext, e: &hir::Expr) {
- if let hir::ExprBlock(ref blk) = e.node {
+ if let hir::ExprBlock(ref blk, _) = e.node {
// Don't warn about generated blocks, that'll just pollute the output.
if blk.rules == hir::UnsafeBlock(hir::UserProvided) {
self.report_unsafe(cx, blk.span, "usage of an `unsafe` block");
}
}
- hir::ExprBlock(ref blk) => ExprKind::Block { body: &blk },
+ hir::ExprBlock(ref blk, _) => ExprKind::Block { body: &blk },
hir::ExprAssign(ref lhs, ref rhs) => {
ExprKind::Assign {
}
// Check if this is an unsafe block, or an item
match node {
- Node::NodeExpr(&hir::Expr { node: hir::ExprBlock(ref block), ..}) => {
+ Node::NodeExpr(&hir::Expr { node: hir::ExprBlock(ref block, _), ..}) => {
if block_is_unsafe(&*block) {
// Found an unsafe block, we can bail out here.
return true;
let mut callee = &**callee;
loop {
callee = match callee.node {
- hir::ExprBlock(ref block) => match block.expr {
+ hir::ExprBlock(ref block, _) => match block.expr {
Some(ref tail) => &tail,
None => break
},
}
}
- hir::ExprBlock(_) |
+ hir::ExprBlock(..) |
hir::ExprIndex(..) |
hir::ExprField(..) |
hir::ExprArray(_) |
// Warn for non-block expressions with diverging children.
match expr.node {
- hir::ExprBlock(_) |
+ hir::ExprBlock(..) |
hir::ExprLoop(..) | hir::ExprWhile(..) |
hir::ExprIf(..) | hir::ExprMatch(..) => {}
hir::ExprClosure(capture, ref decl, body_id, _, gen) => {
self.check_expr_closure(expr, capture, &decl, body_id, gen, expected)
}
- hir::ExprBlock(ref body) => {
+ hir::ExprBlock(ref body, _) => {
self.check_block_with_expected(&body, expected)
}
hir::ExprCall(ref callee, ref args) => {
/// Whether this expression would be valid somewhere that expects a value, for example, an `if`
/// condition.
pub fn returns(&self) -> bool {
- if let ExprKind::Block(ref block) = self.node {
+ if let ExprKind::Block(ref block, _) = self.node {
match block.stmts.last().map(|last_stmt| &last_stmt.node) {
// implicit return
Some(&StmtKind::Expr(_)) => true,
///
/// The final span is the span of the argument block `|...|`
Closure(CaptureBy, Movability, P<FnDecl>, P<Expr>, Span),
- /// A block (`{ ... }`)
- Block(P<Block>),
+ /// A block (`'label: { ... }`)
+ Block(P<Block>, Option<Label>),
/// A catch block (`catch { ... }`)
Catch(P<Block>),
self.expr(span, ast::ExprKind::MethodCall(segment, args))
}
fn expr_block(&self, b: P<ast::Block>) -> P<ast::Expr> {
- self.expr(b.span, ast::ExprKind::Block(b))
+ self.expr(b.span, ast::ExprKind::Block(b, None))
}
fn field_imm(&self, span: Span, ident: Ident, e: P<ast::Expr>) -> ast::Field {
ast::Field {
folder.fold_expr(body),
folder.new_span(span))
}
- ExprKind::Block(blk) => ExprKind::Block(folder.fold_block(blk)),
+ ExprKind::Block(blk, opt_label) => {
+ ExprKind::Block(folder.fold_block(blk),
+ opt_label.map(|label| folder.fold_label(label)))
+ }
ExprKind::Assign(el, er) => {
ExprKind::Assign(folder.fold_expr(el), folder.fold_expr(er))
}
ast::ExprKind::If(..) |
ast::ExprKind::IfLet(..) |
ast::ExprKind::Match(..) |
- ast::ExprKind::Block(_) |
+ ast::ExprKind::Block(..) |
ast::ExprKind::While(..) |
ast::ExprKind::WhileLet(..) |
ast::ExprKind::Loop(..) |
token::NtBlock(ref block) => {
$p.bump();
let span = $p.span;
- let kind = ExprKind::Block((*block).clone());
+ let kind = ExprKind::Block((*block).clone(), None);
return Ok($p.mk_expr(span, kind, ThinVec::new()));
}
_ => {},
};
}
token::OpenDelim(token::Brace) => {
- return self.parse_block_expr(lo, BlockCheckMode::Default, attrs);
+ return self.parse_block_expr(None, lo, BlockCheckMode::Default, attrs);
}
token::BinOp(token::Or) | token::OrOr => {
return self.parse_lambda_expr(attrs);
if self.eat_keyword(keywords::Loop) {
return self.parse_loop_expr(Some(label), lo, attrs)
}
- let msg = "expected `while`, `for`, or `loop` after a label";
+ if self.token == token::OpenDelim(token::Brace) {
+ return self.parse_block_expr(Some(label),
+ lo,
+ BlockCheckMode::Default,
+ attrs);
+ }
+ let msg = "expected `while`, `for`, `loop` or `{` after a label";
let mut err = self.fatal(msg);
err.span_label(self.span, msg);
return Err(err);
}
if self.eat_keyword(keywords::Unsafe) {
return self.parse_block_expr(
+ None,
lo,
BlockCheckMode::Unsafe(ast::UserProvided),
attrs);
}
/// Parse a block or unsafe block
- pub fn parse_block_expr(&mut self, lo: Span, blk_mode: BlockCheckMode,
+ pub fn parse_block_expr(&mut self, opt_label: Option<Label>,
+ lo: Span, blk_mode: BlockCheckMode,
outer_attrs: ThinVec<Attribute>)
-> PResult<'a, P<Expr>> {
self.expect(&token::OpenDelim(token::Brace))?;
attrs.extend(self.parse_inner_attributes()?);
let blk = self.parse_block_tail(lo, blk_mode)?;
- return Ok(self.mk_expr(blk.span, ExprKind::Block(blk), attrs));
+ return Ok(self.mk_expr(blk.span, ExprKind::Block(blk, opt_label), attrs));
}
/// parse a.b or a(13) or a[4] or just a
// If an explicit return type is given, require a
// block to appear (RFC 968).
let body_lo = self.span;
- self.parse_block_expr(body_lo, BlockCheckMode::Default, ThinVec::new())?
+ self.parse_block_expr(None, body_lo, BlockCheckMode::Default, ThinVec::new())?
}
};
return self.parse_if_expr(ThinVec::new());
} else {
let blk = self.parse_block()?;
- return Ok(self.mk_expr(blk.span, ExprKind::Block(blk), ThinVec::new()));
+ return Ok(self.mk_expr(blk.span, ExprKind::Block(blk, None), ThinVec::new()));
}
}
self.print_else(e.as_ref().map(|e| &**e))
}
// "final else"
- ast::ExprKind::Block(ref b) => {
+ ast::ExprKind::Block(ref b, _) => {
self.cbox(INDENT_UNIT - 1)?;
self.ibox(0)?;
self.s.word(" else ")?;
// empty box to satisfy the close.
self.ibox(0)?;
}
- ast::ExprKind::Block(ref blk) => {
+ ast::ExprKind::Block(ref blk, opt_label) => {
+ if let Some(label) = opt_label {
+ self.print_ident(label.ident)?;
+ self.word_space(":")?;
+ }
// containing cbox, will be closed by print-block at }
self.cbox(INDENT_UNIT)?;
// head-box, will be closed by print-block after {
self.word_space("=>")?;
match arm.body.node {
- ast::ExprKind::Block(ref blk) => {
+ ast::ExprKind::Block(ref blk, opt_label) => {
+ if let Some(label) = opt_label {
+ self.print_ident(label.ident)?;
+ self.word_space(":")?;
+ }
+
// the block will close the pattern's ibox
self.print_block_unclosed_indent(blk, INDENT_UNIT)?;
expression.span,
expression.id)
}
- ExprKind::Block(ref block) => visitor.visit_block(block),
+ ExprKind::Block(ref block, ref opt_label) => {
+ walk_list!(visitor, visit_label, opt_label);
+ visitor.visit_block(block);
+ }
ExprKind::Assign(ref left_hand_expression, ref right_hand_expression) => {
visitor.visit_expr(left_hand_expression);
visitor.visit_expr(right_hand_expression);