Skip to content

Fix tooltip reference cycles#52853

Merged
rtfeldman merged 6 commits intomainfrom
AI-120/investigate-reference-count-cycle-on-tooltips
Apr 1, 2026
Merged

Fix tooltip reference cycles#52853
rtfeldman merged 6 commits intomainfrom
AI-120/investigate-reference-count-cycle-on-tooltips

Conversation

@rtfeldman
Copy link
Copy Markdown
Contributor

@rtfeldman rtfeldman commented Mar 31, 2026

Summary

  • break tooltip back-references from stored callbacks and tasks with weak handles
  • keep the tooltip controller as the sole strong owner of tooltip lifecycle state
  • add a regression test that exercises the visible-tooltip ownership graph directly

Testing

  • cargo test -p gpui tooltip_is_released_when_its_owner_disappears --lib

Closes AI-120

Release Notes:

  • Fixed a tooltip memory leak.

@rtfeldman rtfeldman self-assigned this Mar 31, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Mar 31, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Mar 31, 2026
@rtfeldman rtfeldman marked this pull request as ready for review March 31, 2026 21:22
@rtfeldman rtfeldman requested a review from Anthony-Eid March 31, 2026 21:22
@rtfeldman rtfeldman merged commit 0638897 into main Apr 1, 2026
32 checks passed
@rtfeldman rtfeldman deleted the AI-120/investigate-reference-count-cycle-on-tooltips branch April 1, 2026 16:35
KlausUllrich pushed a commit to KlausUllrich/zed that referenced this pull request Apr 8, 2026
## Summary
- break tooltip back-references from stored callbacks and tasks with
weak handles
- keep the tooltip controller as the sole strong owner of tooltip
lifecycle state
- add a regression test that exercises the visible-tooltip ownership
graph directly

## Testing
- cargo test -p gpui tooltip_is_released_when_its_owner_disappears --lib

Closes AI-120

Release Notes:

- Fixed a tooltip memory leak.
piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
## Summary
- break tooltip back-references from stored callbacks and tasks with
weak handles
- keep the tooltip controller as the sole strong owner of tooltip
lifecycle state
- add a regression test that exercises the visible-tooltip ownership
graph directly

## Testing
- cargo test -p gpui tooltip_is_released_when_its_owner_disappears --lib

Closes AI-120

Release Notes:

- Fixed a tooltip memory leak.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants