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

std::auto_ptr will leak for array, using std::vector instead. #504

Merged
merged 1 commit into from Mar 16, 2015

Conversation

Projects
None yet
4 participants
@edwin0cheng
Contributor

edwin0cheng commented Feb 15, 2015

A little memory leak bug fix PR.

@njx

This comment has been minimized.

Show comment
Hide comment
@njx

njx Feb 17, 2015

Member

Good catch!

Member

njx commented Feb 17, 2015

Good catch!

nethip added a commit that referenced this pull request Mar 16, 2015

Merge pull request #504 from edwin0cheng/fix-win-leak
std::auto_ptr will leak for array, using std::vector instead.

@nethip nethip merged commit b31e5cb into adobe:master Mar 16, 2015

@nethip

This comment has been minimized.

Show comment
Hide comment
@nethip

nethip Mar 16, 2015

Contributor

Good change.

Contributor

nethip commented Mar 16, 2015

Good change.

nethip added a commit that referenced this pull request Dec 8, 2015

Merge pull request #504 from edwin0cheng/fix-win-leak
std::auto_ptr will leak for array, using std::vector instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment