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

Fix none_dealloc bug #1506

Merged
merged 8 commits into from
Aug 31, 2023
Merged

Fix none_dealloc bug #1506

merged 8 commits into from
Aug 31, 2023

Conversation

EmilyBourne
Copy link
Member

@EmilyBourne EmilyBourne commented Aug 30, 2023

Increment the counter on Py_None before returning None from functions which return nothing. This avoids unexpected deallocation, which would result in a fatal Python error.

cc: @pyccel/psydac-dev

@pyccel-bot
Copy link

pyccel-bot bot commented Aug 30, 2023

Hello again! Thank you for this new pull request 🤩.

Here is your checklist. Please tick items off when you have completed them or determined that they are not necessary for this pull request:

  • Write a clear PR description
  • Add tests to check your code works as expected
  • Update documentation if necessary
  • Update Changelog
  • Ensure any relevant issues are linked
  • Ensure new tests are passing

@EmilyBourne
Copy link
Member Author

/bot run linux

@EmilyBourne
Copy link
Member Author

/bot run linux

@yguclu
Copy link
Member

yguclu commented Aug 30, 2023

Psydac's PR #326 shows the bug and proves that this PR fixes it.

@EmilyBourne EmilyBourne marked this pull request as ready for review August 31, 2023 04:59
Copy link

@pyccel-bot pyccel-bot bot left a comment

Choose a reason for hiding this comment

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

Good job ! Your PR is using all the code it added/changed.

@github-actions github-actions bot marked this pull request as draft August 31, 2023 05:01
@pyccel-bot
Copy link

pyccel-bot bot commented Aug 31, 2023

Unfortunately your PR is not passing the tests so it is not quite ready for review yet. Let me know when it is fixed with /bot mark as ready.

@EmilyBourne EmilyBourne marked this pull request as ready for review August 31, 2023 05:33
@pyccel-bot pyccel-bot bot added Ready_to_merge Approved by senior developer. Ready for final approval and merge and removed Ready_to_merge Approved by senior developer. Ready for final approval and merge labels Aug 31, 2023
@yguclu yguclu merged commit 2fb0e0d into devel Aug 31, 2023
11 checks passed
@yguclu yguclu deleted the none_dealloc_bug branch August 31, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready_to_merge Approved by senior developer. Ready for final approval and merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants