-
Notifications
You must be signed in to change notification settings - Fork 78
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
Some general Vector4 optimizations #295
Conversation
What is going on with the code coverage? That is a steep drop. |
That is very strange... Are you sure the coveralls check ran properly? |
Just reran for giggles. It seems coverage fell off a cliff before this change. Maybe rebase on master? |
It has the same coverage percentage on master now 🤷♂️ |
78fbb96
to
102efd5
Compare
Since the
Vector4.zero
constructor was already a normal named constructor it was already possible to extend the class.Converts all factory constructors (exceptVector4.random
) to named constructors that directly sets the storage instead of calling intermediary methods to do it.length2
,dot
,isInfinite
andisNaN
.reflect
,relativeError
andabsoluteError
so that they don't create extraVector4
objects._v4storage
from 1 to 0 so that it is consistent across the class and guarantees that bounds checking is only done once.