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.

Sign up for GitHub

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

Jump to bottom

fmt!'s pretty printer dies on vectors of unit struct #7625

Closed
emberian opened this issue Jul 6, 2013 · 1 comment · Fixed by #7683
Closed

fmt!'s pretty printer dies on vectors of unit struct #7625

emberian opened this issue Jul 6, 2013 · 1 comment · Fixed by #7683

Comments

@emberian
Copy link
Member

emberian commented Jul 6, 2013 

fn main() {
    struct Foo;
    let x = ~[Foo, Foo, Foo];
    println(fmt!("%?", x));
    // hangs, gobbles up memory
}
@huonw
Copy link
Member

huonw commented Jul 6, 2013

I'd guess this is somehow linked to unit structs being 0 sized.

@alexcrichton alexcrichton mentioned this issue Jul 10, 2013
Merged
bors added a commit that referenced this issue Jul 11, 2013
@bors
auto merge of #7683 : alexcrichton/rust/issue-7625, r=thestinger
9b5d523 
Closes #7625
flip1995 pushed a commit to flip1995/rust that referenced this issue Sep 28, 2021
@bors
Auto merge of rust-lang#7646 - camsteffen:relative-target, r=flip1995
b556398 
Target directory cleanup

changelog: none

* .cargo/config now has `target-dir` specified so that it is inherited by child projects. The target directory needs to be shared with clippy_dev, but not necessarily at the project root. (cc rust-lang#7625)
* Uses `std::env::current_exe` (and its parent directories) whenever possible
* `CLIPPY_DRIVER_PATH` and `TARGET_LIBS` are no longer required from rustc bootstrap (but `HOST_LIBS` still is). These can be removed from the rustc side after merging.
* `CLIPPY_DOGFOOD` and the separate target directory are removed. This was originally added to mitigate rust-lang#7343.

r? `@flip1995`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't loop infinitely on 0-size structs in repr alexcrichton/rust
2 participants
@emberian @huonw