Clean up suggestion for methods that optionally return

[allsey87]

Just wanted to ask about/point out that most methods in this library denote an optional return type by conditionally returning null in the function body. There are the following exceptions though that seem to express the same concept but instead return undefined.

1. intersectLine from Plane
2. getObjectById from Object3D
3. getObjectByName from Object3D
4. getObjectByProperty from Object3D

Is this intentional? What is the rational behind this?

[allsey87]

Additional information: the cases where null is used to denote an optional return are as follows:

1. getHandler from LoadingManager
2. existingAction from AnimationMixer
3. intersectSphere from Ray
4. intersectPlane from Ray
5. intersectBox from Ray
6. intersectTriangle from Ray
7. getRenderTarget from WebGLRenderer
8. getCurrentRenderTarget from WebGLRenderer
9. getIndex from BufferGeometry
10. getObjectForDistance from LOD

[mrdoob]

You mean that we should unify all these to return null?

[allsey87]

I would suggest null over undefined. In my mind, null better represents the concept of nothing being returned.

It's just a minor nitpick. I came across it while doing the Rust bindings, but thought I would make a note of it in case you were interested in fixing these things.