Author: remm
Date: Tue Jan 19 00:06:41 2016
New Revision: 1725393

URL: http://svn.apache.org/viewvc?rev=1725393&view=rev
Log:
Javadoc fixes.

Modified:
    tomcat/trunk/java/org/apache/tomcat/util/digester/CallMethodRule.java
    tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java
    tomcat/trunk/java/org/apache/tomcat/util/digester/ObjectCreationFactory.java
    tomcat/trunk/java/org/apache/tomcat/util/digester/Rules.java
    tomcat/trunk/java/org/apache/tomcat/util/digester/RulesBase.java
    tomcat/trunk/java/org/apache/tomcat/util/http/FastHttpDateFormat.java
    tomcat/trunk/java/org/apache/tomcat/util/http/MimeHeaders.java
    tomcat/trunk/java/org/apache/tomcat/util/http/ServerCookies.java
    tomcat/trunk/java/org/apache/tomcat/util/log/SystemLogHandler.java
    tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java
    tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
    tomcat/trunk/java/org/apache/tomcat/util/net/Nio2Endpoint.java
    tomcat/trunk/java/org/apache/tomcat/util/net/NioChannel.java
    tomcat/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java
    tomcat/trunk/java/org/apache/tomcat/util/net/SecureNio2Channel.java
    tomcat/trunk/java/org/apache/tomcat/util/net/SecureNioChannel.java
    tomcat/trunk/java/org/apache/tomcat/util/net/SocketWrapperBase.java
    tomcat/trunk/java/org/apache/tomcat/util/net/jsse/JSSEUtil.java
    tomcat/trunk/java/org/apache/tomcat/util/threads/LimitLatch.java
    tomcat/trunk/java/org/apache/tomcat/util/threads/ThreadPoolExecutor.java
    tomcat/trunk/java/org/apache/tomcat/websocket/WsSession.java

Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/CallMethodRule.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/CallMethodRule.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/CallMethodRule.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/CallMethodRule.java Tue 
Jan 19 00:06:41 2016
@@ -218,6 +218,7 @@ public class CallMethodRule extends Rule
     /**
      * Should <code>MethodUtils.invokeExactMethod</code>
      * be used for the reflection.
+     * @return <code>true</code> if invokeExactMethod is used
      */
     public boolean getUseExactMatch() {
         return useExactMatch;
@@ -226,6 +227,7 @@ public class CallMethodRule extends Rule
     /**
      * Set whether <code>MethodUtils.invokeExactMethod</code>
      * should be used for the reflection.
+     * @param useExactMatch The flag value
      */
     public void setUseExactMatch(boolean useExactMatch)
     {

Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java Tue Jan 19 
00:06:41 2016
@@ -297,6 +297,7 @@ public class Digester extends DefaultHan
      * go dynamically as the document is parsed.
      *
      * @param prefix Prefix to look up
+     * @return the namespace URI
      */
     public String findNamespaceURI(String prefix) {
 
@@ -322,6 +323,7 @@ public class Digester extends DefaultHan
      *     <code>useContextClassLoader</code> property is set to true</li>
      * <li>The class loader used to load the Digester class itself.
      * </ul>
+     * @return the classloader
      */
     public ClassLoader getClassLoader() {
 
@@ -355,7 +357,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the current depth of the element stack.
+     * @return the current depth of the element stack.
      */
     public int getCount() {
 
@@ -365,7 +367,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the name of the XML element that is currently being processed.
+     * @return the name of the XML element that is currently being processed.
      */
     public String getCurrentElementName() {
 
@@ -380,7 +382,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the error handler for this Digester.
+     * @return the error handler for this Digester.
      */
     public ErrorHandler getErrorHandler() {
 
@@ -402,10 +404,11 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the SAXParserFactory we will use, creating one if necessary.
-     * @throws ParserConfigurationException
-     * @throws SAXNotSupportedException
-     * @throws SAXNotRecognizedException
+     * SAX parser factory method.
+     * @return the SAXParserFactory we will use, creating one if necessary.
+     * @throws ParserConfigurationException Error creating parser
+     * @throws SAXNotSupportedException Error creating parser
+     * @throws SAXNotRecognizedException Error creating parser
      */
     public SAXParserFactory getFactory()
     throws SAXNotRecognizedException, SAXNotSupportedException,
@@ -469,7 +472,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the current Logger associated with this instance of the Digester
+     * @return the current Logger associated with this instance of the Digester
      */
     public Log getLogger() {
 
@@ -480,6 +483,7 @@ public class Digester extends DefaultHan
 
     /**
      * Set the current logger for this Digester.
+     * @param log The logger that will be used
      */
     public void setLogger(Log log) {
 
@@ -492,6 +496,7 @@ public class Digester extends DefaultHan
      * <strong>Note</strong> the output is finely grained.
      *
      * @since 1.6
+     * @return the SAX logger
      */
     public Log getSAXLogger() {
 
@@ -512,7 +517,7 @@ public class Digester extends DefaultHan
     }
 
     /**
-     * Return the current rule match path
+     * @return the current rule match path
      */
     public String getMatch() {
 
@@ -522,7 +527,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the "namespace aware" flag for parsers we create.
+     * @return the "namespace aware" flag for parsers we create.
      */
     public boolean getNamespaceAware() {
 
@@ -553,7 +558,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the public identifier of the DTD we are currently
+     * @return the public identifier of the DTD we are currently
      * parsing under, if any.
      */
     public String getPublicId() {
@@ -564,7 +569,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the namespace URI that will be applied to all subsequently
+     * @return the namespace URI that will be applied to all subsequently
      * added <code>Rule</code> objects.
      */
     public String getRuleNamespaceURI() {
@@ -590,7 +595,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the SAXParser we will use to parse the input stream.  If there
+     * @return the SAXParser we will use to parse the input stream.  If there
      * is a problem creating the parser, return <code>null</code>.
      */
     public SAXParser getParser() {
@@ -621,7 +626,7 @@ public class Digester extends DefaultHan
      * for information about the standard SAX2 properties.
      *
      * @param property Property name to be retrieved
-     *
+     * @return the property value
      * @exception SAXNotRecognizedException if the property name is
      *  not recognized
      * @exception SAXNotSupportedException if the property name is
@@ -639,6 +644,7 @@ public class Digester extends DefaultHan
      * Return the <code>Rules</code> implementation object containing our
      * rules collection and associated matching policy.  If none has been
      * established, a default implementation will be created and returned.
+     * @return the rules
      */
     public Rules getRules() {
 
@@ -666,7 +672,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the boolean as to whether the context classloader should be used.
+     * @return the boolean as to whether the context classloader should be 
used.
      */
     public boolean getUseContextClassLoader() {
 
@@ -692,7 +698,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the validating parser flag.
+     * @return the validating parser flag.
      */
     public boolean getValidating() {
 
@@ -715,7 +721,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the rules validation flag.
+     * @return the rules validation flag.
      */
     public boolean getRulesValidation() {
 
@@ -738,7 +744,7 @@ public class Digester extends DefaultHan
 
 
     /**
-     * Return the fake attributes list.
+     * @return the fake attributes list.
      */
     public Map<Class<?>, List<String>> getFakeAttributes() {
 
@@ -749,6 +755,9 @@ public class Digester extends DefaultHan
 
     /**
      * Determine if an attribute is a fake attribute.
+     * @param object The object
+     * @param name The attribute name
+     * @return <code>true</code> if this is a fake attribute
      */
     public boolean isFakeAttribute(Object object, String name) {
 
@@ -785,6 +794,7 @@ public class Digester extends DefaultHan
      *
      * FIX ME: there is a bug in JAXP/XERCES that prevent the use of a
      * parser that contains a schema with a DTD.
+     * @return the XML reader
      * @exception SAXException if no XMLReader can be instantiated
      */
     public XMLReader getXMLReader() throws SAXException {
@@ -1423,7 +1433,7 @@ public class Digester extends DefaultHan
      * the root element from the object stack (if any).
      *
      * @param file File containing the XML data to be parsed
-     *
+     * @return the root object
      * @exception IOException if an input/output error occurs
      * @exception SAXException if a parsing exception occurs
      */
@@ -1441,7 +1451,7 @@ public class Digester extends DefaultHan
      * Returns the root element from the object stack (if any).
      *
      * @param input Input source containing the XML data to be parsed
-     *
+     * @return the root object
      * @exception IOException if an input/output error occurs
      * @exception SAXException if a parsing exception occurs
      */
@@ -1459,7 +1469,7 @@ public class Digester extends DefaultHan
      * Returns the root element from the object stack (if any).
      *
      * @param input Input stream containing the XML data to be parsed
-     *
+     * @return the root object
      * @exception IOException if an input/output error occurs
      * @exception SAXException if a parsing exception occurs
      */
@@ -1716,6 +1726,7 @@ public class Digester extends DefaultHan
     /**
      * Return the top object on the stack without removing it.  If there are
      * no objects on the stack, return <code>null</code>.
+     * @return the top object
      */
     public Object peek() {
 
@@ -1736,6 +1747,7 @@ public class Digester extends DefaultHan
      *
      * @param n Index of the desired element, where 0 is the top of the stack,
      *  1 is the next element down, and so on.
+     * @return the specified object
      */
     public Object peek(int n) {
 
@@ -1752,6 +1764,7 @@ public class Digester extends DefaultHan
     /**
      * Pop the top object off of the stack, and return it.  If there are
      * no objects on the stack, return <code>null</code>.
+     * @return the top object
      */
     public Object pop() {
 
@@ -1830,6 +1843,7 @@ public class Digester extends DefaultHan
      *
      * <p>The parameters stack is used to store <code>CallMethodRule</code> 
parameters.
      * See {@link #params}.</p>
+     * @return the top object on the parameters stack
      */
     public Object peekParams() {
 
@@ -1849,6 +1863,7 @@ public class Digester extends DefaultHan
      *
      * <p>The parameters stack is used to store <code>CallMethodRule</code> 
parameters.
      * See {@link #params}.</p>
+     * @return the top object on the parameters stack
      */
     public Object popParams() {
 
@@ -1884,7 +1899,8 @@ public class Digester extends DefaultHan
     /**
      * Create a SAX exception which also understands about the location in
      * the digester file where the exception occurs
-     *
+     * @param message The error message
+     * @param e The root cause
      * @return the new exception
      */
     public SAXException createSAXException(String message, Exception e) {
@@ -1921,7 +1937,7 @@ public class Digester extends DefaultHan
     /**
      * Create a SAX exception which also understands about the location in
      * the digester file where the exception occurs
-     *
+     * @param e The root cause
      * @return the new exception
      */
     public SAXException createSAXException(Exception e) {
@@ -1943,7 +1959,7 @@ public class Digester extends DefaultHan
     /**
      * Create a SAX exception which also understands about the location in
      * the digester file where the exception occurs
-     *
+     * @param message The error message
      * @return the new exception
      */
     public SAXException createSAXException(String message) {

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/digester/ObjectCreationFactory.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/ObjectCreationFactory.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/tomcat/util/digester/ObjectCreationFactory.java 
(original)
+++ 
tomcat/trunk/java/org/apache/tomcat/util/digester/ObjectCreationFactory.java 
Tue Jan 19 00:06:41 2016
@@ -33,23 +33,23 @@ import org.xml.sax.Attributes;
 public interface ObjectCreationFactory {
 
     /**
-     * <p>Factory method called by {@link FactoryCreateRule} to supply an
+     * Factory method called by {@link FactoryCreateRule} to supply an
      * object based on the element's attributes.
      *
      * @param attributes the element's attributes
-     *
+     * @return the creted object
      * @throws Exception any exception thrown will be propagated upwards
      */
     public Object createObject(Attributes attributes) throws Exception;
 
     /**
-     * <p>Returns the {@link Digester} that was set by the
+     * @return the {@link Digester} that was set by the
      * {@link FactoryCreateRule} upon initialization.
      */
     public Digester getDigester();
 
     /**
-     * <p>Set the {@link Digester} to allow the implementation to do logging,
+     * Set the {@link Digester} to allow the implementation to do logging,
      * classloading based on the digester's classloader, etc.
      *
      * @param digester parent Digester object

Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/Rules.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/Rules.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/Rules.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/Rules.java Tue Jan 19 
00:06:41 2016
@@ -36,7 +36,7 @@ public interface Rules {
 
 
     /**
-     * Return the Digester instance with which this Rules instance is
+     * @return the Digester instance with which this Rules instance is
      * associated.
      */
     public Digester getDigester();
@@ -51,7 +51,7 @@ public interface Rules {
 
 
     /**
-     * Return the namespace URI that will be applied to all subsequently
+     * @return the namespace URI that will be applied to all subsequently
      * added <code>Rule</code> objects.
      */
     public String getNamespaceURI();
@@ -96,6 +96,7 @@ public interface Rules {
      * @param namespaceURI Namespace URI for which to select matching rules,
      *  or <code>null</code> to match regardless of namespace URI
      * @param pattern Nesting pattern to be matched
+     * @return a rules list
      */
     public List<Rule> match(String namespaceURI, String pattern);
 
@@ -106,6 +107,7 @@ public interface Rules {
      * instance has been registered, they <strong>must</strong> be returned
      * in the order originally registered through the <code>add()</code>
      * method.
+     * @return a rules list
      */
     public List<Rule> rules();
 

Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/RulesBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/RulesBase.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/digester/RulesBase.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/digester/RulesBase.java Tue Jan 19 
00:06:41 2016
@@ -253,6 +253,7 @@ public class RulesBase implements Rules
      * @param namespaceURI Namespace URI to match, or <code>null</code> to
      *  select matching rules regardless of namespace URI
      * @param pattern Pattern to be matched
+     * @return a rules list
      */
     protected List<Rule> lookup(String namespaceURI, String pattern) {
 

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/FastHttpDateFormat.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/FastHttpDateFormat.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/http/FastHttpDateFormat.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/FastHttpDateFormat.java Tue 
Jan 19 00:06:41 2016
@@ -91,6 +91,7 @@ public final class FastHttpDateFormat {
 
     /**
      * Get the current date in HTTP format.
+     * @return the HTTP date
      */
     public static final String getCurrentDate() {
 
@@ -110,6 +111,9 @@ public final class FastHttpDateFormat {
 
     /**
      * Get the HTTP format of the specified date.
+     * @param value The date
+     * @param threadLocalformat Local format to avoid synchronization
+     * @return the HTTP date
      */
     public static final String formatDate
         (long value, DateFormat threadLocalformat) {
@@ -137,6 +141,9 @@ public final class FastHttpDateFormat {
 
     /**
      * Try to parse the given date as a HTTP date.
+     * @param value The HTTP date
+     * @param threadLocalformats Local format to avoid synchronization
+     * @return the date as a long
      */
     public static final long parseDate(String value,
                                        DateFormat[] threadLocalformats) {

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/MimeHeaders.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/MimeHeaders.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/http/MimeHeaders.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/MimeHeaders.java Tue Jan 19 
00:06:41 2016
@@ -24,9 +24,6 @@ import java.util.Enumeration;
 import org.apache.tomcat.util.buf.MessageBytes;
 import org.apache.tomcat.util.res.StringManager;
 
-/* XXX XXX XXX Need a major rewrite  !!!!
- */
-
 /**
  * This class is used to contain standard internet message headers,
  * used for SMTP (RFC822) and HTTP (RFC2068) messages as well as for
@@ -125,6 +122,7 @@ public class MimeHeaders {
 
     /**
      * Set limit on the number of header fields.
+     * @param limit The new limit
      */
     public void setLimit(int limit) {
         this.limit = limit;
@@ -188,14 +186,15 @@ public class MimeHeaders {
     // -------------------- Idx access to headers ----------
 
     /**
-     * Returns the current number of header fields.
+     * @return the current number of header fields.
      */
     public int size() {
         return count;
     }
 
     /**
-     * Returns the Nth header name, or null if there is no such header.
+     * @param n The header index
+     * @return the Nth header name, or null if there is no such header.
      * This may be used to iterate through all header fields.
      */
     public MessageBytes getName(int n) {
@@ -203,14 +202,19 @@ public class MimeHeaders {
     }
 
     /**
-     * Returns the Nth header value, or null if there is no such header.
+     * @param n The header index
+     * @return the Nth header value, or null if there is no such header.
      * This may be used to iterate through all header fields.
      */
     public MessageBytes getValue(int n) {
         return n >= 0 && n < count ? headers[n].getValue() : null;
     }
 
-    /** Find the index of a header with the given name.
+    /**
+     * Find the index of a header with the given name.
+     * @param name The header name
+     * @param starting Index on which to start looking
+     * @return the header index
      */
     public int findHeader( String name, int starting ) {
         // We can use a hash - but it's not clear how much
@@ -234,6 +238,7 @@ public class MimeHeaders {
      * Returns an enumeration of strings representing the header field names.
      * Field names may appear multiple times in this enumeration, indicating
      * that multiple fields with that name exist in this header.
+     * @return the enumeration
      */
     public Enumeration<String> names() {
         return new NamesEnumerator(this);
@@ -274,18 +279,26 @@ public class MimeHeaders {
         return mh;
     }
 
-    /** Create a new named header , return the MessageBytes
-        container for the new value
-    */
+    /**
+     * Create a new named header , return the MessageBytes
+     * container for the new value
+     * @param name The header name
+     * @return the message bytes container for the value
+     */
     public MessageBytes addValue( String name ) {
          MimeHeaderField mh = createHeader();
         mh.getName().setString(name);
         return mh.getValue();
     }
 
-    /** Create a new named header using un-translated byte[].
-        The conversion to chars can be delayed until
-        encoding is known.
+    /**
+     * Create a new named header using un-translated byte[].
+     * The conversion to chars can be delayed until
+     * encoding is known.
+     * @param b The header name bytes
+     * @param startN Offset
+     * @param len Length
+     * @return the message bytes container for the value
      */
     public MessageBytes addValue(byte b[], int startN, int len)
     {
@@ -294,11 +307,12 @@ public class MimeHeaders {
         return mhf.getValue();
     }
 
-    /** Allow "set" operations -
-        return a MessageBytes container for the
-        header value ( existing header or new
-        if this .
-    */
+    /**
+     * Allow "set" operations, which removes all current values
+     * for this header.
+     * @param name The header name
+     * @return the message bytes container for the value
+     */
     public MessageBytes setValue( String name ) {
         for ( int i = 0; i < count; i++ ) {
             if(headers[i].getName().equalsIgnoreCase(name)) {
@@ -320,6 +334,8 @@ public class MimeHeaders {
      * Finds and returns a header field with the given name.  If no such
      * field exists, null is returned.  If more than one such field is
      * in the header, an arbitrary one is returned.
+     * @param name The header name
+     * @return the value
      */
     public MessageBytes getValue(String name) {
         for (int i = 0; i < count; i++) {
@@ -334,6 +350,9 @@ public class MimeHeaders {
      * Finds and returns a unique header field with the given name. If no such
      * field exists, null is returned. If the specified header field is not
      * unique then an {@link IllegalArgumentException} is thrown.
+     * @param name The header name
+     * @return the value if unique
+     * @throws IllegalArgumentException if the header has multiple values
      */
     public MessageBytes getUniqueValue(String name) {
         MessageBytes result = null;

Modified: tomcat/trunk/java/org/apache/tomcat/util/http/ServerCookies.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/http/ServerCookies.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/http/ServerCookies.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/http/ServerCookies.java Tue Jan 19 
00:06:41 2016
@@ -35,6 +35,7 @@ public class ServerCookies {
      * Register a new, initialized cookie. Cookies are recycled, and most of 
the
      * time an existing ServerCookie object is returned. The caller can set the
      * name/value and attributes for the cookie.
+     * @return the new cookie
      */
     public ServerCookie addCookie() {
         if (cookieCount >= serverCookies.length) {

Modified: tomcat/trunk/java/org/apache/tomcat/util/log/SystemLogHandler.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/log/SystemLogHandler.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/log/SystemLogHandler.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/log/SystemLogHandler.java Tue Jan 
19 00:06:41 2016
@@ -121,6 +121,7 @@ public class SystemLogHandler extends Pr
 
     /**
      * Find PrintStream to which the output must be written to.
+     * @return the print stream
      */
     protected PrintStream findStream() {
         Stack<CaptureLog> stack = logs.get();

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/AbstractEndpoint.java Tue Jan 
19 00:06:41 2016
@@ -818,6 +818,7 @@ public abstract class AbstractEndpoint<S
 
     /**
      * Hook to allow Endpoints to provide a specific Acceptor implementation.
+     * @return the acceptor
      */
     protected abstract Acceptor createAcceptor();
 

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Tue Jan 19 
00:06:41 2016
@@ -708,6 +708,10 @@ public class AprEndpoint extends Abstrac
 
     /**
      * Process the specified connection.
+     * @param socketWrapper The socket wrapper
+     * @return <code>true</code> if the socket was correctly configured
+     *  and processing may continue, <code>false</code> if the socket needs to 
be
+     *  close immediately
      */
     protected boolean setSocketOptions(SocketWrapperBase<Long> socketWrapper) {
         long socket = socketWrapper.getSocket().longValue();
@@ -764,6 +768,10 @@ public class AprEndpoint extends Abstrac
 
     /**
      * Allocate a new poller of the specified size.
+     * @param size The size
+     * @param pool The pool from which the poller will be allocated
+     * @param timeout The timeout
+     * @return the poller pointer
      */
     protected long allocatePoller(int size, long pool, int timeout) {
         try {
@@ -782,6 +790,10 @@ public class AprEndpoint extends Abstrac
     /**
      * Process given socket. This is called when the socket has been
      * accepted.
+     * @param socket The socket
+     * @return <code>true</code> if the socket was correctly configured
+     *  and processing may continue, <code>false</code> if the socket needs to 
be
+     *  close immediately
      */
     protected boolean processSocketWithOptions(long socket) {
         try {

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/Nio2Endpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/Nio2Endpoint.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/Nio2Endpoint.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/Nio2Endpoint.java Tue Jan 19 
00:06:41 2016
@@ -329,6 +329,10 @@ public class Nio2Endpoint extends Abstra
 
     /**
      * Process the specified connection.
+     * @param socket The socket channel
+     * @return <code>true</code> if the socket was correctly configured
+     *  and processing may continue, <code>false</code> if the socket needs to 
be
+     *  close immediately
      */
     protected boolean setSocketOptions(AsynchronousSocketChannel socket) {
         try {

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/NioChannel.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/NioChannel.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/NioChannel.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/NioChannel.java Tue Jan 19 
00:06:41 2016
@@ -225,6 +225,7 @@ public class NioChannel implements ByteC
      * socket is removed from the poller without the socket being selected. 
This
      * results in a connection limit leak for NIO as the endpoint expects the
      * socket to be selected even in error conditions.
+     * @throws IOException If the current thread was interrupted
      */
     protected void checkInterruptStatus() throws IOException {
         if (Thread.interrupted()) {

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/NioEndpoint.java Tue Jan 19 
00:06:41 2016
@@ -382,6 +382,10 @@ public class NioEndpoint extends Abstrac
 
     /**
      * Process the specified connection.
+     * @param socket The socket channel
+     * @return <code>true</code> if the socket was correctly configured
+     *  and processing may continue, <code>false</code> if the socket needs to 
be
+     *  close immediately
      */
     protected boolean setSocketOptions(SocketChannel socket) {
         // Process the connection

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/SecureNio2Channel.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/SecureNio2Channel.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/SecureNio2Channel.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/SecureNio2Channel.java Tue Jan 
19 00:06:41 2016
@@ -445,7 +445,7 @@ public class SecureNio2Channel extends N
 
     /**
      * Executes all the tasks needed on the same thread.
-     * @return HandshakeStatus
+     * @return the status
      */
     protected SSLEngineResult.HandshakeStatus tasks() {
         Runnable r = null;
@@ -457,8 +457,8 @@ public class SecureNio2Channel extends N
 
     /**
      * Performs the WRAP function
-     * @return SSLEngineResult
-     * @throws IOException
+     * @return the result
+     * @throws IOException An IO error occurred
      */
     protected SSLEngineResult handshakeWrap() throws IOException {
         //this should never be called with a network buffer that contains data
@@ -476,8 +476,8 @@ public class SecureNio2Channel extends N
 
     /**
      * Perform handshake unwrap
-     * @return SSLEngineResult
-     * @throws IOException
+     * @return the result
+     * @throws IOException An IO error occurred
      */
     protected SSLEngineResult handshakeUnwrap() throws IOException {
         if (netInBuffer.position() == netInBuffer.limit()) {

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/SecureNioChannel.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/SecureNioChannel.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/SecureNioChannel.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/SecureNioChannel.java Tue Jan 
19 00:06:41 2016
@@ -135,7 +135,7 @@ public class SecureNioChannel extends Ni
      * Flushes the buffer to the network, non blocking
      * @param buf ByteBuffer
      * @return boolean true if the buffer has been emptied out, false otherwise
-     * @throws IOException
+     * @throws IOException An IO error occurred writing data
      */
     protected boolean flush(ByteBuffer buf) throws IOException {
         int remaining = buf.remaining();
@@ -383,7 +383,7 @@ public class SecureNioChannel extends Ni
 
     /**
      * Executes all the tasks needed on the same thread.
-     * @return HandshakeStatus
+     * @return the status
      */
     protected SSLEngineResult.HandshakeStatus tasks() {
         Runnable r = null;
@@ -396,8 +396,8 @@ public class SecureNioChannel extends Ni
     /**
      * Performs the WRAP function
      * @param doWrite boolean
-     * @return SSLEngineResult
-     * @throws IOException
+     * @return the result
+     * @throws IOException An IO error occurred
      */
     protected SSLEngineResult handshakeWrap(boolean doWrite) throws 
IOException {
         //this should never be called with a network buffer that contains data
@@ -418,8 +418,8 @@ public class SecureNioChannel extends Ni
     /**
      * Perform handshake unwrap
      * @param doread boolean
-     * @return SSLEngineResult
-     * @throws IOException
+     * @return the result
+     * @throws IOException An IO error occurred
      */
     protected SSLEngineResult handshakeUnwrap(boolean doread) throws 
IOException {
 

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/SocketWrapperBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/SocketWrapperBase.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/SocketWrapperBase.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/SocketWrapperBase.java Tue Jan 
19 00:06:41 2016
@@ -680,6 +680,7 @@ public abstract class SocketWrapperBase<
      * @param check for the IO operation completion
      * @param handler to call when the IO is complete
      * @param dsts buffers
+     * @param <A> The attachment type
      * @return the completion state (done, done inline, or still pending)
      */
     public <A> CompletionState read(boolean block, long timeout, TimeUnit 
unit, A attachment,
@@ -714,6 +715,7 @@ public abstract class SocketWrapperBase<
      *        used when calling the completion handler
      * @param check for the IO operation completion
      * @param handler to call when the IO is complete
+     * @param <A> The attachment type
      * @return the completion state (done, done inline, or still pending)
      */
     public abstract <A> CompletionState read(ByteBuffer[] dsts, int offset, 
int length,
@@ -742,6 +744,7 @@ public abstract class SocketWrapperBase<
      * @param check for the IO operation completion
      * @param handler to call when the IO is complete
      * @param srcs buffers
+     * @param <A> The attachment type
      * @return the completion state (done, done inline, or still pending)
      */
     public <A> CompletionState write(boolean block, long timeout, TimeUnit 
unit, A attachment,
@@ -777,6 +780,7 @@ public abstract class SocketWrapperBase<
      *        used when calling the completion handler
      * @param check for the IO operation completion
      * @param handler to call when the IO is complete
+     * @param <A> The attachment type
      * @return the completion state (done, done inline, or still pending)
      */
     public abstract <A> CompletionState write(ByteBuffer[] srcs, int offset, 
int length,

Modified: tomcat/trunk/java/org/apache/tomcat/util/net/jsse/JSSEUtil.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/net/jsse/JSSEUtil.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/net/jsse/JSSEUtil.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/net/jsse/JSSEUtil.java Tue Jan 19 
00:06:41 2016
@@ -354,6 +354,7 @@ public class JSSEUtil extends SSLUtilBas
      * @param crlf The path to the CRL file.
      * @param trustStore The configured TrustStore.
      * @return The parameters including the CRLs and TrustStore.
+     * @throws Exception An error occurred
      */
     protected CertPathParameters getParameters(String algorithm, String crlf,
             KeyStore trustStore) throws Exception {
@@ -376,7 +377,11 @@ public class JSSEUtil extends SSLUtilBas
 
     /**
      * Load the collection of CRLs.
-     *
+     * @param crlf The path to the CRL file.
+     * @return the CRLs collection
+     * @throws IOException Error reading CRL file
+     * @throws CRLException CRL error
+     * @throws CertificateException Error processing certificate
      */
     protected Collection<? extends CRL> getCRLs(String crlf)
         throws IOException, CRLException, CertificateException {

Modified: tomcat/trunk/java/org/apache/tomcat/util/threads/LimitLatch.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/threads/LimitLatch.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/threads/LimitLatch.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/util/threads/LimitLatch.java Tue Jan 19 
00:06:41 2016
@@ -82,6 +82,7 @@ public class LimitLatch {
 
     /**
      * Obtain the current limit.
+     * @return the limit
      */
     public long getLimit() {
         return limit;
@@ -107,6 +108,7 @@ public class LimitLatch {
     /**
      * Acquires a shared latch if one is available or waits for one if no 
shared
      * latch is current available.
+     * @throws InterruptedException If the current thread is interrupted
      */
     public void countUpOrAwait() throws InterruptedException {
         if (log.isDebugEnabled()) {
@@ -131,6 +133,7 @@ public class LimitLatch {
     /**
      * Releases all waiting threads and causes the {@link #limit} to be ignored
      * until {@link #reset()} is called.
+     * @return <code>true</code> if release was done
      */
     public boolean releaseAll() {
         released = true;
@@ -149,6 +152,7 @@ public class LimitLatch {
     /**
      * Returns <code>true</code> if there is at least one thread waiting to
      * acquire the shared lock, otherwise returns <code>false</code>.
+     * @return <code>true</code> if threads are waiting
      */
     public boolean hasQueuedThreads() {
         return sync.hasQueuedThreads();
@@ -157,6 +161,7 @@ public class LimitLatch {
     /**
      * Provide access to the list of threads waiting to acquire this limited
      * shared latch.
+     * @return a collection of threads
      */
     public Collection<Thread> getQueuedThreads() {
         return sync.getQueuedThreads();

Modified: 
tomcat/trunk/java/org/apache/tomcat/util/threads/ThreadPoolExecutor.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/threads/ThreadPoolExecutor.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/util/threads/ThreadPoolExecutor.java 
(original)
+++ tomcat/trunk/java/org/apache/tomcat/util/threads/ThreadPoolExecutor.java 
Tue Jan 19 00:06:41 2016
@@ -151,6 +151,8 @@ public class ThreadPoolExecutor extends
      * full after that.
      *
      * @param command the runnable task
+     * @param timeout A timeout for the completion of the task
+     * @param unit The timeout time unit
      * @throws RejectedExecutionException if this task cannot be
      * accepted for execution - the queue is full
      * @throws NullPointerException if command or unit is null

Modified: tomcat/trunk/java/org/apache/tomcat/websocket/WsSession.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/websocket/WsSession.java?rev=1725393&r1=1725392&r2=1725393&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/tomcat/websocket/WsSession.java (original)
+++ tomcat/trunk/java/org/apache/tomcat/websocket/WsSession.java Tue Jan 19 
00:06:41 2016
@@ -614,6 +614,8 @@ public class WsSession implements Sessio
 
     /**
      * Use protected so unit tests can access this method directly.
+     * @param msg The message
+     * @param reason The reason
      */
     protected static void appendCloseReasonWithTruncation(ByteBuffer msg,
             String reason) {
@@ -648,6 +650,7 @@ public class WsSession implements Sessio
      * Make the session aware of a {@link FutureToSendHandler} that will need 
to
      * be forcibly closed if the session closes before the
      * {@link FutureToSendHandler} completes.
+     * @param f2sh The handler
      */
     protected void registerFuture(FutureToSendHandler f2sh) {
         boolean fail = false;
@@ -673,6 +676,7 @@ public class WsSession implements Sessio
 
     /**
      * Remove a {@link FutureToSendHandler} from the set of tracked instances.
+     * @param f2sh The handler
      */
     protected void unregisterFuture(FutureToSendHandler f2sh) {
         futures.remove(f2sh);




---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to