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

cvv: fix small memleak in demo #1800

Merged
merged 1 commit into from
Sep 30, 2018
Merged

cvv: fix small memleak in demo #1800

merged 1 commit into from
Sep 30, 2018

Conversation

berak
Copy link
Contributor

@berak berak commented Sep 30, 2018

the pointer to resolution wasn't freed if we return early, also replacing it with simple width height params removes a lot of clutter.

alalek
alalek approved these changes Sep 30, 2018
View reviewed changes
Copy link
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Thank you!

@opencv-pushbot opencv-pushbot merged commit 502970b into opencv:3.4 Sep 30, 2018
opencv-pushbot pushed a commit that referenced this pull request Sep 30, 2018
@alalek
Merge pull request #1800 from berak:fix_cvv_demo
ff9d313
@berak berak deleted the fix_cvv_demo branch October 1, 2018 06:09
@alalek alalek mentioned this pull request Oct 2, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alalek alalek alalek approved these changes

Assignees

@alalek alalek

Labels
bug category: cvv category: samples
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@berak @alalek @opencv-pushbot