---

yaml
---
r: 4843
b: refs/heads/master
c: 09e21fe
h: refs/heads/master
i:
  4841: d04c1dd
  4839: 3b1a38a
v: v3

Author: brson

[refs]

@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 832741641113590e66b61aba3054fa3c5cb456c0
+refs/heads/master: 09e21fe2b93e926b77f105939f6d7f2fb6f7dcc9

trunk/src/comp/syntax/print/pprust.rs

@@ -1377,18 +1377,28 @@ fn operator_prec(op: ast::binop) -> int {
 }
 
 fn need_parens(expr: &@ast::expr, outer_prec: int) -> bool {
-    let add_them;
     alt expr.node {
       ast::expr_binary(op, _, _) {
-        add_them = operator_prec(op) < outer_prec;
+        operator_prec(op) < outer_prec
       }
-      ast::expr_cast(_, _) { add_them = parse::parser::as_prec < outer_prec; }
+      ast::expr_cast(_, _) { parse::parser::as_prec < outer_prec }
       ast::expr_ternary(_, _, _) {
-        add_them = parse::parser::ternary_prec < outer_prec;
-      }
-      _ { add_them = false; }
+        parse::parser::ternary_prec < outer_prec
+      }
+
+      // This may be too conservative in some cases
+      ast::expr_assign(_, _) { true }
+      ast::expr_move(_, _) { true }
+      ast::expr_swap(_, _) { true }
+      ast::expr_assign_op(_, _, _) { true }
+      ast::expr_ret(_) { true }
+      ast::expr_put(_) { true }
+      ast::expr_be(_) { true }
+      ast::expr_assert(_) { true }
+      ast::expr_check(_, _) { true }
+      ast::expr_log(_, _) { true }
+      _ { false }
     }
-    ret add_them;
 }
 
 fn print_maybe_parens(s: &ps, expr: &@ast::expr, outer_prec: int) {

trunk/src/test/run-pass/wierd-exprs.rs

@@ -1,4 +1,3 @@
-// xfail-pretty
 // Just a grab bug of stuff that you wouldn't want to actualy write
 
 fn strange() -> bool {