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

Skip use_self inside macro expansions of a impl Self block #13128

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Jul 19, 2024

changelog: [use_self] Skip if inside macro expansions of a impl Self block
Fixes #13092.
r? Alexendoo

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jul 19, 2024
@tesuji tesuji force-pushed the fix_thread-local_use_self branch 5 times, most recently from d02153a to 5d81633 Compare July 21, 2024 06:16
clippy_lints/src/use_self.rs Outdated Show resolved Hide resolved
clippy_lints/src/use_self.rs Outdated Show resolved Hide resolved
@tesuji tesuji force-pushed the fix_thread-local_use_self branch from 5d81633 to 135eaf1 Compare July 24, 2024 21:05
@tesuji tesuji changed the title Skip use_self inside thread_local! of a impl Self block Skip use_self inside macro expansions of a impl Self block Jul 24, 2024
@tesuji tesuji force-pushed the fix_thread-local_use_self branch from 135eaf1 to d3d894d Compare July 24, 2024 21:16
@tesuji
Copy link
Contributor Author

tesuji commented Jul 24, 2024

Thank you for your insightful reviews!
Should be ready for another look.

clippy_lints/src/use_self.rs Outdated Show resolved Hide resolved
clippy_lints/src/use_self.rs Outdated Show resolved Hide resolved
@tesuji tesuji force-pushed the fix_thread-local_use_self branch 2 times, most recently from dc46bff to 16f5ed8 Compare September 5, 2024 23:32
co-authored-by: Alex Macleod <alex@macleod.io>
@tesuji tesuji force-pushed the fix_thread-local_use_self branch from 16f5ed8 to 451c584 Compare September 6, 2024 18:16
@bors
Copy link
Contributor

bors commented Oct 18, 2024

☔ The latest upstream changes (presumably #13561) made this pull request unmergeable. Please resolve the merge conflicts.

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.

use_self false positive on outer items
4 participants