Browse Source

Remove single instance of whitelist with allowlist

Given the context, allowlist is an appropriate replacement for the
single usage of whitelist in the code. We could call it exactly what
it is, a supported sub-objects name list, but that is getting verbose.
Justin Rogers 5 năm trước cách đây
mục cha
commit
3911e78138
3 tập tin đã thay đổi với 3 bổ sung3 xóa
  1. 1 1
      build/three.js
  2. 1 1
      build/three.module.js
  3. 1 1
      src/animation/PropertyBinding.js

+ 1 - 1
build/three.js

@@ -44276,7 +44276,7 @@
 
 				var objectName = results.nodeName.substring( lastDot + 1 );
 
-				// Object names must be checked against a whitelist. Otherwise, there
+				// Object names must be checked against an allowlist. Otherwise, there
 				// is no way to parse 'foo.bar.baz': 'baz' must be a property, but
 				// 'bar' could be the objectName, or part of a nodeName (which can
 				// include '.' characters).

+ 1 - 1
build/three.module.js

@@ -44244,7 +44244,7 @@ Object.assign( PropertyBinding, {
 
 			const objectName = results.nodeName.substring( lastDot + 1 );
 
-			// Object names must be checked against a whitelist. Otherwise, there
+			// Object names must be checked against an allowlist. Otherwise, there
 			// is no way to parse 'foo.bar.baz': 'baz' must be a property, but
 			// 'bar' could be the objectName, or part of a nodeName (which can
 			// include '.' characters).

+ 1 - 1
src/animation/PropertyBinding.js

@@ -173,7 +173,7 @@ Object.assign( PropertyBinding, {
 
 			const objectName = results.nodeName.substring( lastDot + 1 );
 
-			// Object names must be checked against a whitelist. Otherwise, there
+			// Object names must be checked against an allowlist. Otherwise, there
 			// is no way to parse 'foo.bar.baz': 'baz' must be a property, but
 			// 'bar' could be the objectName, or part of a nodeName (which can
 			// include '.' characters).