Skip to content

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

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

Merged
merged 7 commits into from
Mar 21, 2014

Conversation

huonw
Copy link
Member

@huonw huonw commented 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

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
@bors bors closed this Mar 21, 2014
@bors bors merged commit 40a52c0 into rust-lang:master Mar 21, 2014
@huonw huonw deleted the devecing-tests branch March 21, 2014 16:34
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Sep 6, 2022
internal: Migrate to using format arg captures in `syntax::make`
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 11, 2024
…ishearth

Remove internal `compiler_lint_functions` lint

This internal lint has effectively been superseded by `disallowed_methods` when we started using that in rust-lang#11811 (I didn't even know that we also had this internal lint at the time of when I created that PR).

Some of the methods that this looks for also don't exist anymore (`span_lint_note` and `span_lint_help`), though there was one that that lint had but wasn't disallowed in clippy.toml (`LintContext::lint`)

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.

3 participants