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

POTCanvas and clampCanvas array #11568

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
2 participants
@tommytee
Copy link

tommytee commented Jun 20, 2017

Use a single canvas for makePowerOfTwo and an array of canvases for clampToMaxSize. This stops the memory leak.

Fixes #11378

@Mugen87

This comment has been minimized.

Copy link
Collaborator

Mugen87 commented Feb 1, 2018

#13201 was merged instead. The PR does only change makePowerOfTwo(). The average call frequency of clampToMaxSize() is much lower so it's not necessary to change the function now.

@Mugen87 Mugen87 closed this Feb 1, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.