---

yaml
---
r: 40252
b: refs/heads/dist-snap
c: 1dc4d02
h: refs/heads/master
v: v3

Author: vertexclique

[refs]

@@ -7,7 +7,7 @@ refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: a810c03263670238bccd64cabb12a23a46e3a278
 refs/heads/incoming: e90142e536c150df0d9b4b2f11352152177509b5
-refs/heads/dist-snap: 62f98c8ff8179c9754718e62285b82b50dfcfe26
+refs/heads/dist-snap: 1dc4d024a88b6d2a6a31aad8d09d62c4dc883a85
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503
 refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0

branches/dist-snap/mk/tests.mk

@@ -93,22 +93,22 @@ cleantestlibs:
          | xargs rm -rf
 
 check: cleantestlibs cleantmptestlogs tidy all check-stage2
-	$(Q)$(CFG_PYTHON) $(S)src/etc/check-summary.py tmp/*.log
+	$(Q)$(S)src/etc/check-summary.py tmp/*.log
 
 check-notidy: cleantestlibs cleantmptestlogs all check-stage2
-	$(Q)$(CFG_PYTHON) $(S)src/etc/check-summary.py tmp/*.log
+	$(Q)$(S)src/etc/check-summary.py tmp/*.log
 
 check-full: cleantestlibs cleantmptestlogs tidy \
             all check-stage1 check-stage2 check-stage3
-	$(Q)$(CFG_PYTHON) $(S)src/etc/check-summary.py tmp/*.log
+	$(Q)$(S)src/etc/check-summary.py tmp/*.log
 
 check-test: cleantestlibs cleantmptestlogs all check-stage2-rfail
-	$(Q)$(CFG_PYTHON) $(S)src/etc/check-summary.py tmp/*.log
+	$(Q)$(S)src/etc/check-summary.py tmp/*.log
 
 check-lite: cleantestlibs cleantmptestlogs rustc-stage2 \
 	check-stage2-core check-stage2-std check-stage2-rpass \
 	check-stage2-rfail check-stage2-cfail
-	$(Q)$(CFG_PYTHON) $(S)src/etc/check-summary.py tmp/*.log
+	$(Q)$(S)src/etc/check-summary.py tmp/*.log
 
 # Run the tidy script in multiple parts to avoid huge 'echo' commands
 ifdef CFG_NOTIDY

branches/dist-snap/src/libsyntax/ast.rs

@@ -744,10 +744,7 @@ enum expr_ {
     expr_struct(@path, ~[field], Option<@expr>),
 
     // A vector literal constructed from one repeated element.
-    expr_repeat(@expr /* element */, @expr /* count */, mutability),
-
-    // No-op: used solely so we can pretty-print faithfully
-    expr_paren(@expr)
+    expr_repeat(@expr /* element */, @expr /* count */, mutability)
 }
 
 #[auto_serialize]

branches/dist-snap/src/libsyntax/fold.rs

@@ -494,8 +494,7 @@ fn noop_fold_expr(e: expr_, fld: ast_fold) -> expr_ {
             expr_struct(fld.fold_path(path),
                         vec::map(fields, |x| fold_field(*x)),
                         option::map(&maybe_expr, |x| fld.fold_expr(*x)))
-          },
-          expr_paren(ex) => expr_paren(fld.fold_expr(ex))
+          }
         }
 }
 

branches/dist-snap/src/libsyntax/parse/classify.rs

@@ -36,3 +36,39 @@ fn stmt_ends_with_semi(stmt: ast::stmt) -> bool {
       }
     }
 }
+
+fn need_parens(expr: @ast::expr, outer_prec: uint) -> bool {
+    match expr.node {
+      ast::expr_binary(op, _, _) => operator_prec(op) < outer_prec,
+      ast::expr_cast(_, _) => parse::prec::as_prec < outer_prec,
+      // This may be too conservative in some cases
+      ast::expr_assign(_, _) => true,
+      ast::expr_swap(_, _) => true,
+      ast::expr_assign_op(_, _, _) => true,
+      ast::expr_ret(_) => true,
+      ast::expr_assert(_) => true,
+      ast::expr_log(_, _, _) => true,
+      _ => !parse::classify::expr_requires_semi_to_be_stmt(expr)
+    }
+}
+
+fn ends_in_lit_int(ex: @ast::expr) -> bool {
+    match ex.node {
+      ast::expr_lit(node) => match node {
+        @{node: ast::lit_int(_, ast::ty_i), _}
+        | @{node: ast::lit_int_unsuffixed(_), _} => true,
+        _ => false
+      },
+      ast::expr_binary(_, _, sub) | ast::expr_unary(_, sub) |
+      ast::expr_copy(sub) | ast::expr_assign(_, sub) |
+      ast::expr_assign_op(_, _, sub) | ast::expr_swap(_, sub) |
+      ast::expr_log(_, _, sub) | ast::expr_assert(sub) => {
+        ends_in_lit_int(sub)
+      }
+      ast::expr_fail(osub) | ast::expr_ret(osub) => match osub {
+        Some(ex) => ends_in_lit_int(ex),
+        _ => false
+      },
+      _ => false
+    }
+}