Make vector class chainable

[mrdoob]

Make vector class chainable

[mrdoob]

Yeah. I've been considering it. I just wondered if it affects performance somehow... ?

[mrdoob]

If you're using Chrome you can just do this

console.time("test1");

for ( var i = 0; i < 1000000; i++ ) {
    x = v.addSelf( new Vector( 1, 2, 3 ) );
}

console.timeEnd("test1");

... and check the results in the console.

[bnolan]

Ok - i ran this a bunch of times:

console.time("vanilla");
for ( var i = 0; i < 10000000; i++ ) {
    v1.addSelf( new THREE.Vector3( 1, 2, 3 ) );
}
console.timeEnd("vanilla");

console.time("chainable");
for ( var i = 0; i < 10000000; i++ ) {
    x = v1.addSelfChainable  ( new THREE.Vector3( 1, 2, 3 ) );
}
console.timeEnd("chainable");

Output:

  vanilla: 734ms
  chainable: 717ms
  vanilla: 728ms
  chainable: 716ms
  vanilla: 725ms
  chainable: 714ms

It looks like the chained version is faster (in chrome).

[mrdoob]

Sounds good! I'll merge then :)