Skip to content

Resolve problems in finalization API #12036

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
Closed
brson opened this issue Feb 5, 2014 · 3 comments
Closed

Resolve problems in finalization API #12036

brson opened this issue Feb 5, 2014 · 3 comments
Milestone
1.0

Comments

@brson
Copy link
Contributor

brson commented Feb 5, 2014

The finalization API may need to change to support GC. I do not know the details. Nominating.

cc @pnkfelix
cc #11778
@pnkfelix
Copy link
Member

pnkfelix commented Feb 6, 2014

P-backcompat-libs, 1.0, because it is strongly intertwined with #11778. (If that is revisited, this should be too.)

@pnkfelix pnkfelix added this to the 1.0 milestone Feb 6, 2014
@brson
Copy link
Contributor Author

brson commented Oct 1, 2014

Nominating for removal from 1.0. #11778 is not 1.0. Is there anything to do here?

@pnkfelix
Copy link
Member

pnkfelix commented Oct 2, 2014

(closing, will move my issues to RFC repo issue.)

@pnkfelix pnkfelix closed this as completed Oct 2, 2014
bors added a commit to rust-lang-ci/rust that referenced this issue Jul 25, 2022
@bors
Auto merge of rust-lang#12037 - lnicola:inlay-hint-config, r=lnicola
34c3e0b 
fix: Remove `rust-analyzer.inlayHints.enable` and set language scope

Closes rust-lang#12036
CC rust-lang/rust-analyzer#12027 (comment)

The key was left there by mistake in rust-lang#12006.

Setting the configuration scope only works if you already have it created, which is fine, but unfortunately not quite discoverable.
flip1995 pushed a commit to flip1995/rust that referenced this issue Jan 11, 2024
@bors
Auto merge of rust-lang#12036 - kpreid:patch-1, r=Alexendoo
eec0f3d 
Polish `missing_enforced_import_renames` documentation

* Fixes a typo in the name of the lint (`enforce-import-renames` instead of `enforced-import-renames`).
* Copyedit “Why” paragraph.
* Make the example configuration use a multi-line list, since it is not particularly expected that a real project will have *exactly one* rename to enforce (and the old formatting had unbalanced whitespace).

changelog: none
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
1.0
Development

No branches or pull requests
2 participants
@brson @pnkfelix