Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pretty printer doesn't preserve built-in trait bounds on trait objects #9253

Closed
huonw opened this issue Sep 17, 2013 · 0 comments · Fixed by #9264
Closed

Pretty printer doesn't preserve built-in trait bounds on trait objects #9253

huonw opened this issue Sep 17, 2013 · 0 comments · Fixed by #9264
Labels
A-pretty Area: Pretty printing (including `-Z unpretty`)

Comments

@huonw
Copy link
Member

huonw commented Sep 17, 2013

fn foo(x: ~Send:Freeze) -> ~Send:Freeze { x }

fn main() {
    let x: ~Send:Freeze;

    ~1 as ~Send:Freeze;
}

pretty prints to

fn foo(x: ~Send) -> ~Send { x }

fn main() {
    let x: ~Send;

    ~1 as ~Send;
}

(I xfail-pretty'd a test in #9225 because of this.)

bors added a commit that referenced this issue Sep 18, 2013
Since 3b6314c the pretty printer seems to only print trait bounds for `ast::ty_path(...)`s that have a generics arguments list. That seems wrong, so let's always print them.

Closes #9253, un-xfails test for #7673.
@bors bors closed this as completed in 823ebb1 Sep 18, 2013
flip1995 pushed a commit to flip1995/rust that referenced this issue Jul 28, 2022
Tell the user how to revert `dev setup intellij`

changelog: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-pretty Area: Pretty printing (including `-Z unpretty`)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant