From f841e894437843c142ebd7e0b0a18ed00ed52457 Mon Sep 17 00:00:00 2001 From: Tim Chevalier Date: Thu, 25 Aug 2011 17:42:38 -0700 Subject: Support unchecked blocks This patch supports the syntax unchecked { ... } to disable purity checking within a block. Presumably it will only be used within a declared "pure fn". However, there is no checking that it doesn't occur elsewhere, and it would be harmless for it to do so. I went with Lindsey's suggestion for the syntax, but it's subject to change. This allows you to write code that uses predicates that call arbitrary Rust functions, but you must declare your intentions by wrapping it in an unchecked { ... } block. The test case run-pass/unchecked-predicates.rs demonstrates how to do that. --- src/comp/syntax/parse/parser.rs | 26 +++++++++++++++++++------- 1 file changed, 19 insertions(+), 7 deletions(-) (limited to 'src/comp/syntax/parse') diff --git a/src/comp/syntax/parse/parser.rs b/src/comp/syntax/parse/parser.rs index f7329bd3b3b..729338b4b30 100644 --- a/src/comp/syntax/parse/parser.rs +++ b/src/comp/syntax/parse/parser.rs @@ -837,7 +837,7 @@ fn parse_bottom_expr(p: &parser) -> @ast::expr { } else if p.peek() == token::BINOP(token::OR) { ret parse_fn_block_expr(p); } else { - let blk = parse_block_tail(p, lo); + let blk = parse_block_tail(p, lo, ast::checked); ret mk_expr(p, blk.span.lo, blk.span.hi, ast::expr_block(blk)); } } else if eat_word(p, "if") { @@ -860,6 +860,10 @@ fn parse_bottom_expr(p: &parser) -> @ast::expr { ret parse_fn_expr(p, ast::proto_block); } else if eat_word(p, "lambda") { ret parse_fn_expr(p, ast::proto_closure); + } else if eat_word(p, "unchecked") { + expect(p, token::LBRACE); + let blk = parse_block_tail(p, lo, ast::unchecked); + ret mk_expr(p, blk.span.lo, blk.span.hi, ast::expr_block(blk)); } else if p.peek() == token::LBRACKET { p.bump(); let mut = parse_mutability(p); @@ -876,7 +880,7 @@ fn parse_bottom_expr(p: &parser) -> @ast::expr { ret mk_mac_expr(p, lo, p.get_hi_pos(), ast::mac_embed_type(ty)) } else if p.peek() == token::POUND_LBRACE { p.bump(); - let blk = ast::mac_embed_block(parse_block_tail(p, lo)); + let blk = ast::mac_embed_block(parse_block_tail(p, lo, ast::checked)); ret mk_mac_expr(p, lo, p.get_hi_pos(), blk); } else if p.peek() == token::ELLIPSIS { p.bump(); @@ -1309,7 +1313,7 @@ fn parse_fn_expr(p: &parser, proto: ast::proto) -> @ast::expr { fn parse_fn_block_expr(p: &parser) -> @ast::expr { let lo = p.get_last_lo_pos(); let decl = parse_fn_block_decl(p); - let body = parse_block_tail(p, lo); + let body = parse_block_tail(p, lo, ast::checked); let _fn = {decl: decl, proto: ast::proto_block, body: body}; ret mk_expr(p, lo, body.span.hi, ast::expr_fn(_fn)); } @@ -1664,12 +1668,20 @@ fn stmt_ends_with_semi(stmt: &ast::stmt) -> bool { fn parse_block(p: &parser) -> ast::blk { let lo = p.get_lo_pos(); - expect(p, token::LBRACE); - be parse_block_tail(p, lo); + if eat_word(p, "unchecked") { + be parse_block_tail(p, lo, ast::unchecked); + } + else { + expect(p, token::LBRACE); + be parse_block_tail(p, lo, ast::checked); + } } +// Precondition: already parsed the '{' or '#{' +// I guess that also means "already parsed the 'impure'" if +// necessary, and this should take a qualifier. // some blocks start with "#{"... -fn parse_block_tail(p: &parser, lo: uint) -> ast::blk { +fn parse_block_tail(p: &parser, lo: uint, s: ast::check_mode) -> ast::blk { let stmts: [@ast::stmt] = []; let expr: option::t<@ast::expr> = none; while p.peek() != token::RBRACE { @@ -1710,7 +1722,7 @@ fn parse_block_tail(p: &parser, lo: uint) -> ast::blk { } let hi = p.get_hi_pos(); p.bump(); - let bloc = {stmts: stmts, expr: expr, id: p.get_id()}; + let bloc = {stmts: stmts, expr: expr, id: p.get_id(), rules: s}; ret spanned(lo, hi, bloc); } -- cgit 1.4.1-3-g733a5