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

std: Ignore a flaky test on freebsd #14046

Merged
merged 1 commit into from
May 9, 2014

Conversation

alexcrichton
Copy link
Member

This test runs successfully manually, but the bots are having trouble getting
this test to pass. Ignore it on freebsd for now.

This test runs successfully manually, but the bots are having trouble getting
this test to pass. Ignore it on freebsd for now.
bors added a commit that referenced this pull request May 9, 2014
…kballard

This test runs successfully manually, but the bots are having trouble getting
this test to pass. Ignore it on freebsd for now.
@bors bors closed this May 9, 2014
@bors bors merged commit 426d022 into rust-lang:master May 9, 2014
@alexcrichton alexcrichton deleted the ignore-a-test-on-freebsd branch May 11, 2014 18:20
lnicola pushed a commit to lnicola/rust that referenced this pull request Mar 13, 2023
…, r=Veykril

Fix associated item visibility in block-local impls

Fixes rust-lang#14046

When we're resolving visibility of block-local items...

> `self` normally refers to the containing non-block module, and `super` to its parent (etc.). However, visibilities must only refer to a module in the DefMap they're written in, so we restrict them when that happens. ([link])

 ...unless we're resolving visibility of associated items in block-local impls, because that impl is semantically "hoisted" to the nearest (non-block) module. With this PR, we skip the adjustment for such items.

Since visibility representation of those items is modified, this PR also adjusts visibility rendering in `HirDisplay`.

[link]: https://github.com/rust-lang/rust-analyzer/blob/a6603fc21d50b3386a488c96225b2d1fd492e533/crates/hir-def/src/nameres/path_resolution.rs#L101-L103
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants