Docs: Document nullable types according to JSDoc spec. (#30466)

* Docs: Document nullable types according to JSDoc spec.

* Clean up.

Author: Mugen87

examples/jsm/controls/ArcballControls.js

@@ -79,8 +79,8 @@ class ArcballControls extends Controls {
 	/**
 	 *
 	 * @param {Camera} camera Virtual camera used in the scene
-	 * @param {HTMLElement?} [domElement=null] Renderer's dom element
-	 * @param {Scene?} [scene=null] The scene to be rendered
+	 * @param {?HTMLElement} [domElement=null] Renderer's dom element
+	 * @param {?Scene} [scene=null] The scene to be rendered
 	 */
 	constructor( camera, domElement = null, scene = null ) {
 
@@ -1250,7 +1250,7 @@ class ArcballControls extends Controls {
 	 * Get the operation associated to mouse and key combination and returns the corresponding FSA state
 	 * @param {0|1|2} mouse Mouse button index (0, 1, 2)
 	 * @param {'CTRL'|'SHIFT'|null} key Keyboard modifier
-	 * @returns {STATE?} The FSA state obtained from the operation associated to mouse/keyboard combination
+	 * @returns {?STATE} The FSA state obtained from the operation associated to mouse/keyboard combination
 	 */
 	getOpStateFromAction( mouse, key ) {
 
@@ -2328,7 +2328,7 @@ class ArcballControls extends Controls {
 	 *
 	 * @param {Vector2} cursor Cursor coordinates in NDC
 	 * @param {Camera} camera Virtual camera
-	 * @returns {Vector3?} The point of intersection with the model, if exist, null otherwise
+	 * @returns {?Vector3} The point of intersection with the model, if exist, null otherwise
 	 */
 	unprojectOnObj( cursor, camera ) {
 

examples/jsm/exporters/GLTFExporter.js

@@ -987,11 +987,11 @@ class GLTFWriter {
 
 	/**
 	 * Process and generate a BufferView
-	 * @param  {BufferAttribute} attribute
-	 * @param  {number} componentType
-	 * @param  {number} start
-	 * @param  {number} count
-	 * @param  {number?} target Target usage of the BufferView
+	 * @param {BufferAttribute} attribute
+	 * @param {number} componentType
+	 * @param {number} start
+	 * @param {number} count
+	 * @param {?number} target Target usage of the BufferView
 	 * @return {Object}
 	 */
 	processBufferView( attribute, componentType, start, count, target ) {
@@ -1179,11 +1179,11 @@ class GLTFWriter {
 
 	/**
 	 * Process attribute to generate an accessor
-	 * @param  {BufferAttribute} attribute Attribute to process
-	 * @param  {THREE.BufferGeometry?} geometry Geometry used for truncated draw range
-	 * @param  {number} [start=0]
-	 * @param  {number} [count=Infinity]
-	 * @return {number?} Index of the processed accessor on the "accessors" array
+	 * @param {BufferAttribute} attribute Attribute to process
+	 * @param {?BufferGeometry} geometry Geometry used for truncated draw range
+	 * @param {number} [start=0]
+	 * @param {number} [count=Infinity]
+	 * @return {?number} Index of the processed accessor on the "accessors" array
 	 */
 	processAccessor( attribute, geometry, start, count ) {
 

examples/jsm/loaders/KTX2Loader.js

@@ -324,7 +324,7 @@ class KTX2Loader extends Loader {
 
 	/**
 	 * @param {ArrayBuffer} buffer
-	 * @param {object?} config
+	 * @param {?Object} config
 	 * @return {Promise<CompressedTexture|CompressedArrayTexture|DataTexture|Data3DTexture>}
 	 */
 	async _createTexture( buffer, config = {} ) {

examples/jsm/math/OBB.js

@@ -322,7 +322,7 @@ class OBB {
 	*
 	* @param {Ray} ray
 	* @param {Vector3} result
-	* @return {Vector3?}
+	* @return {?Vector3}
 	*/
 	intersectRay( ray, result ) {
 

examples/jsm/tsl/display/BloomNode.js

@@ -154,15 +154,15 @@ class BloomNode extends TempNode {
 		 * The material for the composite pass.
 		 *
 		 * @private
-		 * @type {NodeMaterial?}
+		 * @type {?NodeMaterial}
 		 */
 		this._compositeMaterial = null;
 
 		/**
 		 * The material for the luminance pass.
 		 *
 		 * @private
-		 * @type {NodeMaterial?}
+		 * @type {?NodeMaterial}
 		 */
 		this._highPassFilterMaterial = null;
 

examples/jsm/tsl/display/DenoiseNode.js

@@ -25,7 +25,7 @@ class DenoiseNode extends TempNode {
 	 *
 	 * @param {TextureNode} textureNode - The texture node that represents the input of the effect (e.g. AO).
 	 * @param {Node<float>} depthNode - A node that represents the scene's depth.
-	 * @param {Node<vec3>?} normalNode - A node that represents the scene's normals.
+	 * @param {?Node<vec3>} normalNode - A node that represents the scene's normals.
 	 * @param {Camera} camera - The camera the scene is rendered with.
 	 */
 	constructor( textureNode, depthNode, normalNode, camera ) {
@@ -51,7 +51,7 @@ class DenoiseNode extends TempNode {
 		 * constructor (because MRT is not available), normals can be automatically
 		 * reconstructed from depth values in the shader.
 		 *
-		 * @type {Node<vec3>?}
+		 * @type {?Node<vec3>}
 		 */
 		this.normalNode = normalNode;
 
@@ -325,7 +325,7 @@ function generateDefaultNoise( size = 64 ) {
  * @function
  * @param {Node} node - The node that represents the input of the effect (e.g. AO).
  * @param {Node<float>} depthNode - A node that represents the scene's depth.
- * @param {Node<vec3>?} normalNode - A node that represents the scene's normals.
+ * @param {?Node<vec3>} normalNode - A node that represents the scene's normals.
  * @param {Camera} camera - The camera the scene is rendered with.
  * @returns {DenoiseNode}
  */

examples/jsm/tsl/display/FilmNode.js

@@ -18,8 +18,8 @@ class FilmNode extends TempNode {
 	 * Constructs a new film node.
 	 *
 	 * @param {Node} inputNode - The node that represents the input of the effect.
-	 * @param {Node<float>?} [intensityNode=null] - A node that represents the effect's intensity.
-	 * @param {Node<vec2>?} [uvNode=null] - A node that allows to pass custom (e.g. animated) uv data.
+	 * @param {?Node<float>} [intensityNode=null] - A node that represents the effect's intensity.
+	 * @param {?Node<vec2>} [uvNode=null] - A node that allows to pass custom (e.g. animated) uv data.
 	 */
 	constructor( inputNode, intensityNode = null, uvNode = null ) {
 
@@ -35,14 +35,16 @@ class FilmNode extends TempNode {
 		/**
 		 * A node that represents the effect's intensity.
 		 *
-		 * @type {Node<float>}
+		 * @type {?Node<float>}
+		 * @default null
 		 */
 		this.intensityNode = intensityNode;
 
 		/**
 		 * A node that allows to pass custom (e.g. animated) uv data.
 		 *
-		 * @type {Node<vec2>}
+		 * @type {?Node<vec2>}
+		 * @default null
 		 */
 		this.uvNode = uvNode;
 
@@ -91,8 +93,8 @@ export default FilmNode;
  * @tsl
  * @function
  * @param {Node<vec4>} inputNode - The node that represents the input of the effect.
- * @param {Node<float>?} [intensityNode=null] - A node that represents the effect's intensity.
- * @param {Node<vec2>?} [uvNode=null] - A node that allows to pass custom (e.g. animated) uv data.
+ * @param {?Node<float>} [intensityNode=null] - A node that represents the effect's intensity.
+ * @param {?Node<vec2>} [uvNode=null] - A node that allows to pass custom (e.g. animated) uv data.
  * @returns {FilmNode}
  */
 export const film = /*@__PURE__*/ nodeProxy( FilmNode );

examples/jsm/tsl/display/GTAONode.js

@@ -42,7 +42,7 @@ class GTAONode extends TempNode {
 	 * Constructs a new GTAO node.
 	 *
 	 * @param {Node<float>} depthNode - A node that represents the scene's depth.
-	 * @param {Node<vec3>?} normalNode - A node that represents the scene's normals.
+	 * @param {?Node<vec3>} normalNode - A node that represents the scene's normals.
 	 * @param {Camera} camera - The camera the scene is rendered with.
 	 */
 	constructor( depthNode, normalNode, camera ) {
@@ -61,7 +61,7 @@ class GTAONode extends TempNode {
 		 * constructor (because MRT is not available), normals can be automatically
 		 * reconstructed from depth values in the shader.
 		 *
-		 * @type {Node<vec3>?}
+		 * @type {?Node<vec3>}
 		 */
 		this.normalNode = normalNode;
 
@@ -514,7 +514,7 @@ function generateMagicSquare( size ) {
  * @tsl
  * @function
  * @param {Node<float>} depthNode - A node that represents the scene's depth.
- * @param {Node<vec3>?} normalNode - A node that represents the scene's normals.
+ * @param {?Node<vec3>} normalNode - A node that represents the scene's normals.
  * @param {Camera} camera - The camera the scene is rendered with.
  * @returns {GTAONode}
  */

examples/jsm/tsl/display/SSAAPassNode.js

@@ -88,7 +88,8 @@ class SSAAPassNode extends PassNode {
 		 * Reference to the internal render target that holds the current sample.
 		 *
 		 * @private
-		 * @type {RenderTarget?}
+		 * @type {?RenderTarget}
+		 * @default null
 		 */
 		this._sampleRenderTarget = null;
 

examples/jsm/tsl/display/TRAAPassNode.js

@@ -90,15 +90,17 @@ class TRAAPassNode extends PassNode {
 		 * The render target that holds the current sample.
 		 *
 		 * @private
-		 * @type {RenderTarget?}
+		 * @type {?RenderTarget}
+		 * @default null
 		 */
 		this._sampleRenderTarget = null;
 
 		/**
 		 * The render target that represents the history of frame data.
 		 *
 		 * @private
-		 * @type {RenderTarget?}
+		 * @type {?RenderTarget}
+		 * @default null
 		 */
 		this._historyRenderTarget = null;