Remove optionalTarget - part 4 #13518

WestLangley · 2018-03-07T19:00:51Z

As proposed in #12231.

mrdoob · 2018-03-07T20:58:33Z

Thanks thanks!

aardgoose · 2018-03-08T11:03:06Z

src/math/Ray.js

+
+		if ( target === undefined ) {
+
+			console.warn( 'THREE.Ray: .closestPointToPoint() target is now required' );


@WestLangley

This triggers in the Points.raycast() testpoint function.

(I haven't looked at other objects raycast methods)

Thanks @aardgoose Trying to track these down...

Remove optionalTarget

7da047f

mrdoob added this to the r91 milestone Mar 7, 2018

mrdoob merged commit 14e0532 into mrdoob:dev Mar 7, 2018

WestLangley deleted the dev-optTgt_4 branch March 7, 2018 21:09

aardgoose reviewed Mar 8, 2018

View reviewed changes

Provide feedback


		if ( target === undefined ) {

		console.warn( 'THREE.Ray: .closestPointToPoint() target is now required' );