fix warnings logged from ant javadoc build target

src/java/org/jivesoftware/admin/JSTLFunctions.java

@@ -90,7 +90,7 @@ public class JSTLFunctions
      * <table>, etc) and converts the '<' and '>' characters to
      * their HTML escape sequences. It will also replace LF  with <br>.
      *
-     * @see StringUtils#escapeHTMLTags(String);
+     * @see StringUtils#escapeHTMLTags(String)
      */
     public static String escapeHTMLTags( String string )
     {

src/java/org/jivesoftware/openfire/RoutingTable.java

@@ -240,7 +240,7 @@ public interface RoutingTable {
      * if none was found. When running inside of a cluster and a remote node is hosting
      * the session then a session surrage will be returned.
      *
-     * @param jid the address of the session.
+     * @param pair DomainPair that specifies the local/remote server address.
      * @return the outgoing server session associated to the specified XMPP address or null if none was found.
      */
     OutgoingServerSession getServerRoute(DomainPair pair);

src/java/org/jivesoftware/openfire/handler/PresenceSubscribeHandler.java

@@ -504,10 +504,10 @@ public class PresenceSubscribeHandler extends BasicModule implements ChannelHand
      * where X is subscribe, subscribed, etc.
      * </p>
      *
-     * @param  item      The item to be updated
-     * @param  action    The new state change request
-     * @param  isSending True if the roster owner of the item is sending the new state change request
-     * @return Change    changes to apply to the item 
+     * @param  itemSubType The item to be updated
+     * @param  action      The new state change request
+     * @param  isSending   True if the roster owner of the item is sending the new state change request
+     * @return Change      changes to apply to the item 
      */
     public static Change getStateChange(RosterItem.SubType itemSubType, Presence.Type action, boolean isSending) {
         Map<String, Map<Presence.Type, Change>> srTable = stateTable.get(itemSubType);

src/java/org/jivesoftware/openfire/user/UserManager.java

@@ -75,7 +75,7 @@ public class UserManager implements IQResultListener {
      * Returns the currently-installed UserPropertyProvider.
      *
      * <b>Warning:</b> in virtually all cases the user property provider should not be used directly. Instead, use the
-     * Map returned by {@link User#getProperties() ) to create, read, update or delete user properties. Failure to do so
+     * Map returned by {@link User#getProperties()} to create, read, update or delete user properties. Failure to do so
      * is likely to result in inconsistent data behavior and race conditions. Direct access to the user property
      * provider is only provided for special-case logic.
      *

src/java/org/jivesoftware/openfire/user/property/DefaultUserPropertyProvider.java

@@ -35,7 +35,7 @@ import java.util.concurrent.ConcurrentHashMap;
  * this implementation will <em>not</em> throw {@link org.jivesoftware.openfire.user.UserNotFoundException}.
  *
  * <b>Warning:</b> in virtually all cases a user property provider should not be used directly. Instead, use the
- * Map returned by {@link User#getProperties() ) to create, read, update or delete user properties. Failure to do so
+ * Map returned by {@link User#getProperties()} to create, read, update or delete user properties. Failure to do so
  * is likely to result in inconsistent data behavior and race conditions. Direct access to the user property
  * provider is only provided for special-case logic.
  *

src/java/org/jivesoftware/openfire/user/property/UserPropertyProvider.java

@@ -31,7 +31,7 @@ import java.util.Map;
  * this respect.
  *
  * <b>Warning:</b> in virtually all cases a user property provider should not be used directly. Instead, use the
- * Map returned by {@link User#getProperties() ) to create, read, update or delete user properties. Failure to do so
+ * Map returned by {@link User#getProperties()} to create, read, update or delete user properties. Failure to do so
  * is likely to result in inconsistent data behavior and race conditions. Direct access to the user property
  * provider is only provided for special-case logic.
  *

src/java/org/jivesoftware/util/cert/SANCertificateIdentityMapping.java

@@ -242,7 +242,7 @@ public class SANCertificateIdentityMapping implements CertificateIdentityMapping
     /**
      * Parse a UPN value 
      *
-     * @param otherName The ASN.1 representation of the UPN (cannot be null).
+     * @param value The ASN.1 representation of the UPN (cannot be null).
      * @return The parsed UPN value.
      */
     protected String parseOtherNameUpn( ASN1Primitive value )