[SUGGESTION] Cache vertex-count of poly body for performance.

In common ,

var len = arr.length;
for (var i=0;i<len;i++){ ...}

is faster than

for (var i=0;i<arr.length;i++){ ...}

Of course , In most cases , it’s not the performance hotspot . But in Physics Engine , it will be called so many times. And I see there are many for (var i = 0; i < vertices.length; i++) { will be called in one tick.

So I suggest that add a property vertexCount to body for caching the vertex-count. (like box2d)

[photonstorm]

@bchevalier agreed, stuff like caching the array length is pointless with modern VMs and has been for years (unless you’re manipulating the array size within the loop of course!)

[liabru]

Agreed with the above and in general micro-optimisations tend to be bad practice in most cases, unless the effect can be proven to be consistently large. I much prefer algorithmic improvements! Thanks for the suggestion though.