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

test: Remove all ~[T] from tests, libgetopts, compiletest, librustdoc, and libnum #12837

Closed
wants to merge 3 commits into from

Conversation

pcwalton
Copy link
Contributor

r? @brson

@huonw
Copy link
Member

huonw commented Mar 21, 2014

I'm updating this.

@huonw
Copy link
Member

huonw commented Mar 21, 2014

Superseded by #13056

@huonw huonw closed this Mar 21, 2014
bors added a commit that referenced this pull request Mar 21, 2014
test: Remove all `~[T]` from tests, libgetopts, compiletest, librustdoc, and libnum

And most from libtest, libflate, and adds `deny(deprecated_owned_vector)`s to the smaller modules with that have zero (or nearly zero) uses of `~[T]`.

Revival of #12837
lnicola pushed a commit to lnicola/rust that referenced this pull request Aug 9, 2022
…data, r=Veykril

feat: support associated values in "Generate Enum Variant" assist

This change adds support for associated values to the "Generate Enum Variant" assist.

I've split the implementation out into 4 steps to make code review easier:
- Add "add_variant" support to the structural ast editing system in `edit_in_place`
- Migrate `generate_enum_variant` to use structural ast editing instead of string manipulation
- Support tuple fields
- Support record fields

Please let me know if I should leave the commits as-is, or squash before merging.

Fixes rust-lang#12797
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 13, 2024
…=dswij

Fix `to_string_in_format_args` with macro call receiver

Fixes part of rust-lang#12837

changelog: none
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.

4 participants