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

Performance: don't create a double clone in point #1251

Merged

Fuzzyma merged 3 commits into svgdotjs:master from Glandos:patch-3

Jan 27, 2022

+1 −1

Contributor

Glandos commented Jan 27, 2022 •

edited

The Point instance is already created, so don't create a new one.

Edit: also don't clone the newly created Matrix from getScreenCTM


          don't create a double clone in point

ca975b6

Glandos changed the title ~~Performance don't create a double clone in point~~ Performance: don't create a double clone in point

Glandos added 2 commits

January 27, 2022 22:13


          fix typo

66a2f4a


          also don't clone matrix

f57d64a

Fuzzyma merged commit 53af7dd into svgdotjs:master

Member

Fuzzyma commented Jan 27, 2022

Good catch!

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