Rename error key "createsmustcallfor.target.unparseable" to "...unparsable"

Author: mernst

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/Main.java

@@ -795,7 +795,7 @@ public static void main(String[] args) throws IOException {
               // of the line being inserted onto. This puts the annotation on its own line aligned
               // with the contents of the next line.
 
-              // Number of whitespace characters preceeding the insertion position on the same line
+              // Number of whitespace characters preceding the insertion position on the same line
               // (tabs count as one).
               int indentation = 0;
               while ((pos - indentation != 0)

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/find/AnnotationInsertion.java

@@ -16,7 +16,7 @@ public class AnnotationInsertion extends Insertion {
   /**
    * The fully-qualified name of the annotation to be inserted.
    *
-   * <p>E.g. Given an annotation {@code com.foo.Bar(baz)}, its fully quailified name would be {@code
+   * <p>E.g. Given an annotation {@code com.foo.Bar(baz)}, its fully qualified name would be {@code
    * com.foo.Bar}.
    */
   private final String fullyQualifiedAnnotationName;

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/find/InMethodCriterion.java

@@ -76,7 +76,7 @@ public boolean isSatisfiedBy(@Nullable TreePath path) {
       path = path.getParentPath();
     } while (path != null && path.getLeaf() != null);
 
-    // We didn't find the method.  Return true if in a varable declarator,
+    // We didn't find the method.  Return true if in a variable declarator,
     // which is initialization code that will go in <init> or <clinit>.
     boolean result = inDecl && (staticDecl ? "<clinit>()V" : "<init>()V").equals(name);
     Criteria.dbug.debug("InMethodCriterion.isSatisfiedBy => %s%n", result);

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/find/IsSigMethodCriterion.java

@@ -125,7 +125,7 @@ private boolean matchTypeParam(
       if (!typeParamToClassMap.isEmpty()) {
         for (Map.Entry<String, String> p : typeParamToClassMap.entrySet()) {
           @SuppressWarnings(
-              "regex:assignment") // a type parameter name is an identifer, which is a valid regex
+              "regex:assignment") // a type parameter name is an identifier, which is a valid regex
           @Regex String typeParamName = p.getKey();
           // Pattern.quote() is gratuitous because key is a type parameter name, which is an
           // identifier with no special characters.  Matcher.quoteReplacement is gratuitous because

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/find/NewInsertion.java

@@ -15,7 +15,7 @@ public class NewInsertion extends TypedInsertion {
   /**
    * Construct a NewInsertion.
    *
-   * <p>If "new" already exists in the initializer, then pass a {@link DeclaredType} thats name is
+   * <p>If "new" already exists in the initializer, then pass a {@link DeclaredType} whose name is
    * the empty String. This will only insert an annotation on the existing type.
    *
    * <p>To insert the annotation along with "new" and the type (for example, {@code @Anno new Type[]

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/find/TreeFinder.java

@@ -423,7 +423,7 @@ public IPair<ASTRecord, Integer> visitNewClass(NewClassTree node, Insertion ins)
     // For style 3, we need to count the brackets and get to the right one.
     //
     // The AST depth of the initializer is correct unless all arrays are
-    // empty, in which case it is arbitary.  This is legal:
+    // empty, in which case it is arbitrary.  This is legal:
     // String[][][][][] names4 = new String[][][][][] { { {} } };
     //
     // Array initializers can also be multi-dimensional, but this is not
@@ -1179,7 +1179,7 @@ Integer findPosition(TreePath path, Insertion i) {
       // created) receiver.
       Map<Tree, ASTRecord> astIndex = ASTIndex.indexOf(tree);
       ASTRecord insertRecord = astIndex.get(node);
-      dbug.debug("TreeFinder.scan: node=%s%n  critera=%s%n", node, i.getCriteria());
+      dbug.debug("TreeFinder.scan: node=%s%n  criteria=%s%n", node, i.getCriteria());
 
       if (TreePathUtil.hasClassKind(node)
           && i.getCriteria().isOnTypeDeclarationExtendsClause()

annotation-file-utilities/src/main/java/org/checkerframework/afu/annotator/specification/IndexFileSpecification.java

@@ -599,7 +599,7 @@ public static boolean isOnNew(Criteria criteria) {
    * returns false.
    *
    * @param criteria the Criteria
-   * @return true if the Criteria is on a synthethic constructor
+   * @return true if the Criteria is on an implicit default constructor
    */
   private static boolean isOnImplicitDefaultConstructor(Criteria criteria) {
     if (!criteria.isOnMethod("<init>()V")) {

annotation-file-utilities/src/main/java/org/checkerframework/afu/scenelib/el/AnnotationDef.java

@@ -165,7 +165,7 @@ public AnnotationDef(
     setFieldTypes(fieldTypes);
   }
 
-  // This ovverride is necessary because AnnotationDef extends AElement, which implements Cloneable.
+  // This override is necessary because AnnotationDef extends AElement, which implements Cloneable.
   @Override
   public AnnotationDef clone() {
     throw new UnsupportedOperationException("Can't duplicate an AnnotationDef");
@@ -296,7 +296,7 @@ public boolean equals(AnnotationDef o) {
   @Override
   public int hashCode() {
     return name.hashCode()
-        // Omit tlAnnotationsHere, becase it should be unique and, more
+        // Omit tlAnnotationsHere, because it should be unique and, more
         // importantly, including it causes an infinite loop.
         // + tlAnnotationsHere.hashCode()
         + fieldTypes.hashCode();

annotation-file-utilities/src/main/java/org/checkerframework/afu/scenelib/field/BasicAFT.java

@@ -90,11 +90,11 @@ public void format(StringBuilder sb, Object o) {
       sb.append(o.toString());
       sb.append("L");
     } else if (type == double.class && Double.isNaN((double) o)) {
-      // Don't use "Double.NaN" because it is not parseable if the code imports a `Double` class
+      // Don't use "Double.NaN" because it is not parsable if the code imports a `Double` class
       // other than `java.lang.Double`.
       sb.append("0.0/0.0");
     } else if (type == float.class && Float.isNaN((float) o)) {
-      // Don't use "Float.NaN" because it is not parseable if the code imports a `Float` class
+      // Don't use "Float.NaN" because it is not parsable if the code imports a `Float` class
       // other than `java.lang.Float`.
       sb.append("0.0f/0.0f");
     } else {

annotation-file-utilities/src/main/java/org/checkerframework/afu/scenelib/io/IndexFileParser.java

@@ -390,7 +390,11 @@ private int matchNNInteger() throws IOException {
    * "java.util.Map"  for Map.class
    * }</pre>
    *
-   * Thes use fully-qualified names, i.e. "Object" alone won't work.
+   * These use fully-qualified names, i.e. "Object" alone won't work.
+   *
+   * @return the class name that was read, in Class.forName format
+   * @throws IOException if there is trouble reading
+   * @throws ParseException if there is trouble parsing
    */
   private @ClassGetName String expectClassGetName() throws IOException, ParseException {
     int arrays = 0;