Skip to content

Don't eagerly remove golang gc references in Destroy#174

Merged
linxGnu merged 2 commits intolinxGnu:masterfrom
yihuang:master
Mar 25, 2025
Merged

Don't eagerly remove golang gc references in Destroy#174
linxGnu merged 2 commits intolinxGnu:masterfrom
yihuang:master

Conversation

@yihuang
Copy link
Contributor

@yihuang yihuang commented Feb 24, 2025

  1. it's not necessary with golang gc.
  2. user may want to keep reference it indirectly

ref: #172

1. it's not necessary
2. user may want to keep reference it indirectly

ref: linxGnu#172
@linxGnu linxGnu changed the title don't eagerly remove golang gc references in Destroy Don't eagerly remove golang gc references in Destroy Mar 3, 2025
Copy link
Owner

@linxGnu linxGnu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@linxGnu linxGnu merged commit 9cdf8b3 into linxGnu:master Mar 25, 2025
2 checks passed
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