Skip to content

ClearWeak api is deprecated use ClearWeak<void> instead#6669

Merged
tewaro merged 1 commit intomainfrom
tewaro/remove-deprecated-ClearWeak-api
Apr 25, 2026
Merged

ClearWeak api is deprecated use ClearWeak<void> instead#6669
tewaro merged 1 commit intomainfrom
tewaro/remove-deprecated-ClearWeak-api

Conversation

@tewaro
Copy link
Copy Markdown
Contributor

@tewaro tewaro commented Apr 25, 2026

No description provided.

@tewaro tewaro requested review from a team as code owners April 25, 2026 05:24
@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented Apr 25, 2026

All 5 ClearWeak call sites in the codebase have been updated. There are no remaining bare ClearWeak() calls. The change is purely mechanical — replacing a deprecated non-templated ClearWeak() with the non-deprecated ClearWeak<void>() overload. The semantics are identical (both clear the weak state of the handle). No behavioral change, no backward compatibility concern.

LGTM

github run

@tewaro tewaro enabled auto-merge (rebase) April 25, 2026 22:10
@tewaro tewaro merged commit 4281290 into main Apr 25, 2026
45 of 49 checks passed
@tewaro tewaro deleted the tewaro/remove-deprecated-ClearWeak-api branch April 25, 2026 22:43
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