No public description

PiperOrigin-RevId: 852855924

Author: Closure Team

contrib/externs/underscore-1.3.1.js

@@ -18,7 +18,7 @@
  * @fileoverview Externs for Underscore 1.3.1.
  *
  * TODO: Wrapper objects.
- * TODO: _.bind - for some reason this plays up in practice.
+ * TODO: user - for some reason this plays up in practice.
  *
  * @see http://documentcloud.github.com/underscore/
  * @externs

contrib/externs/underscore-1.5.2.js

@@ -18,7 +18,7 @@
  * @fileoverview Externs for Underscore 1.5.2.
  *
  * TODO: Wrapper objects.
- * TODO: _.bind - for some reason this plays up in practice.
+ * TODO: user - for some reason this plays up in practice.
  *
  * @see http://documentcloud.github.com/underscore/
  * @externs

src/com/google/javascript/jscomp/AstFactory.java

@@ -597,7 +597,7 @@ Node createName(String name, Type type) {
     if (lifeCycleStage.isNormalized() && name.startsWith("$jscomp")) {
       // $jscomp will always be a constant and needs to be marked that way to satisfy
       // the normalization invariants.
-      // TODO: b/322009741 - Stop depending on lifeCycleStage.isNormalized() and "$jscomp" prefix to
+      // TODO: user - Stop depending on lifeCycleStage.isNormalized() and "$jscomp" prefix to
       // decide constness. The callers must explicitly use `createConstantName` is they need a NAME
       // node that's set with IS_CONSTANT_NAME prop.
       result.putBooleanProp(Node.IS_CONSTANT_NAME, true);
@@ -614,7 +614,7 @@ Node createConstantName(String name, Type type) {
     Node result = IR.name(name);
     setJSTypeOrColor(type, result);
     if (lifeCycleStage.isNormalized()) {
-      // TODO: b/322009741 - Stop depending on lifeCycleStage.isNormalized() to decide constness
+      // TODO: user - Stop depending on lifeCycleStage.isNormalized() to decide constness
       result.putBooleanProp(Node.IS_CONSTANT_NAME, true);
     }
     return result;

src/com/google/javascript/jscomp/AstValidator.java

@@ -64,7 +64,7 @@ enum TypeInfoValidation {
   /** Validate that a SCRIPT's FeatureSet property includes all features if this is enabled. */
   private final boolean isScriptFeatureValidationEnabled;
 
-  // TODO: varomodt - make this the default.
+  // TODO: user - make this the default.
   /** Validate that all required inlinings were performed. */
   private final boolean shouldValidateRequiredInlinings;
 
@@ -454,7 +454,7 @@ private void validatePseudoExpression(Node n, Token... allowedPseudoexpressions)
   private void validateTypeInformation(Node n) {
     if (n.getIsInClosureUnawareSubtree()) {
       // We don't expect closure-unaware code to have type information.
-      // TODO: b/321233583 - Maybe this should be a separate validation step, to ensure that nothing
+      // TODO: user - Maybe this should be a separate validation step, to ensure that nothing
       // tries to infer type information where we are mostly unsure of it?
 
       return;
@@ -2128,7 +2128,7 @@ private void validateFeature(Feature feature, Node n) {
       // Closure-unaware code is currently hidden from transpilation passes in the compiler when
       // options.setClosureUnawareMode(Mode.PASS_THROUGH) is enabled, so the AST
       // might still contain features that should have been transpiled.
-      // TODO: b/321233583 - Once JSCompiler always transpiles closure-unaware code, remove this
+      // TODO: user - Once JSCompiler always transpiles closure-unaware code, remove this
       // early-return to validate that all closure-unaware code is transpiled properly.
       return;
     }

src/com/google/javascript/jscomp/Compiler.java

@@ -1189,7 +1189,7 @@ private void performPostCompilationTasksInternal() {
       runValidityCheck();
     }
 
-    // TODO: b/421971366 - try removing this. We shouldn't need it as the code printer
+    // TODO: user - try removing this. We shouldn't need it as the code printer
     // will already print out @closureUnaware contents.
     ManageClosureUnawareCode.unwrap(this).process(externsRoot, jsRoot);
 

src/com/google/javascript/jscomp/CompilerOptions.java

@@ -2562,7 +2562,7 @@ public void setReplaceStringsPlaceholderToken(String replaceStringsPlaceholderTo
    * <p>PASS_THROUGH: they are entirely hidden from the compiler, as if they were an evaled string.
    *
    * <p>SIMPLE_OPTIMIZATIONS_AND_TRANSPILATION: This is *experimental* - we want to support some
-   * minimal transpilation & safe optimizations. TODO: b/421971366 - implement this flag.
+   * minimal transpilation & safe optimizations. TODO: user - implement this flag.
    */
   public enum ClosureUnawareMode {
     PASS_THROUGH,
@@ -2810,7 +2810,7 @@ public boolean getInstrumentAsyncContext() {
    * Reflect.construct}. This is correct, but has the disadvantage of pulling in more helper
    * utilities into the compiled output and of the transpiled output being slightly larger.
    *
-   * <p>TODO: b/36789413 - always enable the "safe" transpilation.
+   * <p>TODO: user - always enable the "safe" transpilation.
    */
   public enum Es6SubclassTranspilation {
     CONCISE_UNSAFE,

src/com/google/javascript/jscomp/DefaultPassConfig.java

@@ -588,7 +588,7 @@ protected PassListBuilder getFinalizations() {
     }
 
     if (options.collapseAnonymousFunctions) {
-      // TODO: b/197349249 - Maybe we should just move this pass after denormalization. It seems
+      // TODO: user - Maybe we should just move this pass after denormalization. It seems
       // weird to convert from function expression to function declaration while we're still
       // supposed to be in a normalized state. But it requires testing as perhaps some optimizations
       // in that range will get affected if we skip this rewriting.
@@ -1018,7 +1018,7 @@ private PassListBuilder getLateOptimizationPasses() {
       // above or below from accidentally becoming part of the loop.
       passes.maybeAdd(createEmptyPass(PassNames.BEFORE_EARLY_OPTIMIZATION_LOOP));
       passes.addAll(getEarlyOptimizationLoopPasses());
-      // TODO(): Remove this early loop or rename the option that enables it
+      // TODO():: Remove this early loop or rename the option that enables it
       // to something more appropriate.
       passes.maybeAdd(createEmptyPass(PassNames.AFTER_EARLY_OPTIMIZATION_LOOP));
     }

src/com/google/javascript/jscomp/ErrorManager.java

@@ -100,7 +100,7 @@ default boolean hasHaltingErrors() {
    * being used by ConformanceAllowlister to generate allowlist updates (therefore not reported).
    * For all violations other violations (except in generated code), this method returns true.
    */
-  // TODO: b/332922526 - Rename this method and all its overrides to
+  // TODO: user - Rename this method and all its overrides to
   // {@code shouldReportConformanceViolationIgnoringAllowlists} or similar.
   default boolean shouldReportConformanceViolation(
       Requirement requirement,

src/com/google/javascript/jscomp/Es6NormalizeClasses.java

@@ -716,7 +716,7 @@ private void rewriteStaticMembers(NodeTraversal t, ClassRecord record) {
     }
 
     // If there are no super references, place all the static initialization code after the class.
-    // TODO: b/236744850 - Also skip if there are private field refs.
+    // TODO: user - Also skip if there are private field refs.
     if (!NodeUtil.has(
         staticInitTempParent,
         /* pred= */ Node::isSuper,

src/com/google/javascript/jscomp/Es6RewriteRestAndSpread.java

@@ -114,7 +114,7 @@ private void visitRestParam(NodeTraversal t, Node restParam, Node paramList) {
       functionBody.addChildToBack(let);
     }
     NodeUtil.addFeatureToScript(t.getCurrentScript(), Feature.LET_DECLARATIONS, compiler);
-    // TODO: b/421971366 - we should instead assert that es6/util/restargument was already injected
+    // TODO: user - we should instead assert that es6/util/restargument was already injected
     // before this point, by InjectTranspilationRuntimeLibraries.
     compiler
         .getRuntimeJsLibManager()