Skip to content

privacy: Visit types and traits in impls in type privacy lints - #138317

Merged
bors merged 1 commit into
rust-lang:masterfrom
petrochenkov:libsearch3
Mar 25, 2025
Merged

privacy: Visit types and traits in impls in type privacy lints#138317
bors merged 1 commit into
rust-lang:masterfrom
petrochenkov:libsearch3

Conversation

@petrochenkov

Copy link
Copy Markdown
Contributor

With one exception to avoid false positives.

Fixes the same issue as #134176.

@rustbot

Copy link
Copy Markdown
Collaborator

r? @cjgillot

rustbot has assigned @cjgillot.
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

@rustbotrustbot 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 Mar 10, 2025
Comment threadtests/ui/privacy/pub-priv-dep/pub-priv1.rs
@petrochenkov

Copy link
Copy Markdown
ContributorAuthor

r? compiler

@rustbotrustbot assigned estebank and unassigned cjgillotMar 24, 2025
@compiler-errors

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Mar 24, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit fc76f1b has been approved by compiler-errors

It is now in the queue for this repository.

@borsbors 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 Mar 24, 2025
TaKO8Ki added a commit to TaKO8Ki/rust that referenced this pull request Mar 24, 2025
…r-errors
privacy: Visit types and traits in impls in type privacy lints
With one exception to avoid false positives.
Fixes the same issue as rust-lang#134176.
@TaKO8KiTaKO8Ki mentioned this pull request Mar 24, 2025
@compiler-errors

Copy link
Copy Markdown
Contributor

Needs a rebase b/c ControlFlow became #[must_use] recently. I'll open/close to rerun CI.

@compiler-errors

Copy link
Copy Markdown
Contributor

@bors r-

@borsbors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Mar 24, 2025
@TaKO8Ki

TaKO8Ki commented Mar 24, 2025

Copy link
Copy Markdown
Member

Hi. After merging #137449, ControlFlow is must_use, so could you fix this issue?

 error: unused `ControlFlow` that must be used
--> compiler/rustc_privacy/src/lib.rs:1409:13
|
1409 | self.visit_trait(trait_ref.instantiate_identity());
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: `-D unused-must-use` implied by `-D warnings`
= help: to override `-D warnings` add `#[allow(unused_must_use)]`
help: use `let _ = ...` to ignore the resulting value
|
1409 | let _ = self.visit_trait(trait_ref.instantiate_identity());
| +++++++

rollup pull request: #138909

@rust-log-analyzer

This comment was marked as resolved.

@petrochenkov

Copy link
Copy Markdown
ContributorAuthor

@bors r=compiler-errors

@bors

bors commented Mar 25, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit f756304 has been approved by compiler-errors

It is now in the queue for this repository.

@borsbors 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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Mar 25, 2025
bors added a commit to rust-lang-ci/rust that referenced this pull request Mar 25, 2025
…iaskrgr
Rollup of 8 pull requests
Successful merges:
- rust-lang#135745 (Recognise new IPv6 non-global range from IETF RFC 9602)
- rust-lang#137247 (cg_llvm: Reduce the visibility of types, modules and using declarations in `rustc_codegen_llvm`.)
- rust-lang#138317 (privacy: Visit types and traits in impls in type privacy lints)
- rust-lang#138581 (Abort in deadlock handler if we fail to get a query map)
- rust-lang#138776 (coverage: Separate span-extraction from unexpansion)
- rust-lang#138886 (Fix autofix for `self` and `self as …` in `unused_imports` lint)
- rust-lang#138924 (Reduce `kw::Empty` usage, part 3)
- rust-lang#138929 (Visitors track whether an assoc item is in a trait impl or an inherent impl)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 81e2275 into rust-lang:masterMar 25, 2025
@rustbotrustbot added this to the 1.87.0 milestone Mar 25, 2025
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Mar 25, 2025
Rollup merge of rust-lang#138317 - petrochenkov:libsearch3, r=compiler-errors
privacy: Visit types and traits in impls in type privacy lints
With one exception to avoid false positives.
Fixes the same issue as rust-lang#134176.
@petrochenkov
petrochenkov deleted the libsearch3 branch June 17, 2026 14:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant 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.

8 participants

@petrochenkov@rustbot@compiler-errors@bors@TaKO8Ki@rust-log-analyzer@estebank@cjgillot