Skip to content

One-line patch to remove an unused variable in PAUSE::may_overwrite_file#506

Open
ap wants to merge 1 commit into
andk:masterfrom
ap-contrib:dead-code/may_overwrite_file
Open

One-line patch to remove an unused variable in PAUSE::may_overwrite_file#506
ap wants to merge 1 commit into
andk:masterfrom
ap-contrib:dead-code/may_overwrite_file

Conversation

@ap

@ap ap commented May 6, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@rjbs

rjbs commented May 6, 2024

Copy link
Copy Markdown
Collaborator

I will check this out after a larger branch that touches some of this code lands. I'm sure this is fine, I just want to avoid a merge conflict.

@rspier rspier requested a review from rjbs May 12, 2024 04:22
@ap ap force-pushed the dead-code/may_overwrite_file branch from 19fa101 to 30ceb5f Compare January 2, 2025 15:14
@ap

ap commented Jan 2, 2025

Copy link
Copy Markdown
Contributor Author

Has the larger branch landed?

@ap ap force-pushed the dead-code/may_overwrite_file branch from 30ceb5f to d722445 Compare March 30, 2026 19:37
@ap

ap commented Mar 30, 2026

Copy link
Copy Markdown
Contributor Author

Ping?

This patch does nothing other than remove a narrowly scoped lexical variable that is referenced only in its declaration.

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.

2 participants