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

WIP: Emit a warning in case of possible memory leaks #428

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open

WIP: Emit a warning in case of possible memory leaks #428

wants to merge 27 commits into from

Commits on Feb 2, 2022

  1. add a compiler pass which tries to detect suspicious cases of possibl… 

    …e memory leaks and warns the user
    @antocuni
    antocuni committed Feb 2, 2022
    Configuration menu
    Copy the full SHA
    69217d9 View commit details
    Browse the repository at this point in the history

Commits on Feb 3, 2022

  1. detect calls to BufferPointer.free()

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    bc38bb3 View commit details
    Browse the repository at this point in the history

  2. kill the AutoFreeBuffers pass

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    d250ecf View commit details
    Browse the repository at this point in the history

  3. fix flake8

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    be49a3d View commit details
    Browse the repository at this point in the history

  4. fix missing import

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    3650578 View commit details
    Browse the repository at this point in the history

  5. flake8 again

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    b7d61dd View commit details
    Browse the repository at this point in the history

  6. make sure to detect a call to free_buffer even if it's imported locally

    @antocuni
    antocuni committed Feb 3, 2022
    Configuration menu
    Copy the full SHA
    890629b View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2022

  1. make it possible to silence the memory leaks warnings

    @antocuni
    antocuni committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    50ff854 View commit details
    Browse the repository at this point in the history

  2. git merge upstream/master

    @antocuni
    antocuni committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    8cbac5d View commit details
    Browse the repository at this point in the history

  3. use another approach to propagate the disable_leak_warning function, … 

    …because the old one didn't work with @omnisci
    @antocuni
    antocuni committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    3bbd923 View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2022

  1. Include the function location to the missing free warning message.

    @pearu
    pearu committed Feb 5, 2022
    Configuration menu
    Copy the full SHA
    aac9aab View commit details
    Browse the repository at this point in the history

  2. Fix test_no_warnings_decorator

    @pearu
    pearu committed Feb 5, 2022
    Configuration menu
    Copy the full SHA
    5fc872e View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2022

  1. change disable_leak_warnings into on_missing_free, which now can be '… 

    …warn', 'fail' or 'ignore'
    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    b4707a5 View commit details
    Browse the repository at this point in the history

  2. make sure that test_omnisci_array and test_omnisci_array_functions do… 

    …n't emit MissingFree warnings
    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    957a958 View commit details
    Browse the repository at this point in the history

  3. fix warnings

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    ddc9e48 View commit details
    Browse the repository at this point in the history

  4. fix MissingFree warnings/errors in test_omnisci_array_methods

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    1ad3b1c View commit details
    Browse the repository at this point in the history

  5. fix MissingFree warnings/errors in test_omnisci_array_operators and t… 

    …est_omnisci_caller
    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    482d8ef View commit details
    Browse the repository at this point in the history

  6. make sure to report the name of the function which actually contains … 

    …the missing free
    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    2df5c09 View commit details
    Browse the repository at this point in the history

  7. fix MissingFree warnings/errors in test_omnisci_text

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    8c2269c View commit details
    Browse the repository at this point in the history

  8. fix flake8

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    8cbcf90 View commit details
    Browse the repository at this point in the history

  9. Merge branch 'antocuni/detect-missing-free' of github.com:antocuni/rb… 

    …c into antocuni/detect-missing-free
    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    7038a31 View commit details
    Browse the repository at this point in the history

  10. TEMP: disable fast fail

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    fd6836c View commit details
    Browse the repository at this point in the history

  11. TEMP: disable fast fail

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    561df76 View commit details
    Browse the repository at this point in the history

  12. try this

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    321d18e View commit details
    Browse the repository at this point in the history

  13. fix all the memory leaks in test_omnisci_array_operators.py

    @antocuni
    antocuni committed Feb 7, 2022
    Configuration menu
    Copy the full SHA
    a943884 View commit details
    Browse the repository at this point in the history

Commits on Feb 8, 2022

  1. temporarily disable all the .free()

    @antocuni
    antocuni committed Feb 8, 2022
    Configuration menu
    Copy the full SHA
    055a060 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2022

  1. try to run only test_abs

    @antocuni
    antocuni committed Feb 9, 2022
    Configuration menu
    Copy the full SHA
    82e54bd View commit details
    Browse the repository at this point in the history