Change to allowlist/denylist. (#3546)

Author: lina128

e2e/integration_tests/setup_test.ts

@@ -62,7 +62,7 @@ function parseTags(tagsInput: string): string[] {
 }
 
 /**
- * Run Jasmine tests only for whitelisted tags.
+ * Run Jasmine tests only for allowlisted tags.
  */
 function setupTestFilters(tags: string[] = []) {
   const env = jasmine.getEnv();

scripts/find-affected-packages.js

@@ -21,7 +21,7 @@ const {readdirSync, statSync, writeFileSync} = require('fs');
 const {join} = require('path');
 const fs = require('fs');
 
-const filesWhitelistToTriggerBuild = [
+const filesAllowlistToTriggerBuild = [
   'cloudbuild.yml', 'package.json', 'tsconfig.json', 'tslint.json',
   'scripts/find-affected-packages.js', 'scripts/run-build.sh'
 ];
@@ -84,13 +84,13 @@ shell.cd('..');
 console.log();  // Break up the console for readability.
 
 let triggerAllBuilds = false;
-let whitelistDiffOutput = [];
-filesWhitelistToTriggerBuild.forEach(fileToTriggerBuild => {
+let allowlistDiffOutput = [];
+filesAllowlistToTriggerBuild.forEach(fileToTriggerBuild => {
   const diffOutput = diff(fileToTriggerBuild);
   if (diffOutput !== '') {
     console.log(fileToTriggerBuild, 'has changed. Triggering all builds.');
     triggerAllBuilds = true;
-    whitelistDiffOutput.push(diffOutput);
+    allowlistDiffOutput.push(diffOutput);
   }
 });
 
@@ -108,7 +108,7 @@ dirs.forEach(dir => {
 
   const shouldDiff = diffOutput !== '' || triggerAllBuilds;
   if (shouldDiff) {
-    const diffContents = whitelistDiffOutput.join('\n') + '\n' + diffOutput;
+    const diffContents = allowlistDiffOutput.join('\n') + '\n' + diffOutput;
     writeFileSync(join(dir, 'run-ci'), diffContents);
     triggeredBuilds.push(dir);
   }

tfjs-backend-wasm/tools/cpplint.py

@@ -3866,8 +3866,8 @@ def CheckTrailingSemicolon(filename, clean_lines, linenum, error):
 
   # Block bodies should not be followed by a semicolon.  Due to C++11
   # brace initialization, there are more places where semicolons are
-  # required than not, so we use a whitelist approach to check these
-  # rather than a blacklist.  These are the places where "};" should
+  # required than not, so we use an allowlist approach to check these
+  # rather than a denylist.  These are the places where "};" should
   # be replaced by just "}":
   # 1. Some flavor of block following closing parenthesis:
   #    for (;;) {};
@@ -3924,11 +3924,11 @@ def CheckTrailingSemicolon(filename, clean_lines, linenum, error):
     #  - INTERFACE_DEF
     #  - EXCLUSIVE_LOCKS_REQUIRED, SHARED_LOCKS_REQUIRED, LOCKS_EXCLUDED:
     #
-    # We implement a whitelist of safe macros instead of a blacklist of
+    # We implement an allowlist of safe macros instead of a denylist of
     # unsafe macros, even though the latter appears less frequently in
     # google code and would have been easier to implement.  This is because
-    # the downside for getting the whitelist wrong means some extra
-    # semicolons, while the downside for getting the blacklist wrong
+    # the downside for getting the allowlist wrong means some extra
+    # semicolons, while the downside for getting the denylist wrong
     # would result in compile errors.
     #
     # In addition to macros, we also don't want to warn on
@@ -5124,19 +5124,19 @@ def CheckForNonConstReference(filename, clean_lines, linenum,
   #
   # We also accept & in static_assert, which looks like a function but
   # it's actually a declaration expression.
-  whitelisted_functions = (r'(?:[sS]wap(?:<\w:+>)?|'
+  allowlisted_functions = (r'(?:[sS]wap(?:<\w:+>)?|'
                            r'operator\s*[<>][<>]|'
                            r'static_assert|COMPILE_ASSERT'
                            r')\s*\(')
-  if Search(whitelisted_functions, line):
+  if Search(allowlisted_functions, line):
     return
   elif not Search(r'\S+\([^)]*$', line):
-    # Don't see a whitelisted function on this line.  Actually we
+    # Don't see an allowlisted function on this line.  Actually we
     # didn't see any function name on this line, so this is likely a
     # multi-line parameter list.  Try a bit harder to catch this case.
     for i in xrange(2):
       if (linenum > i and
-          Search(whitelisted_functions, clean_lines.elided[linenum - i - 1])):
+          Search(allowlisted_functions, clean_lines.elided[linenum - i - 1])):
         return
 
   decls = ReplaceAll(r'{[^}]*}', ' ', line)  # exclude function body