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

Free pgsql results on destruct #5908

Merged
merged 1 commit into from
Feb 7, 2023

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Feb 6, 2023

Q A
Type improvement
Fixed issues Follows #5880

Summary

Freeing wrapped ext-pgsql resources in destructors has proven useful so far, so let's also free results when our result object is destructed.

@greg0ire
Copy link
Member

greg0ire commented Feb 7, 2023

continuous-integration/appveyor/branch

Weird that the "branch" job triggers on a PR 🤔 . Let's keep an eye on this.

@derrabus derrabus added this to the 3.6.0 milestone Feb 7, 2023
@derrabus derrabus merged commit b5bf7dc into doctrine:3.6.x Feb 7, 2023
@derrabus derrabus deleted the bugfix/free-result-on-destruct branch February 7, 2023 09:50
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants