Skip to content

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Apr 3, 2024

Fixes #16947

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 3, 2024
@Veykril
Copy link
Member Author

Veykril commented Apr 3, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Apr 3, 2024

📌 Commit 65e7b75 has been approved by Veykril

It is now in the queue for this repository.

@Veykril Veykril force-pushed the cfg-censor-fixes branch from 65e7b75 to 2b62d4b Compare April 3, 2024 14:43
@Veykril
Copy link
Member Author

Veykril commented Apr 3, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Apr 3, 2024

📌 Commit 2b62d4b has been approved by Veykril

It is now in the queue for this repository.

bors added a commit that referenced this pull request Apr 3, 2024
fix: Fix some cfg censoring bugs

Fixes #16947
@bors
Copy link
Contributor

bors commented Apr 3, 2024

⌛ Testing commit 2b62d4b with merge 73dacaf...

@bors
Copy link
Contributor

bors commented Apr 3, 2024

⌛ Testing commit 2b62d4b with merge c0eb78e...

@bors
Copy link
Contributor

bors commented Apr 3, 2024

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing c0eb78e to master...

@bors bors merged commit c0eb78e into rust-lang:master Apr 3, 2024
break 'attrs;
if let Some(enabled) = check_cfg(db, &attr, krate) {
if enabled {
debug!("censoring {:?}", attr.syntax());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on an aside, i wonder if it's possible to write an SRR that rewrites debug!("censoring {:?}", attr.syntax()); to debug!(?attr.syntax(), "censoring").

@Veykril Veykril deleted the cfg-censor-fixes branch September 9, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

False positive: missing structure fields, on structure fields behind #cfg
4 participants