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.

Sign up for GitHub

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

Jump to bottom

fix: Fix the memory leak #1119

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

fix: Fix the memory leak #1119

wants to merge 1 commit into from

Commits on May 2, 2023

  1. fix: Fix the memory leak 

    Closes overblog#721.

Follow up of [my
comment](overblog#721 (comment)).

I cannot investigate further, but my guess is that exceptions are not
cheap objects. When creating an exception you also capture the context
and stack trace which means that even if the extension is unsued:

- it slows down the program because instantiating it is not cheap
- may prevent some objects to be garbage collected because referenced in
  the exception
    @theofidry
    theofidry committed May 2, 2023
    Configuration menu
    Copy the full SHA
    2fd33ed View commit details
    Browse the repository at this point in the history