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 leak of data.devices in cups_enum_dests #317

Merged
merged 1 commit into from
Jan 3, 2022
Merged

Fix leak of data.devices in cups_enum_dests #317

merged 1 commit into from
Jan 3, 2022

Conversation

jwatt
Copy link
Contributor

@jwatt jwatt commented Jan 1, 2022

michaelrsweet
michaelrsweet approved these changes Jan 1, 2022
View reviewed changes
Copy link
Member

@michaelrsweet michaelrsweet left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@jwatt
Copy link
Contributor Author

jwatt commented Jan 1, 2022

Thanks for all your work on CUPS over the years. Hugely appreciated!

@zdohnal zdohnal self-assigned this Jan 3, 2022
@zdohnal zdohnal added bug Something isn't working priority-medium labels Jan 3, 2022
@zdohnal zdohnal added this to the v2.4.1 milestone Jan 3, 2022
@zdohnal
Copy link
Member

zdohnal commented Jan 3, 2022

Hi @jwatt ,

thank you for the PR! I'm merging it now.

@zdohnal zdohnal merged commit cc76ffb into OpenPrinting:master Jan 3, 2022
@jwatt
Copy link
Contributor Author

jwatt commented Jan 3, 2022

Thanks @zdohnal, much appreciated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michaelrsweet michaelrsweet michaelrsweet approved these changes

Assignees

@zdohnal zdohnal

Labels
bug Something isn't working priority-medium
Projects
None yet
Milestone
v2.4.1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jwatt @zdohnal @michaelrsweet