Skip to content

Make managed_boxes feature gate error less opinionated - #10181

Merged
bors merged 1 commit into
rust-lang:masterfrom
brson:managed_boxes
Nov 1, 2013
Merged

Make managed_boxes feature gate error less opinionated#10181
bors merged 1 commit into
rust-lang:masterfrom
brson:managed_boxes

Conversation

@brson

@brson brson commented Oct 30, 2013

Copy link
Copy Markdown
Contributor

No description provided.

@bors bors closed this Nov 1, 2013
@bors
bors merged commit 6a33cc2 into rust-lang:master Nov 1, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 12, 2023
…1995

add a test against rust-lang#10089

This was just an oversight while doing the box default path trimming PR, so I'm adding it rather late than never.

---

changelog: none
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
10284: internal: definition based hover functions r=Veykril a=HKalbasi

This is part of rust-lang#10181 but since it is blocked and `hover.rs` is moving quickly so will cause conflicts, I submitted this PR.

This PR extract some parts of `hover` to `find_definition` (maybe this need to be moved to some other file?) and `hover_for_definition`, with those functions I will be able to calculate definition of every token, and calculate hover (and probably other queries) for each definition only once.

Co-authored-by: hamidreza kalbasi <hamidrezakalbasi@protonmail.com>
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
10181: Begining of lsif r=HKalbasi a=HKalbasi

This PR adds a `lsif` command to cli, which can be used as `rust-analyzer lsif /path/to/project > dump.lsif`. It now generates a valid, but pretty useless lsif (only supports folding ranges). The propose of this PR is to discussing about the structure of lsif generator, before starting anything serious.

cc `@matklad` rust-lang#8696 rust-lang#3098


Co-authored-by: hamidreza kalbasi <hamidrezakalbasi@protonmail.com>
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