From 16bdb85cb408ca9f324a1ca83596b1b3dbec67b1 Mon Sep 17 00:00:00 2001 From: David Rajchenbach-Teller Date: Mon, 7 Nov 2011 14:46:02 +0100 Subject: [PATCH] [Parser] parser.rs: Made two error messages less ambiguous --- src/comp/syntax/parse/parser.rs | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/comp/syntax/parse/parser.rs b/src/comp/syntax/parse/parser.rs index 66a3b3440e8..f859300f3ea 100644 --- a/src/comp/syntax/parse/parser.rs +++ b/src/comp/syntax/parse/parser.rs @@ -1268,7 +1268,9 @@ fn parse_if_expr_1(p: parser) -> hi = elexpr.span.hi; } else if !option::is_none(thn.node.expr) { let sp = option::get(thn.node.expr).span; - p.span_fatal(sp, "if without else can not return a value"); + p.span_fatal(sp, "`if` without `else` can not produce a result"); + //TODO: If a suggestion mechanism appears, suggest that the + //user may have forgotten a ';' } ret {cond: cond, then: thn, els: els, lo: lo, hi: hi}; } @@ -1679,7 +1681,9 @@ fn parse_block_no_value(p: parser) -> ast::blk { let blk = parse_block(p); if !option::is_none(blk.node.expr) { let sp = option::get(blk.node.expr).span; - p.span_fatal(sp, "this block must not return a value"); + p.span_fatal(sp, "this block must not have a result"); + //TODO: If a suggestion mechanism appears, suggest that the + //user may have forgotten a ';' } ret blk; }