Missing parens in tuple pretty printer #6

wants to merge 1 commit into


None yet
2 participants

The non-unit tuples weren't being pretty printed with parens. Is the user supposed to add parens manually
with the parentheses expression?

If not, this patch adds them in the pretty printer.


bjpop commented Apr 10, 2012

Thanks travitch,

The intention is that the parser should insert a Paren into the AST.

If you generate the AST yourself, without parsing some input source, then you will need to add the Paren constructors where needed.

@bjpop bjpop closed this Apr 10, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment