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

Sumary clarification #3824

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joaquinelio
Copy link
Member

the definitions seem ok (maybe a little redundant)
but the explanations look the same both cases

so I propose to clarify responsabilities,

  • weak ref, the engine does the optimization
  • finalization callback, you do additional cleaning related to the trashed object

...

mm...
I think is clearer in this comment than the lines I changed
Maybe an native English speaker could tell...

@javascript-translate-bot javascript-translate-bot added the review needed Review needed, please approve or request changes label Mar 15, 2025
@javascript-translate-bot javascript-translate-bot requested review from a team March 15, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review needed Review needed, please approve or request changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants