Fix errors from merge conflict resolution

Errors were made in manual conflict resolution in commit
a37a38464aeef5ef2eee77b8463f975d6ffa34af (and subsequently
in commit bb9054772d76fde416af3c0e18f98999daa65bd8).

This change resolves:

1. A syntax error that occured due to a missing curly brace.
2. A few tag variables that didn't get renamed properly.
3. Check for UNKNOWN_PARAM_TAG_NAME error was duplicated twice.
4. Preserved isTypeParameter() check for UNKNOWN_PARAM_TAG_NAME
   logic.

Change-Id: I1b79ced0a9803f6487aa5b073b700c0c6d5dafdd
diff --git a/src/com/google/doclava/MethodInfo.java b/src/com/google/doclava/MethodInfo.java
index 5ceb0e1..5d6188a 100644
--- a/src/com/google/doclava/MethodInfo.java
+++ b/src/com/google/doclava/MethodInfo.java
@@ -456,7 +456,7 @@
 
       // Complain about misnamed @param tags
       for (ParamTagInfo tag : paramTags) {
-        if (!isParamTagInMethod(tag)){
+        if (!isParamTagInMethod(tag) && !tag.isTypeParameter()){
           Errors.error(Errors.UNKNOWN_PARAM_TAG_NAME, tag.position(),
               "@param tag with name that doesn't match the parameter list: '"
               + tag.parameterName() + "'");
@@ -481,12 +481,7 @@
         if (index >= 0) {
           comment = paramTags[index].parameterComment();
           position = paramTags[index].position();
-        } else if (!tag.isTypeParameter()) {
-          // Complain about misnamed @param tags. Note that we must exclude type
-          // parameter tags (such as "@param <T> foo") from this analysis.
-          Errors.error(Errors.UNKNOWN_PARAM_TAG_NAME, tag.position(),
-              "@param tag with name that doesn't match the parameter list: '" + tag.parameterName()
-                  + "'");
+        }
 
         // get our parent's tags to fill in the blanks
         MethodInfo overridden = this.findOverriddenMethod(name(), signature());