Author: remm Date: Thu Jan 14 14:55:44 2016 New Revision: 1724626 URL: http://svn.apache.org/viewvc?rev=1724626&view=rev Log: Javadoc fixes.
Modified: tomcat/trunk/java/org/apache/catalina/tribes/ByteMessage.java tomcat/trunk/java/org/apache/catalina/tribes/Channel.java tomcat/trunk/java/org/apache/catalina/tribes/ChannelException.java tomcat/trunk/java/org/apache/catalina/tribes/ChannelInterceptor.java tomcat/trunk/java/org/apache/catalina/tribes/ChannelReceiver.java tomcat/trunk/java/org/apache/catalina/tribes/Member.java tomcat/trunk/java/org/apache/catalina/tribes/MembershipService.java tomcat/trunk/java/org/apache/catalina/tribes/group/GroupChannel.java tomcat/trunk/java/org/apache/catalina/tribes/group/RpcCallback.java tomcat/trunk/java/org/apache/catalina/tribes/group/RpcChannel.java tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/GzipInterceptor.java tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/SimpleCoordinator.java tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java tomcat/trunk/java/org/apache/catalina/tribes/membership/McastService.java tomcat/trunk/java/org/apache/catalina/tribes/membership/McastServiceImpl.java tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java tomcat/trunk/java/org/apache/catalina/tribes/tipis/LazyReplicatedMap.java tomcat/trunk/java/org/apache/catalina/tribes/tipis/ReplicatedMap.java tomcat/trunk/java/org/apache/catalina/tribes/transport/ReceiverBase.java tomcat/trunk/java/org/apache/catalina/tribes/transport/RxTaskPool.java tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioReplicationTask.java tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioSender.java tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReceiver.java tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReplicationTask.java tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioSender.java tomcat/trunk/java/org/apache/catalina/users/MemoryUser.java tomcat/trunk/java/org/apache/catalina/users/MemoryUserDatabase.java tomcat/trunk/java/org/apache/catalina/util/CharsetMapper.java tomcat/trunk/java/org/apache/catalina/util/DOMWriter.java tomcat/trunk/java/org/apache/catalina/util/Extension.java tomcat/trunk/java/org/apache/catalina/util/ExtensionValidator.java tomcat/trunk/java/org/apache/catalina/util/IOTools.java tomcat/trunk/java/org/apache/catalina/util/Introspection.java tomcat/trunk/java/org/apache/catalina/util/MIME2Java.java tomcat/trunk/java/org/apache/catalina/util/ParameterMap.java tomcat/trunk/java/org/apache/catalina/util/ResourceSet.java tomcat/trunk/java/org/apache/catalina/util/ServerInfo.java tomcat/trunk/java/org/apache/catalina/util/SessionConfig.java tomcat/trunk/java/org/apache/catalina/util/Strftime.java tomcat/trunk/java/org/apache/catalina/util/XMLWriter.java tomcat/trunk/java/org/apache/catalina/valves/AbstractAccessLogValve.java tomcat/trunk/java/org/apache/catalina/valves/AccessLogValve.java tomcat/trunk/java/org/apache/catalina/valves/ErrorReportValve.java tomcat/trunk/java/org/apache/catalina/valves/JDBCAccessLogValve.java tomcat/trunk/java/org/apache/catalina/valves/PersistentValve.java tomcat/trunk/java/org/apache/catalina/valves/RemoteAddrValve.java tomcat/trunk/java/org/apache/catalina/valves/RemoteHostValve.java tomcat/trunk/java/org/apache/catalina/valves/RemoteIpValve.java tomcat/trunk/java/org/apache/catalina/valves/RequestFilterValve.java tomcat/trunk/java/org/apache/catalina/valves/SemaphoreValve.java tomcat/trunk/java/org/apache/catalina/valves/rewrite/RewriteCond.java tomcat/trunk/java/org/apache/catalina/valves/rewrite/RewriteRule.java tomcat/trunk/java/org/apache/catalina/valves/rewrite/RewriteValve.java tomcat/trunk/java/org/apache/catalina/valves/rewrite/Substitution.java tomcat/trunk/java/org/apache/catalina/webresources/StandardRoot.java Modified: tomcat/trunk/java/org/apache/catalina/tribes/ByteMessage.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/ByteMessage.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/ByteMessage.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/ByteMessage.java Thu Jan 14 14:55:44 2016 @@ -77,7 +77,7 @@ public class ByteMessage implements Exte /** * @see java.io.Externalizable#readExternal * @param in ObjectInput - * @throws IOException + * @throws IOException An IO error occurred */ @Override public void readExternal(ObjectInput in ) throws IOException { @@ -89,7 +89,7 @@ public class ByteMessage implements Exte /** * @see java.io.Externalizable#writeExternal * @param out ObjectOutput - * @throws IOException + * @throws IOException An IO error occurred */ @Override public void writeExternal(ObjectOutput out) throws IOException { Modified: tomcat/trunk/java/org/apache/catalina/tribes/Channel.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/Channel.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/Channel.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/Channel.java Thu Jan 14 14:55:44 2016 @@ -253,6 +253,7 @@ public interface Channel { * send a pure byte array * @param options int - sender options, see class documentation for each interceptor that is configured in order to trigger interceptors * @return a unique Id that identifies the message that is sent + * @throws ChannelException if a serialization error happens. * @see ByteMessage * @see #SEND_OPTIONS_USE_ACK * @see #SEND_OPTIONS_ASYNCHRONOUS Modified: tomcat/trunk/java/org/apache/catalina/tribes/ChannelException.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/ChannelException.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/ChannelException.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/ChannelException.java Thu Jan 14 14:55:44 2016 @@ -33,7 +33,7 @@ public class ChannelException extends Ex * Empty list to avoid reinstatiating lists */ protected static final FaultyMember[] EMPTY_LIST = new FaultyMember[0]; - /* + /** * Holds a list of faulty members */ private ArrayList<FaultyMember> faultyMembers=null; @@ -48,6 +48,7 @@ public class ChannelException extends Ex /** * Constructor, creates a ChannelException with an error message + * @param message The error message * @see java.lang.Exception#Exception(String) */ public ChannelException(String message) { @@ -56,7 +57,7 @@ public class ChannelException extends Ex /** * Constructor, creates a ChannelException with an error message and a cause - * @param message String + * @param message The error message * @param cause Throwable * @see java.lang.Exception#Exception(String,Throwable) */ @@ -75,7 +76,7 @@ public class ChannelException extends Ex /** * Returns the message for this exception - * @return String + * @return the error message * @see java.lang.Exception#getMessage() */ @Override @@ -98,6 +99,7 @@ public class ChannelException extends Ex * Adds a faulty member, and the reason the member failed. * @param mbr Member * @param x Exception + * @return <code>true</code> if the member was added */ public boolean addFaultyMember(Member mbr, Exception x ) { return addFaultyMember(new FaultyMember(mbr,x)); @@ -106,6 +108,7 @@ public class ChannelException extends Ex /** * Adds a list of faulty members * @param mbrs FaultyMember[] + * @return the number of members added */ public int addFaultyMember(FaultyMember[] mbrs) { int result = 0; @@ -118,6 +121,7 @@ public class ChannelException extends Ex /** * Adds a faulty member * @param mbr FaultyMember + * @return <code>true</code> if the member was added */ public boolean addFaultyMember(FaultyMember mbr) { if ( this.faultyMembers==null ) this.faultyMembers = new ArrayList<>(); Modified: tomcat/trunk/java/org/apache/catalina/tribes/ChannelInterceptor.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/ChannelInterceptor.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/ChannelInterceptor.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/ChannelInterceptor.java Thu Jan 14 14:55:44 2016 @@ -86,7 +86,7 @@ public interface ChannelInterceptor exte * @param destination Member[] - the destination for this message * @param msg ChannelMessage - the message to be sent * @param payload InterceptorPayload - the payload, carrying an error handler and future useful data, can be null - * @throws ChannelException + * @throws ChannelException if a serialization error happens. * @see ErrorHandler * @see InterceptorPayload */ Modified: tomcat/trunk/java/org/apache/catalina/tribes/ChannelReceiver.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/ChannelReceiver.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/ChannelReceiver.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/ChannelReceiver.java Thu Jan 14 14:55:44 2016 @@ -29,7 +29,7 @@ public interface ChannelReceiver extends /** * Start listening for incoming messages on the host/port - * @throws java.io.IOException + * @throws java.io.IOException Listen failed */ public void start() throws java.io.IOException; Modified: tomcat/trunk/java/org/apache/catalina/tribes/Member.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/Member.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/Member.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/Member.java Thu Jan 14 14:55:44 2016 @@ -36,7 +36,7 @@ public interface Member { public static final byte[] SHUTDOWN_PAYLOAD = new byte[] {66, 65, 66, 89, 45, 65, 76, 69, 88}; /** - * Returns the name of this node, should be unique within the group. + * @return the name of this node, should be unique within the group. */ public String getName(); @@ -129,6 +129,7 @@ public interface Member { * Highly optimized version of serializing a member into a byte array * Returns a cached byte[] reference, do not modify this data * @param getalive calculate memberAlive time + * @return the data as a byte array */ public byte[] getData(boolean getalive); @@ -137,12 +138,14 @@ public interface Member { * Returns a cached byte[] reference, do not modify this data * @param getalive calculate memberAlive time * @param reset reset the cached data package, and create a new one + * @return the data as a byte array */ public byte[] getData(boolean getalive, boolean reset); /** * Length of a message obtained by {@link #getData(boolean)} or * {@link #getData(boolean, boolean)}. + * @return the data length */ public int getDataLength(); } Modified: tomcat/trunk/java/org/apache/catalina/tribes/MembershipService.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/MembershipService.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/MembershipService.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/MembershipService.java Thu Jan 14 14:55:44 2016 @@ -35,10 +35,12 @@ public interface MembershipService { * @param properties - to be used to configure the membership service. */ public void setProperties(java.util.Properties properties); + /** - * Returns the properties for the configuration used. + * @return the properties for the configuration used. */ public java.util.Properties getProperties(); + /** * Starts the membership service. If a membership listeners is added * the listener will start to receive membership events. @@ -65,7 +67,6 @@ public interface MembershipService { * stops broad casting the server * @throws java.lang.IllegalArgumentException if the level is incorrect. */ - public void stop(int level); /** @@ -73,36 +74,45 @@ public interface MembershipService { */ public boolean hasMembers(); - /** - * - * @param mbr Member - * @return Member + * Retrieve the specified member from the membership. + * @param mbr The member to retrieve + * @return the member */ public Member getMember(Member mbr); + /** - * Returns a list of all the members in the cluster. + * @return a list of all the members in the cluster. */ - public Member[] getMembers(); /** - * Returns the member object that defines this member + * Get the local member. + * @return the member object that defines this member + * @param incAliveTime <code>true</code> to set the alive time + * on the local member */ public Member getLocalMember(boolean incAliveTime); /** - * Return all members by name + * @return all members by name */ - public String[] getMembersByName() ; + public String[] getMembersByName(); /** - * Return the member by name + * Get a member. + * @param name The member name + * @return the member */ - public Member findMemberByName(String name) ; + public Member findMemberByName(String name); /** - * Sets the local member properties for broadcasting + * Sets the local member properties for broadcasting. + * + * @param listenHost Listen to host + * @param listenPort Listen to port + * @param securePort Use a secure port + * @param udpPort Use UDP */ public void setLocalMemberProperties(String listenHost, int listenPort, int securePort, int udpPort); @@ -114,7 +124,7 @@ public interface MembershipService { public void setMembershipListener(MembershipListener listener); /** - * removes the membership listener. + * Removes the membership listener. */ public void removeMembershipListener(); @@ -128,9 +138,9 @@ public interface MembershipService { public void setDomain(byte[] domain); /** - * Broadcasts a message to all members - * @param message - * @throws ChannelException + * Broadcasts a message to all members. + * @param message The message to broadcast + * @throws ChannelException Message broadcast failed */ public void broadcast(ChannelMessage message) throws ChannelException; Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/GroupChannel.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/GroupChannel.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/group/GroupChannel.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/group/GroupChannel.java Thu Jan 14 14:55:44 2016 @@ -367,7 +367,7 @@ public class GroupChannel extends Channe /** * Sets up the default implementation interceptor stack * if no interceptors have been added - * @throws ChannelException + * @throws ChannelException Cluster error */ protected synchronized void setupDefaultStack() throws ChannelException { if (getFirstInterceptor() != null && @@ -379,7 +379,7 @@ public class GroupChannel extends Channe /** * Validates the option flags that each interceptor is using and reports * an error if two interceptor share the same flag. - * @throws ChannelException + * @throws ChannelException Error with option flag */ protected void checkOptionFlags() throws ChannelException { StringBuilder conflicts = new StringBuilder(); @@ -412,9 +412,9 @@ public class GroupChannel extends Channe } /** - * Starts the channel + * Starts the channel. * @param svc int - what service to start - * @throws ChannelException + * @throws ChannelException Start error * @see org.apache.catalina.tribes.Channel#start(int) */ @Override @@ -429,9 +429,9 @@ public class GroupChannel extends Channe } /** - * Stops the channel + * Stops the channel. * @param svc int - * @throws ChannelException + * @throws ChannelException Stop error * @see org.apache.catalina.tribes.Channel#stop(int) */ @Override Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/RpcCallback.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/RpcCallback.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/group/RpcCallback.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/group/RpcCallback.java Thu Jan 14 14:55:44 2016 @@ -28,16 +28,17 @@ import org.apache.catalina.tribes.Member public interface RpcCallback { /** - * - * @param msg Serializable - * @return Serializable - null if no reply should be sent + * Allows sending a response to a recieved message. + * @param msg The message + * @param sender Member + * @return Serializable object, <code>null</code> if no reply should be sent */ public Serializable replyRequest(Serializable msg, Member sender); /** * If the reply has already been sent to the requesting thread, * the rpc callback can handle any data that comes in after the fact. - * @param msg Serializable + * @param msg The message * @param sender Member */ public void leftOver(Serializable msg, Member sender); Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/RpcChannel.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/RpcChannel.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/group/RpcChannel.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/group/RpcChannel.java Thu Jan 14 14:55:44 2016 @@ -75,7 +75,7 @@ public class RpcChannel implements Chann * @param channelOptions channel sender options * @param timeout long - timeout in milliseconds, if no reply is received within this time null is returned * @return Response[] - an array of response objects. - * @throws ChannelException + * @throws ChannelException Error sending message */ public Response[] send(Member[] destination, Serializable message, Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/GzipInterceptor.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/GzipInterceptor.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/GzipInterceptor.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/GzipInterceptor.java Thu Jan 14 14:55:44 2016 @@ -80,7 +80,7 @@ public class GzipInterceptor extends Cha /** * @param data Data to decompress * @return Decompressed data - * @throws IOException + * @throws IOException Compression error */ public static byte[] decompress(byte[] data) throws IOException { ByteArrayOutputStream bout = Modified: tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/SimpleCoordinator.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/SimpleCoordinator.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/SimpleCoordinator.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/group/interceptors/SimpleCoordinator.java Thu Jan 14 14:55:44 2016 @@ -58,7 +58,7 @@ public class SimpleCoordinator extends C /** * Override to receive view changes. * - * @param view + * @param view The members array */ protected void viewChange(final Member[] view) { } Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/io/ObjectReader.java Thu Jan 14 14:55:44 2016 @@ -93,9 +93,8 @@ public class ObjectReader { * @param len length in buffer * @param count whether to return the count * @return number of messages that was sent to callback (or -1 if count == false) - * @throws java.io.IOException */ - public int append(ByteBuffer data, int len, boolean count) throws java.io.IOException { + public int append(ByteBuffer data, int len, boolean count) { buffer.append(data,len); int pkgCnt = -1; if ( count ) pkgCnt = buffer.countPackages(); @@ -118,9 +117,8 @@ public class ObjectReader { * @see XByteBuffer#extractPackage(boolean) * * @return number of received packages/messages - * @throws java.io.IOException */ - public ChannelMessage[] execute() throws java.io.IOException { + public ChannelMessage[] execute() { int pkgCnt = buffer.countPackages(); ChannelMessage[] result = new ChannelMessage[pkgCnt]; for (int i=0; i<pkgCnt; i++) { Modified: tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/io/XByteBuffer.java Thu Jan 14 14:55:44 2016 @@ -84,7 +84,8 @@ public class XByteBuffer { /** * Constructs a new XByteBuffer.<br> * TODO use a pool of byte[] for performance - * @param size - the initial size of the byte buffer + * @param size the initial size of the byte buffer + * @param discard Flag for discarding invalid packages */ public XByteBuffer(int size, boolean discard) { buf = new byte[size]; @@ -128,7 +129,7 @@ public class XByteBuffer { } /** - * Returns the bytes in the buffer, in its exact length + * @return the bytes in the buffer, in its exact length */ public byte[] getBytes() { byte[] b = new byte[bufSize]; @@ -392,7 +393,6 @@ public class XByteBuffer { * @param b - the byte array containing the four bytes * @param off - the offset * @return the integer value constructed from the four bytes - * @exception java.lang.ArrayIndexOutOfBoundsException */ public static int toInt(byte[] b,int off){ return ( ( b[off+3]) & 0xFF) + @@ -406,7 +406,6 @@ public class XByteBuffer { * @param b - the byte array containing the four bytes * @param off - the offset * @return the long value constructed from the eight bytes - * @exception java.lang.ArrayIndexOutOfBoundsException */ public static long toLong(byte[] b,int off){ return ( ( (long) b[off+7]) & 0xFF) + @@ -421,9 +420,11 @@ public class XByteBuffer { /** - * Converts a boolean to a 1-byte array - * @param bool - the integer - * @return - 1-byte array + * Converts a boolean and put it in a byte array. + * @param bool the integer + * @param data the byte buffer in which the boolean will be placed + * @param offset the offset in the byte array + * @return the byte array */ public static byte[] toBytes(boolean bool, byte[] data, int offset) { data[offset] = (byte)(bool?1:0); @@ -431,7 +432,7 @@ public class XByteBuffer { } /** - * Converts a byte array entry to boolean + * Converts a byte array entry to boolean. * @param b byte array * @param offset within byte array * @return true if byte array entry is non-zero, false otherwise @@ -442,11 +443,13 @@ public class XByteBuffer { /** - * Converts an integer to four bytes - * @param n - the integer - * @return - four bytes in an array + * Converts an integer to four bytes. + * @param n the integer + * @param b the byte buffer in which the integer will be placed + * @param offset the offset in the byte array + * @return four bytes in an array */ - public static byte[] toBytes(int n,byte[] b, int offset) { + public static byte[] toBytes(int n, byte[] b, int offset) { b[offset+3] = (byte) (n); n >>>= 8; b[offset+2] = (byte) (n); @@ -458,9 +461,11 @@ public class XByteBuffer { } /** - * Converts an long to eight bytes - * @param n - the long - * @return - eight bytes in an array + * Converts an long to eight bytes. + * @param n the long + * @param b the byte buffer in which the integer will be placed + * @param offset the offset in the byte array + * @return eight bytes in an array */ public static byte[] toBytes(long n, byte[] b, int offset) { b[offset+7] = (byte) (n); @@ -482,7 +487,7 @@ public class XByteBuffer { } /** - * Similar to a String.IndexOf, but uses pure bytes + * Similar to a String.IndexOf, but uses pure bytes. * @param src - the source bytes to be searched * @param srcOff - offset on the source buffer * @param find - the string to be found within src @@ -567,7 +572,7 @@ public class XByteBuffer { * Serializes a message into cluster data * @param msg ClusterMessage * @return serialized content as byte[] array - * @throws IOException + * @throws IOException Serialization error */ public static byte[] serialize(Serializable msg) throws IOException { ByteArrayOutputStream outs = new ByteArrayOutputStream(); Modified: tomcat/trunk/java/org/apache/catalina/tribes/membership/McastService.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/membership/McastService.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/membership/McastService.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/membership/McastService.java Thu Jan 14 14:55:44 2016 @@ -91,7 +91,7 @@ public class McastService implements Mem } /** - * + * Sets the properties for the membership service. * @param properties * <br>All are required<br> * 1. mcastPort - the port to listen to<BR> @@ -115,7 +115,7 @@ public class McastService implements Mem } /** - * Return the properties, see setProperties + * {@inheritDoc} */ @Override public Properties getProperties() { @@ -123,14 +123,14 @@ public class McastService implements Mem } /** - * Return the local member name + * @return the local member name */ public String getLocalMemberName() { return localMember.toString() ; } /** - * Return the local member + * {@inheritDoc} */ @Override public Member getLocalMember(boolean alive) { @@ -139,7 +139,7 @@ public class McastService implements Mem } /** - * Sets the local member properties for broadcasting + * {@inheritDoc} */ @Override public void setLocalMemberProperties(String listenHost, int listenPort, int securePort, int udpPort) { Modified: tomcat/trunk/java/org/apache/catalina/tribes/membership/McastServiceImpl.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/membership/McastServiceImpl.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/membership/McastServiceImpl.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/membership/McastServiceImpl.java Thu Jan 14 14:55:44 2016 @@ -156,16 +156,19 @@ public class McastServiceImpl { protected final boolean localLoopbackDisabled; /** - * Create a new mcast service impl + * Create a new mcast service instance. * @param member - the local member * @param sendFrequency - the time (ms) in between pings sent out * @param expireTime - the time (ms) for a member to expire * @param port - the mcast port * @param bind - the bind address (not sure this is used yet) * @param mcastAddress - the mcast address + * @param ttl multicast ttl that will be set on the socket + * @param soTimeout Socket timeout * @param service - the callback service + * @param msgservice Message listener * @param localLoopbackDisabled - disable loopbackMode - * @throws IOException + * @throws IOException Init error */ public McastServiceImpl( MemberImpl member, @@ -298,7 +301,9 @@ public class McastServiceImpl { } /** - * Stops the service + * Stops the service. + * @param level Stop status + * @return <code>true</code> if the stop is complete * @throws IOException if the service fails to disconnect from the sockets */ public synchronized boolean stop(int level) throws IOException { @@ -336,7 +341,7 @@ public class McastServiceImpl { /** * Receive a datagram packet, locking wait - * @throws IOException + * @throws IOException Received failed */ public void receive() throws IOException { boolean checkexpired = true; @@ -477,16 +482,17 @@ public class McastServiceImpl { } /** - * Send a ping - * @throws IOException + * Send a ping. + * @param checkexpired <code>true</code> to check for expiration + * @throws IOException Send error */ - public void send(boolean checkexpired) throws IOException{ + public void send(boolean checkexpired) throws IOException { send(checkexpired,null); } private final Object sendLock = new Object(); - public void send(boolean checkexpired, DatagramPacket packet) throws IOException{ + public void send(boolean checkexpired, DatagramPacket packet) throws IOException { checkexpired = (checkexpired && (packet==null)); //ignore if we haven't started the sender //if ( (startLevel&Channel.MBR_TX_SEQ) != Channel.MBR_TX_SEQ ) return; Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/tipis/AbstractReplicatedMap.java Thu Jan 14 14:55:44 2016 @@ -156,12 +156,14 @@ public abstract class AbstractReplicated //------------------------------------------------------------------------------ /** - * Creates a new map + * Creates a new map. + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel * @param initialCapacity int - the size of this map, see HashMap * @param loadFactor float - load factor, see HashMap + * @param channelSendOptions Send options * @param cls - a list of classloaders to be used for deserialization of objects. * @param terminate - Flag for whether to terminate this map that failed to start. */ @@ -248,7 +250,7 @@ public abstract class AbstractReplicated * Sends a ping out to all the members in the cluster, not just map members * that this map is alive. * @param timeout long - * @throws ChannelException + * @throws ChannelException Send error */ protected void ping(long timeout) throws ChannelException { //send out a map membership message, only wait for the first reply @@ -307,7 +309,7 @@ public abstract class AbstractReplicated * Helper method to broadcast a message to all members in a channel * @param msgtype int * @param rpc boolean - * @throws ChannelException + * @throws ChannelException Send error */ protected void broadcast(int msgtype, boolean rpc) throws ChannelException { Member[] members = channel.getMembers(); @@ -400,6 +402,7 @@ public abstract class AbstractReplicated /** * Replicates any changes to the object since the last time * The object has to be primary, ie, if the object is a proxy or a backup, it will not be replicated<br> + * @param key The object to replicate * @param complete - if set to true, the object is replicated to its backup * if set to false, only objects that implement ReplicatedMapEntry and the isDirty() returns true will * be replicated @@ -1301,8 +1304,8 @@ public abstract class AbstractReplicated * @param offset int * @param length int * @param diff boolean - * @throws IOException - * @throws ClassNotFoundException + * @throws IOException IO error + * @throws ClassNotFoundException Deserialization error */ @SuppressWarnings("unchecked") public void apply(byte[] data, int offset, int length, boolean diff) throws IOException, ClassNotFoundException { Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/LazyReplicatedMap.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/tipis/LazyReplicatedMap.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/tipis/LazyReplicatedMap.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/tipis/LazyReplicatedMap.java Thu Jan 14 14:55:44 2016 @@ -72,11 +72,13 @@ public class LazyReplicatedMap<K,V> exte //------------------------------------------------------------------------------ /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel * @param initialCapacity int - the size of this map, see HashMap * @param loadFactor float - load factor, see HashMap + * @param cls Class loaders */ public LazyReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, int initialCapacity, float loadFactor, ClassLoader[] cls) { super(owner,channel,timeout,mapContextName,initialCapacity,loadFactor, Channel.SEND_OPTIONS_DEFAULT,cls, true); @@ -84,10 +86,12 @@ public class LazyReplicatedMap<K,V> exte /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel * @param initialCapacity int - the size of this map, see HashMap + * @param cls Class loaders */ public LazyReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, int initialCapacity, ClassLoader[] cls) { super(owner, channel,timeout,mapContextName,initialCapacity, AbstractReplicatedMap.DEFAULT_LOAD_FACTOR, Channel.SEND_OPTIONS_DEFAULT, cls, true); @@ -95,9 +99,11 @@ public class LazyReplicatedMap<K,V> exte /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel + * @param cls Class loaders */ public LazyReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, ClassLoader[] cls) { super(owner, channel,timeout,mapContextName, AbstractReplicatedMap.DEFAULT_INITIAL_CAPACITY,AbstractReplicatedMap.DEFAULT_LOAD_FACTOR,Channel.SEND_OPTIONS_DEFAULT, cls, true); @@ -105,9 +111,11 @@ public class LazyReplicatedMap<K,V> exte /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel + * @param cls Class loaders * @param terminate boolean - Flag for whether to terminate this map that failed to start. */ public LazyReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, ClassLoader[] cls, boolean terminate) { @@ -134,7 +142,7 @@ public class LazyReplicatedMap<K,V> exte * @param key Object * @param value Object * @return Member - the backup node - * @throws ChannelException + * @throws ChannelException Cluster error */ @Override protected Member[] publishEntryInfo(Object key, Object value) throws ChannelException { Modified: tomcat/trunk/java/org/apache/catalina/tribes/tipis/ReplicatedMap.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/tipis/ReplicatedMap.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/tipis/ReplicatedMap.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/tipis/ReplicatedMap.java Thu Jan 14 14:55:44 2016 @@ -64,11 +64,13 @@ public class ReplicatedMap<K,V> extends //-------------------------------------------------------------------------- /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel * @param initialCapacity int - the size of this map, see HashMap * @param loadFactor float - load factor, see HashMap + * @param cls Class loaders */ public ReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, int initialCapacity,float loadFactor, ClassLoader[] cls) { super(owner,channel, timeout, mapContextName, initialCapacity, loadFactor, Channel.SEND_OPTIONS_DEFAULT, cls, true); @@ -76,10 +78,12 @@ public class ReplicatedMap<K,V> extends /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel * @param initialCapacity int - the size of this map, see HashMap + * @param cls Class loaders */ public ReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, int initialCapacity, ClassLoader[] cls) { super(owner,channel, timeout, mapContextName, initialCapacity, AbstractReplicatedMap.DEFAULT_LOAD_FACTOR,Channel.SEND_OPTIONS_DEFAULT, cls, true); @@ -87,9 +91,11 @@ public class ReplicatedMap<K,V> extends /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel + * @param cls Class loaders */ public ReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, ClassLoader[] cls) { super(owner, channel, timeout, mapContextName,AbstractReplicatedMap.DEFAULT_INITIAL_CAPACITY, AbstractReplicatedMap.DEFAULT_LOAD_FACTOR, Channel.SEND_OPTIONS_DEFAULT, cls, true); @@ -97,9 +103,11 @@ public class ReplicatedMap<K,V> extends /** * Creates a new map + * @param owner The map owner * @param channel The channel to use for communication * @param timeout long - timeout for RPC messags * @param mapContextName String - unique name for this map, to allow multiple maps per channel + * @param cls Class loaders * @param terminate boolean - Flag for whether to terminate this map that failed to start. */ public ReplicatedMap(MapOwner owner, Channel channel, long timeout, String mapContextName, ClassLoader[] cls, boolean terminate) { @@ -125,7 +133,7 @@ public class ReplicatedMap<K,V> extends * @param key Object * @param value Object * @return Member - the backup node - * @throws ChannelException + * @throws ChannelException Cluster error */ @Override protected Member[] publishEntryInfo(Object key, Object value) throws ChannelException { Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/ReceiverBase.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/ReceiverBase.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/ReceiverBase.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/ReceiverBase.java Thu Jan 14 14:55:44 2016 @@ -175,7 +175,7 @@ public abstract class ReceiverBase imple * @param portstart Starting port for bind attempts * @param retries Number of times to attempt to bind (port incremented * between attempts) - * @throws IOException + * @throws IOException Socket bind error */ protected void bind(ServerSocket socket, int portstart, int retries) throws IOException { synchronized (bindLock) { @@ -202,11 +202,12 @@ public abstract class ReceiverBase imple /** * Same as bind() except it does it for the UDP port - * @param socket - * @param portstart - * @param retries - * @return int - * @throws IOException + * @param socket The socket to bind + * @param portstart Starting port for bind attempts + * @param retries Number of times to attempt to bind (port incremented + * between attempts) + * @return int The retry count + * @throws IOException Socket bind error */ protected int bindUdp(DatagramSocket socket, int portstart, int retries) throws IOException { InetSocketAddress addr = null; Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/RxTaskPool.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/RxTaskPool.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/RxTaskPool.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/RxTaskPool.java Thu Jan 14 14:55:44 2016 @@ -59,6 +59,7 @@ public class RxTaskPool { /** * Find an idle worker thread, if any. Could return null. + * @return a worker */ public AbstractRxTask getRxTask() { @@ -95,6 +96,7 @@ public class RxTaskPool { /** * Called by the worker thread to return itself to the * idle pool. + * @param worker The worker */ public void returnWorker (AbstractRxTask worker) { if ( running ) { Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioReplicationTask.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioReplicationTask.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioReplicationTask.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioReplicationTask.java Thu Jan 14 14:55:44 2016 @@ -134,8 +134,9 @@ public class BioReplicationTask extends * interest in OP_READ. When this method completes it * re-enables OP_READ and calls wakeup() on the selector * so the selector will resume watching this channel. + * @throws Exception IO exception or execute exception */ - protected void drainSocket () throws Exception { + protected void drainSocket() throws Exception { InputStream in = socket.getInputStream(); // loop while data available, channel is non-blocking byte[] buf = new byte[1024]; @@ -149,8 +150,8 @@ public class BioReplicationTask extends /** - * send a reply-acknowledgment (6,2,3) - * @param command + * Send a reply-acknowledgment (6,2,3) + * @param command The command to write */ protected void sendAck(byte[] command) { try { Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioSender.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioSender.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioSender.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/bio/BioSender.java Thu Jan 14 14:55:44 2016 @@ -99,6 +99,9 @@ public class BioSender extends AbstractS /** * Send message. + * @param data The data to send + * @param waitForAck Wait for an ack + * @throws IOException An IO error occured sending the message */ public void sendMessage(byte[] data, boolean waitForAck) throws IOException { IOException exception = null; @@ -129,9 +132,6 @@ public class BioSender extends AbstractS } - /** - * Name of this SockerSender - */ @Override public String toString() { StringBuilder buf = new StringBuilder("DataSender[("); @@ -143,8 +143,9 @@ public class BioSender extends AbstractS // --------------------------------------------------------- Protected Methods /** - * open real socket and set time out when waitForAck is enabled - * is socket open return directly + * Open real socket and set time out when waitForAck is enabled + * is socket open return directly. + * @throws IOException Error opening socket */ protected void openSocket() throws IOException { if(isConnected()) return ; @@ -215,8 +216,10 @@ public class BioSender extends AbstractS * @see #openSocket() * @see #sendMessage(byte[], boolean) * - * @param data - * data to send + * @param data Data to send + * @param reconnect Do a reconnect (close socket then reopen) + * @param waitForAck Wait for an acknowledgement + * @throws IOException IO error writing data * @since 5.5.10 */ @@ -234,8 +237,7 @@ public class BioSender extends AbstractS /** * Wait for Acknowledgement from other server. * FIXME Please, not wait only for three characters, better control that the wait ack message is correct. - * @throws java.io.IOException - * @throws java.net.SocketTimeoutException + * @throws IOException An IO error occurred */ protected void waitForAck() throws java.io.IOException { try { Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReceiver.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReceiver.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReceiver.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReceiver.java Thu Jan 14 14:55:44 2016 @@ -236,10 +236,9 @@ public class NioReceiver extends Receive /** - * get data from channel and store in byte array + * Get data from channel and store in byte array * send it to cluster - * @throws IOException - * @throws java.nio.channels.ClosedChannelException + * @throws IOException IO error */ protected void listen() throws Exception { if (doListen()) { @@ -397,6 +396,11 @@ public class NioReceiver extends Receive /** * Register the given channel with the given selector for * the given operations of interest + * @param selector The selector to use + * @param channel The channel + * @param ops The operations to register + * @param attach Attachment object + * @throws Exception IO error with channel */ protected void registerChannel(Selector selector, SelectableChannel channel, @@ -433,6 +437,7 @@ public class NioReceiver extends Receive * channel returns an EOF condition, it is closed here, which * automatically invalidates the associated key. The selector * will then de-register the channel on the next select call. + * @throws Exception IO error with channel */ protected void readDataFromSocket(SelectionKey key) throws Exception { NioReplicationTask task = (NioReplicationTask) getTaskPool().getRxTask(); Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReplicationTask.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReplicationTask.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReplicationTask.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioReplicationTask.java Thu Jan 14 14:55:44 2016 @@ -130,6 +130,7 @@ public class NioReplicationTask extends * updated to remove OP_READ. This will cause the selector * to ignore read-readiness for this channel while the * worker thread is servicing it. + * @param key The key to process */ public synchronized void serviceChannel (SelectionKey key) { if ( log.isTraceEnabled() ) log.trace("About to service key:"+key); @@ -147,6 +148,9 @@ public class NioReplicationTask extends * interest in OP_READ. When this method completes it * re-enables OP_READ and calls wakeup() on the selector * so the selector will resume watching this channel. + * @param key The key to process + * @param reader The reader + * @throws Exception IO error */ protected void drainChannel (final SelectionKey key, ObjectReader reader) throws Exception { reader.access(); @@ -292,10 +296,12 @@ public class NioReplicationTask extends /** - * send a reply-acknowledgement (6,2,3), sends it doing a busy write, the ACK is so small - * that it should always go to the buffer - * @param key - * @param channel + * Send a reply-acknowledgement (6,2,3), sends it doing a busy write, the ACK is so small + * that it should always go to the buffer. + * @param key The key to use + * @param channel The channel + * @param command The command to write + * @param udpaddr Target address */ protected void sendAck(SelectionKey key, WritableByteChannel channel, byte[] command, SocketAddress udpaddr) { try { Modified: tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioSender.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioSender.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioSender.java (original) +++ tomcat/trunk/java/org/apache/catalina/tribes/transport/nio/NioSender.java Thu Jan 14 14:55:44 2016 @@ -75,10 +75,11 @@ public class NioSender extends AbstractS } /** - * State machine to send data - * @param key SelectionKey - * @return boolean - * @throws IOException + * State machine to send data. + * @param key The key to use + * @param waitForAck Wait for an ack + * @return <code>true</code> if the processing was successful + * @throws IOException An IO error occurred */ public boolean process(SelectionKey key, boolean waitForAck) throws IOException { int ops = key.readyOps(); Modified: tomcat/trunk/java/org/apache/catalina/users/MemoryUser.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/users/MemoryUser.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/users/MemoryUser.java (original) +++ tomcat/trunk/java/org/apache/catalina/users/MemoryUser.java Thu Jan 14 14:55:44 2016 @@ -254,6 +254,7 @@ public class MemoryUser extends Abstract * the reader that processes this entry will accept either * <code>username</code> or <code>name</code> for the username * property.</p> + * @return the XML representation */ public String toXml() { Modified: tomcat/trunk/java/org/apache/catalina/users/MemoryUserDatabase.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/users/MemoryUserDatabase.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/users/MemoryUserDatabase.java (original) +++ tomcat/trunk/java/org/apache/catalina/users/MemoryUserDatabase.java Thu Jan 14 14:55:44 2016 @@ -139,7 +139,7 @@ public class MemoryUserDatabase implemen /** - * Return the set of {@link Group}s defined in this user database. + * @return the set of {@link Group}s defined in this user database. */ @Override public Iterator<Group> getGroups() { @@ -152,7 +152,7 @@ public class MemoryUserDatabase implemen /** - * Return the unique global identifier of this user database. + * @return the unique global identifier of this user database. */ @Override public String getId() { @@ -163,7 +163,7 @@ public class MemoryUserDatabase implemen /** - * Return the relative or absolute pathname to the persistent storage file. + * @return the relative or absolute pathname to the persistent storage file. */ public String getPathname() { @@ -187,7 +187,7 @@ public class MemoryUserDatabase implemen /** - * Returning the readonly status of the user database + * @return the readonly status of the user database */ public boolean getReadonly() { @@ -209,7 +209,7 @@ public class MemoryUserDatabase implemen /** - * Return the set of {@link Role}s defined in this user database. + * @return the set of {@link Role}s defined in this user database. */ @Override public Iterator<Role> getRoles() { @@ -222,7 +222,7 @@ public class MemoryUserDatabase implemen /** - * Return the set of {@link User}s defined in this user database. + * @return the set of {@link User}s defined in this user database. */ @Override public Iterator<User> getUsers() { @@ -485,6 +485,7 @@ public class MemoryUserDatabase implemen /** * Check for permissions to save this user database to persistent storage * location. + * @return <code>true</code> if the database is writable */ public boolean isWriteable() { Modified: tomcat/trunk/java/org/apache/catalina/util/CharsetMapper.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/CharsetMapper.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/CharsetMapper.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/CharsetMapper.java Thu Jan 14 14:55:44 2016 @@ -97,6 +97,7 @@ public class CharsetMapper { * content type header. * * @param locale The locale for which to calculate a character set + * @return the charset name */ public String getCharset(Locale locale) { // Match full language_country_variant first, then language_country, Modified: tomcat/trunk/java/org/apache/catalina/util/DOMWriter.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/DOMWriter.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/DOMWriter.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/DOMWriter.java Thu Jan 14 14:55:44 2016 @@ -52,7 +52,10 @@ public class DOMWriter { } - /** Prints the specified node, recursively. */ + /** + * Prints the specified node, recursively. + * @param node The node to output + */ public void print(Node node) { // is there anything to do? @@ -161,7 +164,11 @@ public class DOMWriter { } - /** Returns a sorted list of attributes. */ + /** + * Returns a sorted list of attributes. + * @param attrs The map to sort + * @return a sorted attribute array + */ protected Attr[] sortAttributes(NamedNodeMap attrs) { if (attrs == null) { return new Attr[0]; @@ -195,7 +202,11 @@ public class DOMWriter { } - /** Normalizes the given string. */ + /** + * Normalizes the given string. + * @param s The string to escape + * @return the escaped string + */ protected String normalize(String s) { if (s == null) { return ""; Modified: tomcat/trunk/java/org/apache/catalina/util/Extension.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/Extension.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/Extension.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/Extension.java Thu Jan 14 14:55:44 2016 @@ -176,6 +176,7 @@ public final class Extension { * <code>false</code>. * * @param required Extension of the required optional package + * @return <code>true</code> if the extension is satisfied */ public boolean isCompatibleWith(Extension required) { @@ -213,9 +214,6 @@ public final class Extension { } - /** - * Return a String representation of this object. - */ @Override public String toString() { Modified: tomcat/trunk/java/org/apache/catalina/util/ExtensionValidator.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/ExtensionValidator.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/ExtensionValidator.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/ExtensionValidator.java Thu Jan 14 14:55:44 2016 @@ -121,6 +121,7 @@ public final class ExtensionValidator { * application * * @return true if all required extensions satisfied + * @throws IOException Error reading resources needed for validation */ public static synchronized boolean validateApplication( WebResourceRoot resources, @@ -168,6 +169,7 @@ public final class ExtensionValidator { * it to the container's manifest resources. * * @param jarFile The system JAR whose manifest to add + * @throws IOException Error reading JAR file */ public static void addSystemResource(File jarFile) throws IOException { try (InputStream is = new FileInputStream(jarFile)) { Modified: tomcat/trunk/java/org/apache/catalina/util/IOTools.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/IOTools.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/IOTools.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/IOTools.java Thu Jan 14 14:55:44 2016 @@ -42,6 +42,7 @@ public class IOTools { * @param reader the reader to read from. * @param writer the writer to write to. * @param buf the char array to use as a buffer + * @throws IOException IO error */ public static void flow( Reader reader, Writer writer, char[] buf ) throws IOException { @@ -52,6 +53,12 @@ public class IOTools { } /** + * Read input from reader and write it to writer until there is no more + * input from reader. + * + * @param reader the reader to read from. + * @param writer the writer to write to. + * @throws IOException IO error * @see #flow( Reader, Writer, char[] ) */ public static void flow( Reader reader, Writer writer ) Modified: tomcat/trunk/java/org/apache/catalina/util/Introspection.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/Introspection.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/Introspection.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/Introspection.java Thu Jan 14 14:55:44 2016 @@ -44,6 +44,8 @@ public class Introspection { * * Note: This method assumes that the method name has already been checked * for correctness. + * @param setter The setter method + * @return the bean property name */ public static String getPropertyName(Method setter) { return Introspector.decapitalize(setter.getName().substring(3)); @@ -91,6 +93,8 @@ public class Introspection { /** * Obtain the declared fields for a class taking account of any security * manager that may be configured. + * @param clazz The class to introspect + * @return the class fields as an array */ public static Field[] getDeclaredFields(final Class<?> clazz) { Field[] fields = null; @@ -112,6 +116,8 @@ public class Introspection { /** * Obtain the declared methods for a class taking account of any security * manager that may be configured. + * @param clazz The class to introspect + * @return the class methods as an array */ public static Method[] getDeclaredMethods(final Class<?> clazz) { Method[] methods = null; @@ -134,6 +140,10 @@ public class Introspection { * Attempt to load a class using the given Container's class loader. If the * class cannot be loaded, a debug level log message will be written to the * Container's log and null will be returned. + * @param context The class loader of this context will be used to attemmpt + * to load the class + * @param className The class name + * @return the loaded class or <code>null</code> if loading failed */ public static Class<?> loadClass(Context context, String className) { ClassLoader cl = context.getLoader().getClassLoader(); Modified: tomcat/trunk/java/org/apache/catalina/util/MIME2Java.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/MIME2Java.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/MIME2Java.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/MIME2Java.java Thu Jan 14 14:55:44 2016 @@ -30,441 +30,440 @@ import java.util.Map; * as <code>TXDocument#setEncoding</code> and <code>DTD#setEncoding</code>. * <p>Java encoding names are used on <var>encoding</var> parameters to * methods such as <code>TXDocument#printWithFormat</code> and <code>DTD#printExternal</code>. - * <P> * <TABLE BORDER="0" WIDTH="100%"> * <caption>MIME charset name to Java encoding name mapping</caption> * <TR> - * <TD WIDTH="33%"> - * <P ALIGN="CENTER"><B>Common Name</B> + * <TD> + * <P><B>Common Name</B> * </TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER"><B>Use this name in XML files</B> + * <TD> + * <P><B>Use this name in XML files</B> * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER"><B>Name Type</B> + * <TD> + * <P><B>Name Type</B> * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER"><B>Xerces converts to this Java Encoder Name</B> + * <TD> + * <P><B>Xerces converts to this Java Encoder Name</B> * </TD> * </TR> * <TR> - * <TD WIDTH="33%">8 bit Unicode</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">UTF-8 + * <TD>8 bit Unicode</TD> + * <TD> + * <P>UTF-8 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">UTF8 + * <TD> + * <P>UTF8 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin 1</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-1 + * <TD>ISO Latin 1</TD> + * <TD> + * <P>ISO-8859-1 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-1 + * <TD> + * <P>ISO-8859-1 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin 2</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-2 + * <TD>ISO Latin 2</TD> + * <TD> + * <P>ISO-8859-2 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-2 + * <TD> + * <P>ISO-8859-2 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin 3</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-3 + * <TD>ISO Latin 3</TD> + * <TD> + * <P>ISO-8859-3 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-3 + * <TD> + * <P>ISO-8859-3 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin 4</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-4 + * <TD>ISO Latin 4</TD> + * <TD> + * <P>ISO-8859-4 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-4 + * <TD> + * <P>ISO-8859-4 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin Cyrillic</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-5 + * <TD>ISO Latin Cyrillic</TD> + * <TD> + * <P>ISO-8859-5 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-5 + * <TD> + * <P>ISO-8859-5 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin Arabic</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-6 + * <TD>ISO Latin Arabic</TD> + * <TD> + * <P>ISO-8859-6 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-6 + * <TD> + * <P>ISO-8859-6 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin Greek</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-7 + * <TD>ISO Latin Greek</TD> + * <TD> + * <P>ISO-8859-7 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-7 + * <TD> + * <P>ISO-8859-7 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin Hebrew</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-8 + * <TD>ISO Latin Hebrew</TD> + * <TD> + * <P>ISO-8859-8 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-8 + * <TD> + * <P>ISO-8859-8 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">ISO Latin 5</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ISO-8859-9 + * <TD>ISO Latin 5</TD> + * <TD> + * <P>ISO-8859-9 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">ISO-8859-9 + * <TD> + * <P>ISO-8859-9 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: US</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-us + * <TD>EBCDIC: US</TD> + * <TD> + * <P>ebcdic-cp-us * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp037 + * <TD> + * <P>cp037 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Canada</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-ca + * <TD>EBCDIC: Canada</TD> + * <TD> + * <P>ebcdic-cp-ca * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp037 + * <TD> + * <P>cp037 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Netherlands</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-nl + * <TD>EBCDIC: Netherlands</TD> + * <TD> + * <P>ebcdic-cp-nl * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp037 + * <TD> + * <P>cp037 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Denmark</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-dk + * <TD>EBCDIC: Denmark</TD> + * <TD> + * <P>ebcdic-cp-dk * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp277 + * <TD> + * <P>cp277 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Norway</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-no + * <TD>EBCDIC: Norway</TD> + * <TD> + * <P>ebcdic-cp-no * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp277 + * <TD> + * <P>cp277 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Finland</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-fi + * <TD>EBCDIC: Finland</TD> + * <TD> + * <P>ebcdic-cp-fi * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp278 + * <TD> + * <P>cp278 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Sweden</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-se + * <TD>EBCDIC: Sweden</TD> + * <TD> + * <P>ebcdic-cp-se * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp278 + * <TD> + * <P>cp278 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Italy</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-it + * <TD>EBCDIC: Italy</TD> + * <TD> + * <P>ebcdic-cp-it * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp280 + * <TD> + * <P>cp280 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Spain, Latin America</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-es + * <TD>EBCDIC: Spain, Latin America</TD> + * <TD> + * <P>ebcdic-cp-es * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp284 + * <TD> + * <P>cp284 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Great Britain</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-gb + * <TD>EBCDIC: Great Britain</TD> + * <TD> + * <P>ebcdic-cp-gb * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp285 + * <TD> + * <P>cp285 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: France</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-fr + * <TD>EBCDIC: France</TD> + * <TD> + * <P>ebcdic-cp-fr * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp297 + * <TD> + * <P>cp297 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Arabic</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-ar1 + * <TD>EBCDIC: Arabic</TD> + * <TD> + * <P>ebcdic-cp-ar1 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp420 + * <TD> + * <P>cp420 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Hebrew</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-he + * <TD>EBCDIC: Hebrew</TD> + * <TD> + * <P>ebcdic-cp-he * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp424 + * <TD> + * <P>cp424 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Switzerland</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-ch + * <TD>EBCDIC: Switzerland</TD> + * <TD> + * <P>ebcdic-cp-ch * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp500 + * <TD> + * <P>cp500 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Roece</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-roece + * <TD>EBCDIC: Roece</TD> + * <TD> + * <P>ebcdic-cp-roece * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp870 + * <TD> + * <P>cp870 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Yogoslavia</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-yu + * <TD>EBCDIC: Yogoslavia</TD> + * <TD> + * <P>ebcdic-cp-yu * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp870 + * <TD> + * <P>cp870 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Iceland</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-is + * <TD>EBCDIC: Iceland</TD> + * <TD> + * <P>ebcdic-cp-is * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp871 + * <TD> + * <P>cp871 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">EBCDIC: Urdu</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">ebcdic-cp-ar2 + * <TD>EBCDIC: Urdu</TD> + * <TD> + * <P>ebcdic-cp-ar2 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">IANA + * <TD> + * <P>IANA * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">cp918 + * <TD> + * <P>cp918 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Chinese for PRC, mixed 1/2 byte</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">gb2312 + * <TD>Chinese for PRC, mixed 1/2 byte</TD> + * <TD> + * <P>gb2312 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">GB2312 + * <TD> + * <P>GB2312 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Extended Unix Code, packed for Japanese</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">euc-jp + * <TD>Extended Unix Code, packed for Japanese</TD> + * <TD> + * <P>euc-jp * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">eucjis + * <TD> + * <P>eucjis * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Japanese: iso-2022-jp</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">iso-2020-jp + * <TD>Japanese: iso-2022-jp</TD> + * <TD> + * <P>iso-2020-jp * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">JIS + * <TD> + * <P>JIS * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Japanese: Shift JIS</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">Shift_JIS + * <TD>Japanese: Shift JIS</TD> + * <TD> + * <P>Shift_JIS * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">SJIS + * <TD> + * <P>SJIS * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Chinese: Big5</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">Big5 + * <TD>Chinese: Big5</TD> + * <TD> + * <P>Big5 * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">Big5 + * <TD> + * <P>Big5 * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Extended Unix Code, packed for Korean</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">euc-kr + * <TD>Extended Unix Code, packed for Korean</TD> + * <TD> + * <P>euc-kr * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">iso2022kr + * <TD> + * <P>iso2022kr * </TD> * </TR> * <TR> - * <TD WIDTH="33%">Cyrillic</TD> - * <TD WIDTH="15%"> - * <P ALIGN="CENTER">koi8-r + * <TD>Cyrillic</TD> + * <TD> + * <P>koi8-r * </TD> - * <TD WIDTH="12%"> - * <P ALIGN="CENTER">MIME + * <TD> + * <P>MIME * </TD> - * <TD WIDTH="31%"> - * <P ALIGN="CENTER">koi8-r + * <TD> + * <P>koi8-r * </TD> * </TR> * </TABLE> Modified: tomcat/trunk/java/org/apache/catalina/util/ParameterMap.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/ParameterMap.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/ParameterMap.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/ParameterMap.java Thu Jan 14 14:55:44 2016 @@ -96,7 +96,7 @@ public final class ParameterMap<K,V> ext /** - * Return the locked state of this parameter map. + * @return the locked state of this parameter map. */ public boolean isLocked() { Modified: tomcat/trunk/java/org/apache/catalina/util/ResourceSet.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/ResourceSet.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/ResourceSet.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/ResourceSet.java Thu Jan 14 14:55:44 2016 @@ -99,7 +99,7 @@ public final class ResourceSet<T> extend /** - * Return the locked state of this parameter map. + * @return the locked state of this parameter map. */ public boolean isLocked() { Modified: tomcat/trunk/java/org/apache/catalina/util/ServerInfo.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/ServerInfo.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/ServerInfo.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/ServerInfo.java Thu Jan 14 14:55:44 2016 @@ -85,7 +85,7 @@ public class ServerInfo { /** - * Return the server identification for this version of Tomcat. + * @return the server identification for this version of Tomcat. */ public static String getServerInfo() { @@ -94,7 +94,7 @@ public class ServerInfo { } /** - * Return the server built time for this version of Tomcat. + * @return the server built time for this version of Tomcat. */ public static String getServerBuilt() { @@ -103,7 +103,7 @@ public class ServerInfo { } /** - * Return the server's version number. + * @return the server's version number. */ public static String getServerNumber() { Modified: tomcat/trunk/java/org/apache/catalina/util/SessionConfig.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/SessionConfig.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/SessionConfig.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/SessionConfig.java Thu Jan 14 14:55:44 2016 @@ -28,7 +28,8 @@ public class SessionConfig { /** * Determine the name to use for the session cookie for the provided * context. - * @param context + * @param context The context + * @return the cookie name for the context */ public static String getSessionCookieName(Context context) { @@ -44,7 +45,8 @@ public class SessionConfig { /** * Determine the name to use for the session cookie for the provided * context. - * @param context + * @param context The context + * @return the parameter name for the session */ public static String getSessionUriParamName(Context context) { Modified: tomcat/trunk/java/org/apache/catalina/util/Strftime.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/Strftime.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/Strftime.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/Strftime.java Thu Jan 14 14:55:44 2016 @@ -139,6 +139,7 @@ public class Strftime { /** * Change the timezone used to format dates * + * @param timeZone The new time zone * @see SimpleDateFormat#setTimeZone */ public void setTimeZone( TimeZone timeZone ) { Modified: tomcat/trunk/java/org/apache/catalina/util/XMLWriter.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/util/XMLWriter.java?rev=1724626&r1=1724625&r2=1724626&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/util/XMLWriter.java (original) +++ tomcat/trunk/java/org/apache/catalina/util/XMLWriter.java Thu Jan 14 14:55:44 2016 @@ -22,8 +22,6 @@ import java.io.Writer; /** * XMLWriter helper class. - * - * @author <a href="mailto:r...@apache.org">Remy Maucherat</a> */ public class XMLWriter { @@ -68,7 +66,7 @@ public class XMLWriter { /** - * Constructor. + * New XML writer utility that will store its data in an internal buffer. */ public XMLWriter() { this(null); @@ -76,7 +74,10 @@ public class XMLWriter { /** - * Constructor. + * New XML writer utility that will store its data in an internal buffer + * and can write it to the specified writer. + * @see sendData + * @param writer The writer to use */ public XMLWriter(Writer writer) { this.writer = writer; @@ -204,7 +205,8 @@ public class XMLWriter { /** - * Send data and reinitializes buffer. + * Send data and reinitializes buffer, if a writer has been specified. + * @throws IOException Error writing XML data */ public void sendData() throws IOException { --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org