pretty-printer does not parenthesize ret expr in callee position

[jruderman]

(ret g)(41)

becomes

ret g(41)

Here's an invalid program that gets transformed into a program that prints 42.

fn f(g: fn(int) -> int) -> int { (ret g)(41) }
fn succ(x: int) -> int { x + 1 }
fn main() { log_err f(succ); }

[brson]

I don't know if we should expend compiler complexity on pretty-printing bogus code. Besides making life easier for fuzzers, is there any value in making this work? Is it possible to express valid code that has this same problem?

[jruderman]

Making life easier for fuzzers can be worthwhile ;)

[marijnh]

I think consistency is worthwhile. Bad corner-cases, even if they are for weird code, are a sign of an ambiguous/confusing grammar, which is not a nice property of a language.