Skip to content

Remove FIXMEs referencing #44232#159360

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
theemathas:del-44232-comments
Jul 16, 2026
Merged

Remove FIXMEs referencing #44232#159360
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
theemathas:del-44232-comments

Conversation

@theemathas

@theemathas theemathas commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

#44232 is already closed, since #152164 implemented the unused_features lint.

The deleted tests, as currently written, make no sense. Relevant functionality is already tested in:

  • tests/ui/feature-gates/unknown-feature.rs
  • tests/ui/feature-gates/stable-features.rs
  • tests/ui/lint/unused-features/unused-language-features.rs

These tests, as currently written, make no sense.

Relevant functionality is already tested in:
* `tests/ui/feature-gates/unknown-feature.rs`
* `tests/ui/feature-gates/stable-features.rs`
* `tests/ui/lint/unused-features/unused-language-features.rs`
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 16, 2026
@rustbot

rustbot commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

r? @petrochenkov

rustbot has assigned @petrochenkov.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 75 candidates
  • Random selection from 20 candidates

@rustbot

This comment has been minimized.

@theemathas
theemathas force-pushed the del-44232-comments branch from 5963f72 to 4b1bfb5 Compare July 16, 2026 05:08
@riking

riking commented Jul 16, 2026

Copy link
Copy Markdown

lint-unknown-feature has sure gone on an Incredible Journey. When originally added in a9bd049, the test read as follows:

// Copyright 2013 The Rust Project Developers.
// ... copyright header continues ...

#[deny(unknown_features)];

#[feature(this_is_not_a_feature)]; //~ ERROR: unknown feature

fn main() {}

And today it is:

//@ check-pass

#![warn(unused_features)]

#![allow(stable_features)]
// FIXME(#44232) we should warn that this isn't used.

fn main() {}

@petrochenkov

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-bors Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 4b1bfb5 has been approved by petrochenkov

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 16, 2026
jdonszelmann added a commit to jdonszelmann/rust that referenced this pull request Jul 16, 2026
…etrochenkov

Remove FIXMEs referencing rust-lang#44232

rust-lang#44232 is already closed, since rust-lang#152164 implemented the unused_features lint.

The deleted tests, as currently written, make no sense. Relevant functionality is already tested in:
* `tests/ui/feature-gates/unknown-feature.rs`
* `tests/ui/feature-gates/stable-features.rs`
* `tests/ui/lint/unused-features/unused-language-features.rs`
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Jul 16, 2026
…etrochenkov

Remove FIXMEs referencing rust-lang#44232

rust-lang#44232 is already closed, since rust-lang#152164 implemented the unused_features lint.

The deleted tests, as currently written, make no sense. Relevant functionality is already tested in:
* `tests/ui/feature-gates/unknown-feature.rs`
* `tests/ui/feature-gates/stable-features.rs`
* `tests/ui/lint/unused-features/unused-language-features.rs`
rust-bors Bot pushed a commit that referenced this pull request Jul 16, 2026
…uwer

Rollup of 18 pull requests

Successful merges:

 - #150679 (dirfd file operations (2/4))
 - #158758 (Rename `lazy_type_alias` feature gate to `checked_type_aliases` )
 - #159177 (Explicitly materialize debuginfo tests for all debuggers)
 - #159392 (Update Rust crate rand to v0.9.3 [SECURITY])
 - #159393 (Update Rust crate rkyv to v0.8.16 [SECURITY])
 - #159394 (Update Rust crate tar to v0.4.46 [SECURITY])
 - #159395 (Update Rust crate tracing-subscriber [SECURITY])
 - #159193 (Port compiletest's CLI to clap)
 - #158992 (Shorten types more when only interesting part is lifetimes)
 - #159226 (Track extra_lifetime_params_map per-owner)
 - #159273 (tidy: document u64 limit for numeric sort keys)
 - #159322 (Fix safety doc in intrinsics::simd)
 - #159329 (Fix the stale metrics directory warning)
 - #159357 (Add regression tests for two fixed issues)
 - #159360 (Remove FIXMEs referencing #44232)
 - #159375 (make a couple codegen-llvm tests compatible with 2021 edition)
 - #159381 (emit `(lldb)` prefix to make commands more distinct)
 - #159396 (Update actions/checkout action to v7)
@rust-bors
rust-bors Bot merged commit 7abb2b5 into rust-lang:main Jul 16, 2026
13 checks passed
@theemathas
theemathas deleted the del-44232-comments branch July 17, 2026 01:40
github-actions Bot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Jul 20, 2026
…uwer

Rollup of 18 pull requests

Successful merges:

 - rust-lang/rust#150679 (dirfd file operations (2/4))
 - rust-lang/rust#158758 (Rename `lazy_type_alias` feature gate to `checked_type_aliases` )
 - rust-lang/rust#159177 (Explicitly materialize debuginfo tests for all debuggers)
 - rust-lang/rust#159392 (Update Rust crate rand to v0.9.3 [SECURITY])
 - rust-lang/rust#159393 (Update Rust crate rkyv to v0.8.16 [SECURITY])
 - rust-lang/rust#159394 (Update Rust crate tar to v0.4.46 [SECURITY])
 - rust-lang/rust#159395 (Update Rust crate tracing-subscriber [SECURITY])
 - rust-lang/rust#159193 (Port compiletest's CLI to clap)
 - rust-lang/rust#158992 (Shorten types more when only interesting part is lifetimes)
 - rust-lang/rust#159226 (Track extra_lifetime_params_map per-owner)
 - rust-lang/rust#159273 (tidy: document u64 limit for numeric sort keys)
 - rust-lang/rust#159322 (Fix safety doc in intrinsics::simd)
 - rust-lang/rust#159329 (Fix the stale metrics directory warning)
 - rust-lang/rust#159357 (Add regression tests for two fixed issues)
 - rust-lang/rust#159360 (Remove FIXMEs referencing rust-lang/rust#44232)
 - rust-lang/rust#159375 (make a couple codegen-llvm tests compatible with 2021 edition)
 - rust-lang/rust#159381 (emit `(lldb)` prefix to make commands more distinct)
 - rust-lang/rust#159396 (Update actions/checkout action to v7)
@cuviper cuviper added this to the 1.99.0 milestone Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants