You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by er...@apache.org on 2007/05/21 02:50:03 UTC

svn commit: r539987 [1/2] - in /directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages: components/ value/

Author: erodriguez
Date: Sun May 20 17:50:01 2007
New Revision: 539987

URL: http://svn.apache.org/viewvc?view=rev&rev=539987
Log:
Javadocs and minor warning clean-up.

Modified:
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Authenticator.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/AuthenticatorModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPartModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncAsRepPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKdcRepPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbCredPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPartModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTgsRepPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPart.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPartModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/MessageComponentType.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Ticket.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/TicketModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Checksum.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedData.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedDataModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptionTypeInfoEntry.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/HostAddresses.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KdcOptions.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosPrincipalModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosTime.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KrbCredInfo.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequest.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequestEntry.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Options.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationData.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationDataModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PrincipalName.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PrincipalNameModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/RequestBody.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/RequestBodyModifier.java
    directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/TransitedEncoding.java

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Authenticator.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Authenticator.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Authenticator.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Authenticator.java Sun May 20 17:50:01 2007
@@ -35,6 +35,9 @@
  */
 public class Authenticator implements Encodable
 {
+    /**
+     * Constant for the authenticator version number.
+     */
     public static final int AUTHENTICATOR_VNO = 5;
 
     private int versionNumber;
@@ -47,17 +50,40 @@
     private AuthorizationData authorizationData;
 
 
-    public Authenticator(KerberosPrincipal clientPrincipal, Checksum checksum, int clientMicroSecond,
-        KerberosTime clientTime, EncryptionKey subSessionKey, int sequenceNumber, AuthorizationData authorizationData)
+    /**
+     * Creates a new instance of Authenticator.
+     *
+     * @param clientPrincipal
+     * @param checksum
+     * @param clientMicroSecond
+     * @param clientTime
+     * @param subSessionKey
+     * @param sequenceNumber
+     * @param authorizationData
+     */
+    public Authenticator( KerberosPrincipal clientPrincipal, Checksum checksum, int clientMicroSecond,
+        KerberosTime clientTime, EncryptionKey subSessionKey, int sequenceNumber, AuthorizationData authorizationData )
     {
         this( AUTHENTICATOR_VNO, clientPrincipal, checksum, clientMicroSecond, clientTime, subSessionKey,
             sequenceNumber, authorizationData );
     }
 
 
-    public Authenticator(int versionNumber, KerberosPrincipal clientPrincipal, Checksum checksum,
+    /**
+     * Creates a new instance of Authenticator.
+     *
+     * @param versionNumber
+     * @param clientPrincipal
+     * @param checksum
+     * @param clientMicroSecond
+     * @param clientTime
+     * @param subSessionKey
+     * @param sequenceNumber
+     * @param authorizationData
+     */
+    public Authenticator( int versionNumber, KerberosPrincipal clientPrincipal, Checksum checksum,
         int clientMicroSecond, KerberosTime clientTime, EncryptionKey subSessionKey, int sequenceNumber,
-        AuthorizationData authorizationData)
+        AuthorizationData authorizationData )
     {
         this.versionNumber = versionNumber;
         this.clientPrincipal = clientPrincipal;
@@ -70,48 +96,88 @@
     }
 
 
+    /**
+     * Returns the client {@link KerberosPrincipal}.
+     *
+     * @return The client {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getClientPrincipal()
     {
         return clientPrincipal;
     }
 
 
+    /**
+     * Returns the client {@link KerberosTime}.
+     *
+     * @return The client {@link KerberosTime}.
+     */
     public KerberosTime getClientTime()
     {
         return clientTime;
     }
 
 
+    /**
+     * Returns the client microsecond.
+     *
+     * @return The client microsecond.
+     */
     public int getClientMicroSecond()
     {
         return clientMicroSecond;
     }
 
 
+    /**
+     * Returns the {@link AuthorizationData}.
+     *
+     * @return The {@link AuthorizationData}.
+     */
     public AuthorizationData getAuthorizationData()
     {
         return authorizationData;
     }
 
 
+    /**
+     * Returns the {@link Checksum}.
+     *
+     * @return The {@link Checksum}.
+     */
     public Checksum getChecksum()
     {
         return checksum;
     }
 
 
+    /**
+     * Returns the sequence number.
+     *
+     * @return The sequence number.
+     */
     public int getSequenceNumber()
     {
         return sequenceNumber;
     }
 
 
+    /**
+     * Returns the sub-session key.
+     *
+     * @return The sub-session key.
+     */
     public EncryptionKey getSubSessionKey()
     {
         return subSessionKey;
     }
 
 
+    /**
+     * Returns the version number of the {@link Authenticator}.
+     *
+     * @return The version number of the {@link Authenticator}.
+     */
     public int getVersionNumber()
     {
         return versionNumber;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/AuthenticatorModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/AuthenticatorModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/AuthenticatorModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/AuthenticatorModifier.java Sun May 20 17:50:01 2007
@@ -46,9 +46,13 @@
     private AuthorizationData authorizationData;
 
 
+    /**
+     * Returns the {@link Authenticator}.
+     *
+     * @return The {@link Authenticator}.
+     */
     public Authenticator getAuthenticator()
     {
-
         KerberosPrincipal clientPrincipal = clientModifier.getKerberosPrincipal();
 
         return new Authenticator( versionNumber, clientPrincipal, checksum, clientMicroSecond, clientTime,
@@ -56,54 +60,99 @@
     }
 
 
+    /**
+     * Sets the version number.
+     *
+     * @param versionNumber
+     */
     public void setVersionNumber( int versionNumber )
     {
         this.versionNumber = versionNumber;
     }
 
 
+    /**
+     * Sets the client {@link PrincipalName}.
+     *
+     * @param name
+     */
     public void setClientName( PrincipalName name )
     {
         clientModifier.setPrincipalName( name );
     }
 
 
+    /**
+     * Sets the client realm.
+     *
+     * @param realm
+     */
     public void setClientRealm( String realm )
     {
         clientModifier.setRealm( realm );
     }
 
 
+    /**
+     * Sets the {@link AuthorizationData}.
+     *
+     * @param data
+     */
     public void setAuthorizationData( AuthorizationData data )
     {
         authorizationData = data;
     }
 
 
+    /**
+     * Sets the {@link Checksum}.
+     *
+     * @param checksum
+     */
     public void setChecksum( Checksum checksum )
     {
         this.checksum = checksum;
     }
 
 
+    /**
+     * Sets the client microsecond.
+     *
+     * @param microSecond
+     */
     public void setClientMicroSecond( int microSecond )
     {
         clientMicroSecond = microSecond;
     }
 
 
+    /**
+     * Sets the client {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setClientTime( KerberosTime time )
     {
         clientTime = time;
     }
 
 
+    /**
+     * Sets the sequence number.
+     *
+     * @param number
+     */
     public void setSequenceNumber( int number )
     {
         sequenceNumber = number;
     }
 
 
+    /**
+     * Sets the sub-session {@link EncryptionKey}.
+     *
+     * @param sessionKey
+     */
     public void setSubSessionKey( EncryptionKey sessionKey )
     {
         subSessionKey = sessionKey;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPart.java Sun May 20 17:50:01 2007
@@ -41,7 +41,15 @@
     private Integer sequenceNumber; //optional
 
 
-    public EncApRepPart(KerberosTime clientTime, int cusec, EncryptionKey subSessionKey, Integer sequenceNumber)
+    /**
+     * Creates a new instance of EncApRepPart.
+     *
+     * @param clientTime
+     * @param cusec
+     * @param subSessionKey
+     * @param sequenceNumber
+     */
+    public EncApRepPart( KerberosTime clientTime, int cusec, EncryptionKey subSessionKey, Integer sequenceNumber )
     {
         super( MessageType.ENC_AP_REP_PART );
 
@@ -52,24 +60,44 @@
     }
 
 
+    /**
+     * Returns the client {@link KerberosTime}.
+     *
+     * @return The client {@link KerberosTime}.
+     */
     public KerberosTime getClientTime()
     {
         return clientTime;
     }
 
 
+    /**
+     * Returns the client microsecond.
+     *
+     * @return The client microsecond.
+     */
     public int getClientMicroSecond()
     {
         return cusec;
     }
 
 
+    /**
+     * Returns the sequence number.
+     *
+     * @return The sequence number.
+     */
     public Integer getSequenceNumber()
     {
         return sequenceNumber;
     }
 
 
+    /**
+     * Returns the sub-session {@link EncryptionKey}.
+     *
+     * @return The sub-session {@link EncryptionKey}.
+     */
     public EncryptionKey getSubSessionKey()
     {
         return subSessionKey;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPartModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPartModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPartModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncApRepPartModifier.java Sun May 20 17:50:01 2007
@@ -36,30 +36,55 @@
     private Integer sequenceNumber; //optional
 
 
+    /**
+     * Returns the {@link EncApRepPart}.
+     *
+     * @return The {@link EncApRepPart}.
+     */
     public EncApRepPart getEncApRepPart()
     {
         return new EncApRepPart( clientTime, cusec, subSessionKey, sequenceNumber );
     }
 
 
+    /**
+     * Sets the client {@link KerberosTime}.
+     *
+     * @param clientTime
+     */
     public void setClientTime( KerberosTime clientTime )
     {
         this.clientTime = clientTime;
     }
 
 
+    /**
+     * Sets the client microsecond.
+     *
+     * @param cusec
+     */
     public void setClientMicroSecond( int cusec )
     {
         this.cusec = cusec;
     }
 
 
+    /**
+     * Sets the sub-session {@link EncryptionKey}.
+     *
+     * @param subSessionKey
+     */
     public void setSubSessionKey( EncryptionKey subSessionKey )
     {
         this.subSessionKey = subSessionKey;
     }
 
 
+    /**
+     * Sets the sequence number.
+     *
+     * @param sequenceNumber
+     */
     public void setSequenceNumber( Integer sequenceNumber )
     {
         this.sequenceNumber = sequenceNumber;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncAsRepPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncAsRepPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncAsRepPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncAsRepPart.java Sun May 20 17:50:01 2007
@@ -38,7 +38,19 @@
 public class EncAsRepPart extends EncKdcRepPart
 {
     /**
-     * Class constructor
+     * Creates a new instance of EncAsRepPart.
+     *
+     * @param key
+     * @param lastReq
+     * @param nonce
+     * @param keyExpiration
+     * @param flags
+     * @param authTime
+     * @param startTime
+     * @param endTime
+     * @param renewTill
+     * @param serverPrincipal
+     * @param caddr
      */
     public EncAsRepPart(EncryptionKey key, LastRequest lastReq, int nonce, KerberosTime keyExpiration,
         TicketFlags flags, KerberosTime authTime, KerberosTime startTime, KerberosTime endTime, KerberosTime renewTill,

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKdcRepPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKdcRepPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKdcRepPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKdcRepPart.java Sun May 20 17:50:01 2007
@@ -52,7 +52,7 @@
 
 
     /**
-     * Class constructors
+     * Creates a new instance of EncKdcRepPart.
      */
     public EncKdcRepPart()
     {
@@ -60,9 +60,25 @@
     }
 
 
-    public EncKdcRepPart(EncryptionKey key, LastRequest lastReq, int nonce, KerberosTime keyExpiration,
+    /**
+     * Creates a new instance of EncKdcRepPart.
+     *
+     * @param key
+     * @param lastReq
+     * @param nonce
+     * @param keyExpiration
+     * @param flags
+     * @param authtime
+     * @param starttime
+     * @param endtime
+     * @param renewTill
+     * @param serverPrincipal
+     * @param caddr
+     * @param componentType
+     */
+    public EncKdcRepPart( EncryptionKey key, LastRequest lastReq, int nonce, KerberosTime keyExpiration,
         TicketFlags flags, KerberosTime authtime, KerberosTime starttime, KerberosTime endtime, KerberosTime renewTill,
-        KerberosPrincipal serverPrincipal, HostAddresses caddr, MessageComponentType componentType)
+        KerberosPrincipal serverPrincipal, HostAddresses caddr, MessageComponentType componentType )
     {
         this.key = key;
         this.lastRequest = lastReq;
@@ -79,146 +95,264 @@
     }
 
 
-    // getters
+    /**
+     * Returns the auth {@link KerberosTime}.
+     *
+     * @return The auth {@link KerberosTime}.
+     */
     public KerberosTime getAuthTime()
     {
         return authTime;
     }
 
 
+    /**
+     * Returns the client {@link HostAddresses}.
+     *
+     * @return The client {@link HostAddresses}.
+     */
     public HostAddresses getClientAddresses()
     {
         return clientAddresses;
     }
 
 
+    /**
+     * Returns the end {@link KerberosTime}.
+     *
+     * @return The end {@link KerberosTime}.
+     */
     public KerberosTime getEndTime()
     {
         return endTime;
     }
 
 
+    /**
+     * Returns the {@link TicketFlags}.
+     *
+     * @return The {@link TicketFlags}.
+     */
     public TicketFlags getFlags()
     {
         return flags;
     }
 
 
+    /**
+     * Returns the {@link EncryptionKey}.
+     *
+     * @return The {@link EncryptionKey}.
+     */
     public EncryptionKey getKey()
     {
         return key;
     }
 
 
+    /**
+     * Returns the key expiration {@link KerberosTime}.
+     *
+     * @return The key expiration {@link KerberosTime}.
+     */
     public KerberosTime getKeyExpiration()
     {
         return keyExpiration;
     }
 
 
+    /**
+     * Returns the {@link LastRequest}.
+     *
+     * @return The {@link LastRequest}.
+     */
     public LastRequest getLastRequest()
     {
         return lastRequest;
     }
 
 
+    /**
+     * Returns the nonce.
+     *
+     * @return The nonce.
+     */
     public int getNonce()
     {
         return nonce;
     }
 
 
+    /**
+     * Returns the renew till {@link KerberosTime}.
+     *
+     * @return The renew till {@link KerberosTime}.
+     */
     public KerberosTime getRenewTill()
     {
         return renewTill;
     }
 
 
+    /**
+     * Returns the server {@link KerberosPrincipal}.
+     *
+     * @return The server {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getServerPrincipal()
     {
         return serverPrincipal;
     }
 
 
+    /**
+     * Returns the server realm.
+     *
+     * @return The server realm.
+     */
     public String getServerRealm()
     {
         return serverPrincipal.getRealm();
     }
 
 
+    /**
+     * Returns the start {@link KerberosTime}.
+     *
+     * @return The start {@link KerberosTime}.
+     */
     public KerberosTime getStartTime()
     {
         return startTime;
     }
 
 
+    /**
+     * Returns the {@link MessageComponentType}.
+     *
+     * @return The {@link MessageComponentType}.
+     */
     public MessageComponentType getComponentType()
     {
         return componentType;
     }
 
 
-    // setters
+    /**
+     * Sets the auth {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setAuthTime( KerberosTime time )
     {
         authTime = time;
     }
 
 
+    /**
+     * Sets the client {@link HostAddresses}.
+     *
+     * @param addresses
+     */
     public void setClientAddresses( HostAddresses addresses )
     {
         clientAddresses = addresses;
     }
 
 
+    /**
+     * Sets the end {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setEndTime( KerberosTime time )
     {
         endTime = time;
     }
 
 
+    /**
+     * Sets the {@link TicketFlags}.
+     *
+     * @param flags
+     */
     public void setFlags( TicketFlags flags )
     {
         this.flags = flags;
     }
 
 
+    /**
+     * Sets the {@link EncryptionKey}.
+     *
+     * @param key
+     */
     public void setKey( EncryptionKey key )
     {
         this.key = key;
     }
 
 
+    /**
+     * Sets the key expiration {@link KerberosTime}.
+     *
+     * @param expiration
+     */
     public void setKeyExpiration( KerberosTime expiration )
     {
         keyExpiration = expiration;
     }
 
 
+    /**
+     * Sets the {@link LastRequest}.
+     *
+     * @param request
+     */
     public void setLastRequest( LastRequest request )
     {
         lastRequest = request;
     }
 
 
+    /**
+     * Sets the nonce.
+     *
+     * @param nonce
+     */
     public void setNonce( int nonce )
     {
         this.nonce = nonce;
     }
 
 
+    /**
+     * Sets the renew till {@link KerberosTime}.
+     *
+     * @param till
+     */
     public void setRenewTill( KerberosTime till )
     {
         renewTill = till;
     }
 
 
+    /**
+     * Sets the server {@link KerberosPrincipal}.
+     *
+     * @param principal
+     */
     public void setServerPrincipal( KerberosPrincipal principal )
     {
         serverPrincipal = principal;
     }
 
 
+    /**
+     * Sets the start {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setStartTime( KerberosTime time )
     {
         startTime = time;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbCredPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbCredPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbCredPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbCredPart.java Sun May 20 17:50:01 2007
@@ -34,19 +34,26 @@
  */
 public class EncKrbCredPart
 {
-    public KrbCredInfo[] ticketInfo;
-    public Integer nonce; //optional
-    public KerberosTime timeStamp; //optional
-    public Integer usec; //optional
-    public HostAddress sAddress; //optional
-    public HostAddresses rAddress; //optional
+    private KrbCredInfo[] ticketInfo;
+    private Integer nonce; //optional
+    private KerberosTime timeStamp; //optional
+    private Integer usec; //optional
+    private HostAddress sAddress; //optional
+    private HostAddresses rAddress; //optional
 
 
     /**
-     * Class constructor
+     * Creates a new instance of EncKrbCredPart.
+     *
+     * @param ticketInfo
+     * @param timeStamp
+     * @param usec
+     * @param nonce
+     * @param sAddress
+     * @param rAddress
      */
-    public EncKrbCredPart(KrbCredInfo[] ticketInfo, KerberosTime timeStamp, Integer usec, Integer nonce,
-        HostAddress sAddress, HostAddresses rAddress)
+    public EncKrbCredPart( KrbCredInfo[] ticketInfo, KerberosTime timeStamp, Integer usec, Integer nonce,
+        HostAddress sAddress, HostAddresses rAddress )
     {
         this.ticketInfo = ticketInfo;
         this.nonce = nonce;
@@ -54,5 +61,71 @@
         this.usec = usec;
         this.sAddress = sAddress;
         this.rAddress = rAddress;
+    }
+
+
+    /**
+     * Returns the nonce.
+     * 
+     * @return The nonce.
+     */
+    public Integer getNonce()
+    {
+        return nonce;
+    }
+
+
+    /**
+     * Returns the "R" {@link HostAddresses}.
+     * 
+     * @return The "R" {@link HostAddresses}.
+     */
+    public HostAddresses getRAddress()
+    {
+        return rAddress;
+    }
+
+
+    /**
+     * Returns the "S" {@link HostAddresses}.
+     * 
+     * @return The "S" {@link HostAddresses}.
+     */
+    public HostAddress getSAddress()
+    {
+        return sAddress;
+    }
+
+
+    /**
+     * Returns the {@link KrbCredInfo}s.
+     * 
+     * @return The {@link KrbCredInfo}s.
+     */
+    public KrbCredInfo[] getTicketInfo()
+    {
+        return ticketInfo;
+    }
+
+
+    /**
+     * Returns the timestamp.
+     * 
+     * @return The timeStamp.
+     */
+    public KerberosTime getTimeStamp()
+    {
+        return timeStamp;
+    }
+
+
+    /**
+     * Returns the microseconds.
+     * 
+     * @return The microseconds.
+     */
+    public Integer getUsec()
+    {
+        return usec;
     }
 }

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPart.java Sun May 20 17:50:01 2007
@@ -43,8 +43,18 @@
     private HostAddress recipientAddress; //optional
 
 
-    public EncKrbPrivPart(byte[] userData, KerberosTime timestamp, Integer usec, Integer sequenceNumber,
-        HostAddress senderAddress, HostAddress recipientAddress)
+    /**
+     * Creates a new instance of EncKrbPrivPart.
+     *
+     * @param userData
+     * @param timestamp
+     * @param usec
+     * @param sequenceNumber
+     * @param senderAddress
+     * @param recipientAddress
+     */
+    public EncKrbPrivPart( byte[] userData, KerberosTime timestamp, Integer usec, Integer sequenceNumber,
+        HostAddress senderAddress, HostAddress recipientAddress )
     {
         super( MessageType.ENC_PRIV_PART );
 
@@ -57,36 +67,66 @@
     }
 
 
+    /**
+     * Returns the recipient {@link HostAddress}.
+     *
+     * @return The recipient {@link HostAddress}.
+     */
     public HostAddress getRecipientAddress()
     {
         return recipientAddress;
     }
 
 
+    /**
+     * Returns the sender {@link HostAddress}.
+     *
+     * @return The sender {@link HostAddress}.
+     */
     public HostAddress getSenderAddress()
     {
         return senderAddress;
     }
 
 
+    /**
+     * Returns the sequence number.
+     *
+     * @return The sequence number.
+     */
     public Integer getSequenceNumber()
     {
         return sequenceNumber;
     }
 
 
+    /**
+     * Returns the {@link KerberosTime} timestamp.
+     *
+     * @return The {@link KerberosTime} timestamp.
+     */
     public KerberosTime getTimestamp()
     {
         return timestamp;
     }
 
 
+    /**
+     * Returns the microsecond.
+     *
+     * @return The microsecond.
+     */
     public Integer getMicroSecond()
     {
         return usec;
     }
 
 
+    /**
+     * Returns the user data.
+     *
+     * @return The user data.
+     */
     public byte[] getUserData()
     {
         return userData;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPartModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPartModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPartModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncKrbPrivPartModifier.java Sun May 20 17:50:01 2007
@@ -38,42 +38,77 @@
     private HostAddress recipientAddress; //optional
 
 
+    /**
+     * Returns the {@link EncKrbPrivPart}.
+     *
+     * @return The {@link EncKrbPrivPart}.
+     */
     public EncKrbPrivPart getEncKrbPrivPart()
     {
         return new EncKrbPrivPart( userData, timestamp, usec, sequenceNumber, senderAddress, recipientAddress );
     }
 
 
+    /**
+     * Sets the recipient {@link HostAddress}.
+     *
+     * @param address
+     */
     public void setRecipientAddress( HostAddress address )
     {
         recipientAddress = address;
     }
 
 
+    /**
+     * Sets the sender {@link HostAddress}.
+     *
+     * @param address
+     */
     public void setSenderAddress( HostAddress address )
     {
         senderAddress = address;
     }
 
 
+    /**
+     * Sets the sequence number.
+     *
+     * @param number
+     */
     public void setSequenceNumber( Integer number )
     {
         sequenceNumber = number;
     }
 
 
+    /**
+     * Sets the {@link KerberosTime} timestamp.
+     *
+     * @param timestamp
+     */
     public void setTimestamp( KerberosTime timestamp )
     {
         this.timestamp = timestamp;
     }
 
 
+    /**
+     * Sets the microsecond.
+     *
+     * @param usec
+     */
     public void setMicroSecond( Integer usec )
     {
         this.usec = usec;
     }
 
 
+    /**
+     * Sets the user data.
+     *
+     * @param data
+     */
     public void setUserData( byte[] data )
     {
         userData = data;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTgsRepPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTgsRepPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTgsRepPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTgsRepPart.java Sun May 20 17:50:01 2007
@@ -38,13 +38,24 @@
 public class EncTgsRepPart extends EncKdcRepPart
 {
     /**
-     * Class constructor
+     * Creates a new instance of EncTgsRepPart.
+     *
+     * @param key
+     * @param lastReq
+     * @param nonce
+     * @param keyExpiration
+     * @param flags
+     * @param authtime
+     * @param starttime
+     * @param endtime
+     * @param renewTill
+     * @param serverPrincipal
+     * @param caddr
      */
     public EncTgsRepPart(EncryptionKey key, LastRequest lastReq, int nonce, KerberosTime keyExpiration,
         TicketFlags flags, KerberosTime authtime, KerberosTime starttime, KerberosTime endtime, KerberosTime renewTill,
         KerberosPrincipal serverPrincipal, HostAddresses caddr)
     {
-
         super( key, lastReq, nonce, keyExpiration, flags, authtime, starttime, endtime, renewTill, serverPrincipal,
             caddr, MessageComponentType.KRB_ENC_TGS_REP_PART );
     }

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPart.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPart.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPart.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPart.java Sun May 20 17:50:01 2007
@@ -51,11 +51,24 @@
     private AuthorizationData authorizationData; //optional
 
 
-    public EncTicketPart(TicketFlags flags, EncryptionKey key, KerberosPrincipal clientPrincipal,
+    /**
+     * Creates a new instance of EncTicketPart.
+     *
+     * @param flags
+     * @param key
+     * @param clientPrincipal
+     * @param transited
+     * @param authtime
+     * @param starttime
+     * @param endtime
+     * @param renewTill
+     * @param caddr
+     * @param authorizationData
+     */
+    public EncTicketPart( TicketFlags flags, EncryptionKey key, KerberosPrincipal clientPrincipal,
         TransitedEncoding transited, KerberosTime authtime, KerberosTime starttime, KerberosTime endtime,
-        KerberosTime renewTill, HostAddresses caddr, AuthorizationData authorizationData)
+        KerberosTime renewTill, HostAddresses caddr, AuthorizationData authorizationData )
     {
-
         this.flags = flags;
         this.sessionKey = key;
         this.clientPrincipal = clientPrincipal;
@@ -69,66 +82,121 @@
     }
 
 
+    /**
+     * Returns the {@link AuthorizationData}.
+     *
+     * @return The {@link AuthorizationData}.
+     */
     public AuthorizationData getAuthorizationData()
     {
         return authorizationData;
     }
 
 
+    /**
+     * Returns the auth {@link KerberosTime}
+     *
+     * @return The auth {@link KerberosTime}
+     */
     public KerberosTime getAuthTime()
     {
         return authtime;
     }
 
 
+    /**
+     * Returns the client {@link HostAddresses}.
+     *
+     * @return The client {@link HostAddresses}.
+     */
     public HostAddresses getClientAddresses()
     {
         return clientAddresses;
     }
 
 
+    /**
+     * Returns the client {@link KerberosPrincipal}.
+     *
+     * @return The client {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getClientPrincipal()
     {
         return clientPrincipal;
     }
 
 
+    /**
+     * Returns the client realm.
+     *
+     * @return The client realm.
+     */
     public String getClientRealm()
     {
         return clientPrincipal.getRealm();
     }
 
 
+    /**
+     * Returns the end {@link KerberosTime}
+     *
+     * @return The end {@link KerberosTime}
+     */
     public KerberosTime getEndTime()
     {
         return endTime;
     }
 
 
+    /**
+     * Returns the {@link TicketFlags}.
+     *
+     * @return The {@link TicketFlags}.
+     */
     public TicketFlags getFlags()
     {
         return flags;
     }
 
 
+    /**
+     * Returns the session {@link EncryptionKey}.
+     *
+     * @return The session {@link EncryptionKey}.
+     */
     public EncryptionKey getSessionKey()
     {
         return sessionKey;
     }
 
 
+    /**
+     * Returns the renew till {@link KerberosTime}
+     *
+     * @return The renew till {@link KerberosTime}
+     */
     public KerberosTime getRenewTill()
     {
         return renewTill;
     }
 
 
+    /**
+     * Returns the start {@link KerberosTime}
+     *
+     * @return The start {@link KerberosTime}
+     */
     public KerberosTime getStartTime()
     {
         return startTime;
     }
 
 
+    /**
+     * Returns the {@link TransitedEncoding}.
+     *
+     * @return The {@link TransitedEncoding}.
+     */
     public TransitedEncoding getTransitedEncoding()
     {
         return transitedEncoding;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPartModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPartModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPartModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/EncTicketPartModifier.java Sun May 20 17:50:01 2007
@@ -53,6 +53,11 @@
     private AuthorizationData authorizationData; //optional
 
 
+    /**
+     * Returns the {@link EncTicketPart}.
+     *
+     * @return The {@link EncTicketPart}.
+     */
     public EncTicketPart getEncTicketPart()
     {
         if ( clientPrincipal == null )
@@ -65,84 +70,154 @@
     }
 
 
+    /**
+     * Sets the client {@link PrincipalName}.
+     *
+     * @param name
+     */
     public void setClientName( PrincipalName name )
     {
         modifier.setPrincipalName( name );
     }
 
 
+    /**
+     * Sets the client realm.
+     *
+     * @param realm
+     */
     public void setClientRealm( String realm )
     {
         modifier.setRealm( realm );
     }
 
 
+    /**
+     * Sets the client {@link KerberosPrincipal}.
+     *
+     * @param principal
+     */
     public void setClientPrincipal( KerberosPrincipal principal )
     {
         clientPrincipal = principal;
     }
 
 
+    /**
+     * Sets the {@link AuthorizationData}.
+     *
+     * @param data
+     */
     public void setAuthorizationData( AuthorizationData data )
     {
         authorizationData = data;
     }
 
 
+    /**
+     * Sets the auth {@link KerberosTime}.
+     *
+     * @param authtime
+     */
     public void setAuthTime( KerberosTime authtime )
     {
         authTime = authtime;
     }
 
 
+    /**
+     * Sets the client {@link HostAddresses}.
+     *
+     * @param addresses
+     */
     public void setClientAddresses( HostAddresses addresses )
     {
         clientAddresses = addresses;
     }
 
 
+    /**
+     * Sets the end {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setEndTime( KerberosTime time )
     {
         endTime = time;
     }
 
 
+    /**
+     * Sets the {@link TicketFlags}.
+     *
+     * @param flags
+     */
     public void setFlags( TicketFlags flags )
     {
         this.flags = flags;
     }
 
 
+    /**
+     * Sets the flag at the given index.
+     *
+     * @param flag
+     */
     public void setFlag( int flag )
     {
         flags.set( flag );
     }
 
 
+    /**
+     * Clears the flag at the given index.
+     *
+     * @param flag
+     */
     public void clearFlag( int flag )
     {
         flags.clear( flag );
     }
 
 
+    /**
+     * Sets the renew till {@link KerberosTime}.
+     *
+     * @param till
+     */
     public void setRenewTill( KerberosTime till )
     {
         renewTill = till;
     }
 
 
+    /**
+     * Sets the sesson {@link EncryptionKey}.
+     *
+     * @param key
+     */
     public void setSessionKey( EncryptionKey key )
     {
         sessionKey = key;
     }
 
 
+    /**
+     * Sets the start {@link KerberosTime}.
+     *
+     * @param time
+     */
     public void setStartTime( KerberosTime time )
     {
         startTime = time;
     }
 
 
+    /**
+     * Sets the {@link TransitedEncoding}.
+     *
+     * @param encoding
+     */
     public void setTransitedEncoding( TransitedEncoding encoding )
     {
         transitedEncoding = encoding;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/MessageComponentType.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/MessageComponentType.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/MessageComponentType.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/MessageComponentType.java Sun May 20 17:50:01 2007
@@ -34,47 +34,94 @@
 public class MessageComponentType implements Comparable
 {
     /**
-     * Enumeration elements are constructed once upon class loading.
-     * Order of appearance here determines the order of compareTo.
+     * Constant for the "null" message component type.
      */
     public static final MessageComponentType NULL = new MessageComponentType( 0, "null" );
+
+    /**
+     * Constant for the "ticket" message component type.
+     */
     public static final MessageComponentType KRB_TKT = new MessageComponentType( 1, "ticket" );
+
+    /**
+     * Constant for the "authenticator" message component type.
+     */
     public static final MessageComponentType KRB_AUTHENTICATOR = new MessageComponentType( 2, "authenticator" );
+
+    /**
+     * Constant for the "encrypted ticket part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_TKT_PART = new MessageComponentType( 3, "encrypted ticket part" );
+
+    /**
+     * Constant for the "encrypted initial authentication part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_AS_REP_PART = new MessageComponentType( 25,
         "encrypted initial authentication part" );
+
+    /**
+     * Constant for the "encrypted TGS request part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_TGS_REP_PART = new MessageComponentType( 26,
         "encrypted TGS request part" );
+
+    /**
+     * Constant for the "encrypted application request part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_AP_REP_PART = new MessageComponentType( 27,
         "encrypted application request part" );
+
+    /**
+     * Constant for the "encrypted application message part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_KRB_PRIV_PART = new MessageComponentType( 28,
         "encrypted application message part" );
+
+    /**
+     * Constant for the "encrypted credentials forward part" message component type.
+     */
     public static final MessageComponentType KRB_ENC_KRB_CRED_PART = new MessageComponentType( 29,
         "encrypted credentials forward part" );
 
     /**
-     * These two lines are all that's necessary to export a List of VALUES.
+     * Array for building a List of VALUES.
      */
     private static final MessageComponentType[] values =
         { NULL, KRB_TKT, KRB_AUTHENTICATOR, KRB_ENC_TKT_PART, KRB_ENC_AS_REP_PART, KRB_ENC_TGS_REP_PART,
             KRB_ENC_AP_REP_PART, KRB_ENC_KRB_PRIV_PART, KRB_ENC_KRB_CRED_PART };
 
+    /**
+     * A List of all the message component type constants.
+     */
     public static final List VALUES = Collections.unmodifiableList( Arrays.asList( values ) );
 
+    /**
+     * The name of the message component type.
+     */
     private final String name;
+
+    /**
+     * The value/code for the message component type.
+     */
     private final int ordinal;
 
 
     /**
      * Private constructor prevents construction outside of this class.
      */
-    private MessageComponentType(int ordinal, String name)
+    private MessageComponentType( int ordinal, String name )
     {
         this.ordinal = ordinal;
         this.name = name;
     }
 
 
+    /**
+     * Returns the message component type when specified by its ordinal.
+     *
+     * @param type
+     * @return The message component type.
+     */
     public static MessageComponentType getTypeByOrdinal( int type )
     {
         for ( int ii = 0; ii < values.length; ii++ )
@@ -89,20 +136,25 @@
     }
 
 
+    /**
+     * Returns the number associated with this message component type.
+     *
+     * @return The message component type ordinal.
+     */
     public int getOrdinal()
     {
         return ordinal;
     }
 
 
-    public String toString()
+    public int compareTo( Object that )
     {
-        return name + " (" + ordinal + ")";
+        return ordinal - ( ( MessageComponentType ) that ).ordinal;
     }
 
 
-    public int compareTo( Object that )
+    public String toString()
     {
-        return ordinal - ( ( MessageComponentType ) that ).ordinal;
+        return name + " (" + ordinal + ")";
     }
 }

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Ticket.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Ticket.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Ticket.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/Ticket.java Sun May 20 17:50:01 2007
@@ -39,6 +39,9 @@
  */
 public class Ticket
 {
+    /**
+     * Constant for the {@link Ticket} version number (5).
+     */
     public static final int TICKET_VNO = 5;
 
     private int versionNumber;
@@ -47,13 +50,26 @@
     private EncTicketPart encTicketPart;
 
 
-    public Ticket(KerberosPrincipal serverPrincipal, EncryptedData encPart)
+    /**
+     * Creates a new instance of Ticket.
+     *
+     * @param serverPrincipal
+     * @param encPart
+     */
+    public Ticket( KerberosPrincipal serverPrincipal, EncryptedData encPart )
     {
         this( TICKET_VNO, serverPrincipal, encPart );
     }
 
 
-    public Ticket(int versionNumber, KerberosPrincipal serverPrincipal, EncryptedData encPart)
+    /**
+     * Creates a new instance of Ticket.
+     *
+     * @param versionNumber
+     * @param serverPrincipal
+     * @param encPart
+     */
+    public Ticket( int versionNumber, KerberosPrincipal serverPrincipal, EncryptedData encPart )
     {
         this.versionNumber = versionNumber;
         this.serverPrincipal = serverPrincipal;
@@ -61,111 +77,199 @@
     }
 
 
+    /**
+     * Sets the {@link EncTicketPart}.
+     *
+     * @param decryptedPart
+     */
     public void setEncTicketPart( EncTicketPart decryptedPart )
     {
         encTicketPart = decryptedPart;
     }
 
 
-    // getters
+    /**
+     * Returns the version number.
+     *
+     * @return The version number.
+     */
     public int getVersionNumber()
     {
         return versionNumber;
     }
 
 
+    /**
+     * Returns the server {@link KerberosPrincipal}.
+     *
+     * @return The server {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getServerPrincipal()
     {
         return serverPrincipal;
     }
 
 
+    /**
+     * Returns the server realm.
+     *
+     * @return The server realm.
+     */
     public String getRealm()
     {
         return serverPrincipal.getRealm();
     }
 
 
+    /**
+     * Returns the {@link EncryptedData}.
+     *
+     * @return The {@link EncryptedData}.
+     */
     public EncryptedData getEncPart()
     {
         return encPart;
     }
 
 
+    /**
+     * Returns the {@link EncTicketPart}.
+     *
+     * @return The {@link EncTicketPart}.
+     */
     public EncTicketPart getEncTicketPart()
     {
         return encTicketPart;
     }
 
 
-    // EncTicketPart delegate getters
+    /**
+     * Returns the {@link AuthorizationData}.
+     *
+     * @return The {@link AuthorizationData}.
+     */
     public AuthorizationData getAuthorizationData()
     {
         return encTicketPart.getAuthorizationData();
     }
 
 
+    /**
+     * Returns the auth {@link KerberosTime}.
+     *
+     * @return The auth {@link KerberosTime}.
+     */
     public KerberosTime getAuthTime()
     {
         return encTicketPart.getAuthTime();
     }
 
 
+    /**
+     * Returns the client {@link HostAddresses}.
+     *
+     * @return The client {@link HostAddresses}.
+     */
     public HostAddresses getClientAddresses()
     {
         return encTicketPart.getClientAddresses();
     }
 
 
+    /**
+     * Returns the client {@link KerberosPrincipal}.
+     *
+     * @return The client {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getClientPrincipal()
     {
         return encTicketPart.getClientPrincipal();
     }
 
 
+    /**
+     * Returns the client realm.
+     *
+     * @return The client realm.
+     */
     public String getClientRealm()
     {
         return encTicketPart.getClientPrincipal().getRealm();
     }
 
 
+    /**
+     * Returns the end {@link KerberosTime}.
+     *
+     * @return The end {@link KerberosTime}.
+     */
     public KerberosTime getEndTime()
     {
         return encTicketPart.getEndTime();
     }
 
 
+    /**
+     * Returns the {@link TicketFlags}.
+     *
+     * @return The {@link TicketFlags}.
+     */
     public TicketFlags getFlags()
     {
         return encTicketPart.getFlags();
     }
 
 
+    /**
+     * Returns the renew till {@link KerberosTime}.
+     *
+     * @return The renew till {@link KerberosTime}.
+     */
     public KerberosTime getRenewTill()
     {
         return encTicketPart.getRenewTill();
     }
 
 
+    /**
+     * Returns the session {@link EncryptionKey}.
+     *
+     * @return The session {@link EncryptionKey}.
+     */
     public EncryptionKey getSessionKey()
     {
         return encTicketPart.getSessionKey();
     }
 
 
+    /**
+     * Returns the start {@link KerberosTime}.
+     *
+     * @return The start {@link KerberosTime}.
+     */
     public KerberosTime getStartTime()
     {
         return encTicketPart.getStartTime();
     }
 
 
+    /**
+     * Returns the {@link TransitedEncoding}.
+     *
+     * @return The {@link TransitedEncoding}.
+     */
     public TransitedEncoding getTransitedEncoding()
     {
         return encTicketPart.getTransitedEncoding();
     }
 
 
-    // EncTicketPart TicketFlag delegates
+    /**
+     * Returns the flag at the given index.
+     *
+     * @param flag
+     * @return true if the flag at the given index is set.
+     */
     public boolean getFlag( int flag )
     {
         return encTicketPart.getFlags().get( flag );

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/TicketModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/TicketModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/TicketModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/components/TicketModifier.java Sun May 20 17:50:01 2007
@@ -38,6 +38,11 @@
     private EncryptedData encPart;
 
 
+    /**
+     * Returns the {@link Ticket}.
+     *
+     * @return The {@link Ticket}.
+     */
     public Ticket getTicket()
     {
         KerberosPrincipal serverPrincipal = serverModifier.getKerberosPrincipal();
@@ -45,24 +50,44 @@
     }
 
 
+    /**
+     * Sets the {@link Ticket} version number.
+     *
+     * @param versionNumber
+     */
     public void setTicketVersionNumber( int versionNumber )
     {
         ticketVersionNumber = versionNumber;
     }
 
 
+    /**
+     * Sets the {@link EncryptedData}.
+     *
+     * @param part
+     */
     public void setEncPart( EncryptedData part )
     {
         encPart = part;
     }
 
 
+    /**
+     * Sets the server {@link PrincipalName}.
+     *
+     * @param name
+     */
     public void setServerName( PrincipalName name )
     {
         serverModifier.setPrincipalName( name );
     }
 
 
+    /**
+     * Sets the server realm.
+     *
+     * @param realm
+     */
     public void setServerRealm( String realm )
     {
         serverModifier.setRealm( realm );

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Checksum.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Checksum.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Checksum.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Checksum.java Sun May 20 17:50:01 2007
@@ -35,7 +35,13 @@
     private byte[] checksum;
 
 
-    public Checksum(ChecksumType checksumType, byte[] checksum)
+    /**
+     * Creates a new instance of Checksum.
+     *
+     * @param checksumType
+     * @param checksum
+     */
+    public Checksum( ChecksumType checksumType, byte[] checksum )
     {
         this.checksumType = checksumType;
         this.checksum = checksum;
@@ -60,12 +66,22 @@
     }
 
 
+    /**
+     * Returns the checksum value.
+     *
+     * @return The checksum value.
+     */
     public byte[] getChecksumValue()
     {
         return checksum;
     }
 
 
+    /**
+     * Returns the {@link ChecksumType}.
+     *
+     * @return The {@link ChecksumType}.
+     */
     public ChecksumType getChecksumType()
     {
         return checksumType;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedData.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedData.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedData.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedData.java Sun May 20 17:50:01 2007
@@ -34,7 +34,14 @@
     private byte[] cipherText;
 
 
-    public EncryptedData(EncryptionType type, int version, byte[] cipherText)
+    /**
+     * Creates a new instance of EncryptedData.
+     *
+     * @param type
+     * @param version
+     * @param cipherText
+     */
+    public EncryptedData( EncryptionType type, int version, byte[] cipherText )
     {
         encryptionType = type;
         keyVersion = version;
@@ -42,18 +49,33 @@
     }
 
 
+    /**
+     * Returns the {@link EncryptionType}.
+     *
+     * @return The {@link EncryptionType}.
+     */
     public EncryptionType getEncryptionType()
     {
         return encryptionType;
     }
 
 
+    /**
+     * Returns the key version.
+     *
+     * @return The key version.
+     */
     public int getKeyVersion()
     {
         return keyVersion;
     }
 
 
+    /**
+     * Returns the raw cipher text.
+     *
+     * @return The raw cipher text.
+     */
     public byte[] getCipherText()
     {
         return cipherText;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedDataModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedDataModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedDataModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptedDataModifier.java Sun May 20 17:50:01 2007
@@ -34,24 +34,44 @@
     private byte[] cipherText;
 
 
+    /**
+     * Returns the {@link EncryptedData}.
+     *
+     * @return The {@link EncryptedData}.
+     */
     public EncryptedData getEncryptedData()
     {
         return new EncryptedData( encryptionType, keyVersion, cipherText );
     }
 
 
+    /**
+     * Sets the raw cipher text bytes.
+     *
+     * @param cipherText
+     */
     public void setCipherText( byte[] cipherText )
     {
         this.cipherText = cipherText;
     }
 
 
+    /**
+     * Sets the {@link EncryptionType}.
+     *
+     * @param type
+     */
     public void setEncryptionType( EncryptionType type )
     {
         encryptionType = type;
     }
 
 
+    /**
+     * Sets the key version.
+     *
+     * @param version
+     */
     public void setKeyVersion( int version )
     {
         keyVersion = version;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptionTypeInfoEntry.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptionTypeInfoEntry.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptionTypeInfoEntry.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/EncryptionTypeInfoEntry.java Sun May 20 17:50:01 2007
@@ -33,19 +33,35 @@
     private byte[] salt;
 
 
-    public EncryptionTypeInfoEntry(EncryptionType encryptionType, byte[] salt)
+    /**
+     * Creates a new instance of EncryptionTypeInfoEntry.
+     *
+     * @param encryptionType
+     * @param salt
+     */
+    public EncryptionTypeInfoEntry( EncryptionType encryptionType, byte[] salt )
     {
         this.encryptionType = encryptionType;
         this.salt = salt;
     }
 
 
+    /**
+     * Returns the salt.
+     *
+     * @return The salt.
+     */
     public byte[] getSalt()
     {
         return salt;
     }
 
 
+    /**
+     * Returns the {@link EncryptionType}.
+     *
+     * @return The {@link EncryptionType}.
+     */
     public EncryptionType getEncryptionType()
     {
         return encryptionType;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/HostAddresses.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/HostAddresses.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/HostAddresses.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/HostAddresses.java Sun May 20 17:50:01 2007
@@ -30,14 +30,22 @@
 
 
     /**
-     * Class constructors
+     * Creates a new instance of HostAddresses.
+     *
+     * @param addresses
      */
-    public HostAddresses(HostAddress[] addresses)
+    public HostAddresses( HostAddress[] addresses )
     {
         this.addresses = addresses;
     }
 
 
+    /**
+     * Returns true if this {@link HostAddresses} contains a specified {@link HostAddress}.
+     *
+     * @param address
+     * @return true if this {@link HostAddresses} contains a specified {@link HostAddress}.
+     */
     public boolean contains( HostAddress address )
     {
         if ( addresses != null )
@@ -55,6 +63,12 @@
     }
 
 
+    /**
+     * Returns true if two {@link HostAddresses} are equal.
+     *
+     * @param that
+     * @return true if two {@link HostAddresses} are equal.
+     */
     public boolean equals( HostAddresses that )
     {
         if ( ( this.addresses == null && that.addresses != null )
@@ -83,6 +97,11 @@
     }
 
 
+    /**
+     * Returns the contained {@link HostAddress}s as an array.
+     *
+     * @return An array of {@link HostAddress}s.
+     */
     public HostAddress[] getAddresses()
     {
         return addresses;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KdcOptions.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KdcOptions.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KdcOptions.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KdcOptions.java Sun May 20 17:50:01 2007
@@ -26,51 +26,108 @@
  */
 public class KdcOptions extends Options
 {
-    // KDC option - reserved
+    /**
+     * KDC option - reserved.
+     */
     public static final int RESERVED = 0;
-    // KDC option - forwardable
+    /**
+     * KDC option - forwardable.
+     */
     public static final int FORWARDABLE = 1;
-    // KDC option - forwarded
+
+    /**
+     * KDC option - forwarded.
+     */
     public static final int FORWARDED = 2;
-    // KDC option - proxiable
+
+    /**
+     * KDC option - proxiable.
+     */
     public static final int PROXIABLE = 3;
-    // KDC option - proxy
+
+    /**
+     * KDC option - proxy.
+     */
     public static final int PROXY = 4;
-    // KDC option - allow postdate
+
+    /**
+     * KDC option - allow postdate.
+     */
     public static final int ALLOW_POSTDATE = 5;
-    // KDC option - postdated
+
+    /**
+     * KDC option - postdated.
+     */
     public static final int POSTDATED = 6;
-    // KDC option - unused7
+
+    /**
+     * KDC option - unused7.
+     */
     public static final int UNUSED7 = 7;
-    // KDC option - renewable
+
+    /**
+     * KDC option - renewable.
+     */
     public static final int RENEWABLE = 8;
-    // KDC option - unused9
+
+    /**
+     * KDC option - unused9.
+     */
     public static final int UNUSED9 = 9;
-    // KDC option - unused10
+
+    /**
+     * KDC option - unused10.
+     */
     public static final int UNUSED10 = 10;
-    // KDC option - unused11
+
+    /**
+     * KDC option - unused11.
+     */
     public static final int UNUSED11 = 11;
-    // KDC option - unused12
+
+    /**
+     * KDC option - unused12.
+     */
     public static final int UNUSED12 = 12;
-    // KDC option - unused13
+
+    /**
+     * KDC option - unused13.
+     */
     public static final int UNUSED13 = 13;
-    // KDC option - disable transisted checked
+
+    /**
+     * KDC option - disable transisted checked.
+     */
     public static final int DISABLE_TRANSISTED_CHECKED = 26;
-    // KDC option - renewable is ok
+
+    /**
+     * KDC option - renewable is ok.
+     */
     public static final int RENEWABLE_OK = 27;
-    // KDC option - encrypted key in skey
+
+    /**
+     * KDC option - encrypted key in skey.
+     */
     public static final int ENC_TKT_IN_SKEY = 28;
-    // KDC option - renew
+
+    /**
+     * KDC option - renew.
+     */
     public static final int RENEW = 30;
-    // KDC option - validate
+
+    /**
+     * KDC option - validate.
+     */
     public static final int VALIDATE = 31;
 
-    // KDC option - maximum value
+    /**
+     * KDC option - maximum value.
+     */
     public static final int MAX_VALUE = 32;
 
 
     /**
-     * Class constructors
+     * Creates a new instance of KdcOptions.
      */
     public KdcOptions()
     {
@@ -78,7 +135,12 @@
     }
 
 
-    public KdcOptions(byte[] bytes)
+    /**
+     * Creates a new instance of KdcOptions.
+     *
+     * @param bytes
+     */
+    public KdcOptions( byte[] bytes )
     {
         super( MAX_VALUE );
         setBytes( bytes );
@@ -86,7 +148,7 @@
 
 
     /**
-     * Converts the object to a printable string
+     * Converts the object to a printable string.
      */
     public String toString()
     {

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosPrincipalModifier.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosPrincipalModifier.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosPrincipalModifier.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosPrincipalModifier.java Sun May 20 17:50:01 2007
@@ -35,6 +35,11 @@
     String realm;
 
 
+    /**
+     * Returns the {@link KerberosPrincipal}.
+     *
+     * @return The {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getKerberosPrincipal()
     {
         if ( nameComponent != null )
@@ -55,12 +60,22 @@
     }
 
 
+    /**
+     * Sets the {@link PrincipalName}.
+     *
+     * @param principalName
+     */
     public void setPrincipalName( PrincipalName principalName )
     {
         nameComponent = principalName;
     }
 
 
+    /**
+     * Sets the realm.
+     *
+     * @param realm
+     */
     public void setRealm( String realm )
     {
         this.realm = realm;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosTime.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosTime.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosTime.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KerberosTime.java Sun May 20 17:50:01 2007
@@ -23,7 +23,6 @@
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.Date;
-import java.util.List;
 import java.util.TimeZone;
 
 
@@ -35,6 +34,9 @@
  */
 public class KerberosTime implements Comparable<KerberosTime>
 {
+    /**
+     * Constant for the {@link KerberosTime} "infinity."
+     */
     public static final KerberosTime INFINITY = new KerberosTime( Long.MAX_VALUE );
 
     private static final TimeZone UTC_TIME_ZONE = TimeZone.getTimeZone( "UTC" );
@@ -48,24 +50,44 @@
     private long kerberosTime;
 
 
+    /**
+     * Creates a new instance of KerberosTime.
+     */
     public KerberosTime()
     {
         kerberosTime = System.currentTimeMillis();
     }
 
 
-    public KerberosTime(long time)
+    /**
+     * Creates a new instance of KerberosTime.
+     *
+     * @param time
+     */
+    public KerberosTime( long time )
     {
         kerberosTime = time;
     }
 
 
-    public KerberosTime(Date time)
+    /**
+     * Creates a new instance of KerberosTime.
+     *
+     * @param time
+     */
+    public KerberosTime( Date time )
     {
         kerberosTime = time.getTime();
     }
 
-    
+
+    /**
+     * Returns the {@link KerberosTime} for a given zulu time.
+     *
+     * @param zuluTime
+     * @return The {@link KerberosTime}.
+     * @throws ParseException
+     */
     public static KerberosTime getTime( String zuluTime ) throws ParseException
     {
         Date date = null;
@@ -76,24 +98,19 @@
         return new KerberosTime( date );
     }
 
-    
-    public int compareTo( KerberosTime o )
+
+    public int compareTo( KerberosTime that )
     {
         final int BEFORE = -1;
         final int EQUAL = 0;
         final int AFTER = 1;
 
         // this optimization is usually worthwhile, and can always be added
-        if ( this == o )
+        if ( this == that )
         {
             return EQUAL;
         }
 
-        // Performing explicit checks for nullity and type are made redundant by
-        // the following cast, which will throw NullPointerException and
-        // ClassCastException in these respective cases.
-        final KerberosTime that = ( KerberosTime ) o;
-
         // primitive numbers follow this form
         if ( this.kerberosTime < that.kerberosTime )
         {
@@ -109,42 +126,81 @@
     }
 
 
+    /**
+     * Returns the {@link KerberosTime} as a long.
+     *
+     * @return The {@link KerberosTime} as a long.
+     */
     public long getTime()
     {
         return kerberosTime;
     }
 
 
+    /**
+     * Returns the {@link KerberosTime} as a {@link Date}.
+     *
+     * @return The {@link KerberosTime} as a {@link Date}.
+     */
     public Date toDate()
     {
         return new Date( kerberosTime );
     }
 
 
+    /**
+     * Returns whether this {@link KerberosTime} is within the given clockskew.
+     *
+     * @param clockSkew
+     * @return true if this {@link KerberosTime} is within the given clockskew.
+     */
     public boolean isInClockSkew( long clockSkew )
     {
         return Math.abs( kerberosTime - System.currentTimeMillis() ) < clockSkew;
     }
 
 
+    /**
+     * Returns whether this {@link KerberosTime} is greater than a given {@link KerberosTime}.
+     *
+     * @param time
+     * @return true if this {@link KerberosTime} is greater than a given {@link KerberosTime}. 
+     */
     public boolean greaterThan( KerberosTime time )
     {
         return kerberosTime > time.kerberosTime;
     }
 
 
+    /**
+     * Returns whether this {@link KerberosTime} is less than a given {@link KerberosTime}.
+     *
+     * @param time
+     * @return true if this {@link KerberosTime} is less than a given {@link KerberosTime}. 
+     */
     public boolean lessThan( KerberosTime time )
     {
         return kerberosTime < time.kerberosTime;
     }
 
 
+    /**
+     * Returns whether this {@link KerberosTime} is equal to another {@link KerberosTime}.
+     *
+     * @param time
+     * @return true if the two {@link KerberosTime}s are equal.
+     */
     public boolean equals( KerberosTime time )
     {
         return kerberosTime == time.kerberosTime;
     }
 
 
+    /**
+     * Returns whether this {@link KerberosTime} is zero.
+     *
+     * @return true if this {@link KerberosTime} is zero.
+     */
     public boolean isZero()
     {
         return kerberosTime == 0;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KrbCredInfo.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KrbCredInfo.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KrbCredInfo.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/KrbCredInfo.java Sun May 20 17:50:01 2007
@@ -42,9 +42,22 @@
     private HostAddresses clientAddresses; //optional
 
 
-    public KrbCredInfo(EncryptionKey key, KerberosPrincipal clientPrincipal, TicketFlags flags, KerberosTime authTime,
+    /**
+     * Creates a new instance of KrbCredInfo.
+     *
+     * @param key
+     * @param clientPrincipal
+     * @param flags
+     * @param authTime
+     * @param startTime
+     * @param endTime
+     * @param renewTill
+     * @param serverPrincipal
+     * @param clientAddresses
+     */
+    public KrbCredInfo( EncryptionKey key, KerberosPrincipal clientPrincipal, TicketFlags flags, KerberosTime authTime,
         KerberosTime startTime, KerberosTime endTime, KerberosTime renewTill, KerberosPrincipal serverPrincipal,
-        HostAddresses clientAddresses)
+        HostAddresses clientAddresses )
     {
         this.key = key;
         this.clientPrincipal = clientPrincipal;
@@ -58,54 +71,99 @@
     }
 
 
+    /**
+     * Returns the auth {@link KerberosTime}.
+     *
+     * @return The auth {@link KerberosTime}.
+     */
     public KerberosTime getAuthTime()
     {
         return authTime;
     }
 
 
+    /**
+     * Returns the client {@link HostAddresses}.
+     *
+     * @return The client {@link HostAddresses}.
+     */
     public HostAddresses getClientAddresses()
     {
         return clientAddresses;
     }
 
 
+    /**
+     * Returns the end {@link KerberosTime}.
+     *
+     * @return The end {@link KerberosTime}.
+     */
     public KerberosTime getEndTime()
     {
         return endTime;
     }
 
 
+    /**
+     * Returns the {@link TicketFlags}.
+     *
+     * @return The {@link TicketFlags}.
+     */
     public TicketFlags getFlags()
     {
         return flags;
     }
 
 
+    /**
+     * Returns the {@link EncryptionKey}.
+     *
+     * @return The {@link EncryptionKey}.
+     */
     public EncryptionKey getKey()
     {
         return key;
     }
 
 
+    /**
+     * Returns the client {@link KerberosPrincipal}.
+     *
+     * @return The client {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getClientPrincipal()
     {
         return clientPrincipal;
     }
 
 
+    /**
+     * Returns the renew till {@link KerberosTime}.
+     *
+     * @return The renew till {@link KerberosTime}.
+     */
     public KerberosTime getRenewTill()
     {
         return renewTill;
     }
 
 
+    /**
+     * Returns the server {@link KerberosPrincipal}.
+     *
+     * @return The server {@link KerberosPrincipal}.
+     */
     public KerberosPrincipal getServerPrincipal()
     {
         return serverPrincipal;
     }
 
 
+    /**
+     * Returns the start {@link KerberosTime}.
+     *
+     * @return The start {@link KerberosTime}.
+     */
     public KerberosTime getStartTime()
     {
         return startTime;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequest.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequest.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequest.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequest.java Sun May 20 17:50:01 2007
@@ -29,18 +29,31 @@
     private LastRequestEntry[] entries = new LastRequestEntry[1];
 
 
+    /**
+     * Creates a new instance of LastRequest.
+     */
     public LastRequest()
     {
         entries[0] = new LastRequestEntry( LastRequestType.NONE, new KerberosTime() );
     }
 
 
-    public LastRequest(LastRequestEntry[] entries)
+    /**
+     * Creates a new instance of LastRequest.
+     *
+     * @param entries
+     */
+    public LastRequest( LastRequestEntry[] entries )
     {
         this.entries = entries;
     }
 
 
+    /**
+     * Returns an array of {@link LastRequestEntry}s.
+     *
+     * @return The array of {@link LastRequestEntry}s.
+     */
     public LastRequestEntry[] getEntries()
     {
         return entries;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequestEntry.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequestEntry.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequestEntry.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/LastRequestEntry.java Sun May 20 17:50:01 2007
@@ -30,19 +30,35 @@
     private KerberosTime lastRequestValue;
 
 
-    public LastRequestEntry(LastRequestType type, KerberosTime value)
+    /**
+     * Creates a new instance of LastRequestEntry.
+     *
+     * @param type
+     * @param value
+     */
+    public LastRequestEntry( LastRequestType type, KerberosTime value )
     {
         lastRequestType = type;
         lastRequestValue = value;
     }
 
 
+    /**
+     * Returns the {@link LastRequestType}.
+     *
+     * @return The {@link LastRequestType}.
+     */
     public LastRequestType getLastRequestType()
     {
         return lastRequestType;
     }
 
 
+    /**
+     * Returns the {@link KerberosTime} of the last request.
+     *
+     * @return The {@link KerberosTime} of the last request.
+     */
     public KerberosTime getLastRequestValue()
     {
         return lastRequestValue;

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Options.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Options.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Options.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/Options.java Sun May 20 17:50:01 2007
@@ -33,41 +33,66 @@
     private int maxSize;
 
 
-    protected Options(int maxSize)
+    protected Options( int maxSize )
     {
         this.maxSize = maxSize;
         options = new BitSet( maxSize );
     }
 
 
+    /**
+     * Returns whether the option at a given index matches the option in this {@link Options}.
+     *
+     * @param options
+     * @param option
+     * @return true if two options are the same.
+     */
     public boolean match( Options options, int option )
     {
         return options.get( option ) == this.get( option );
     }
 
 
+    /**
+     * Returns the value of the option at the given index.
+     *
+     * @param index
+     * @return true if the option at the given index is set.
+     */
     public boolean get( int index )
     {
         return options.get( index );
     }
 
 
+    /**
+     * Sets the option at a given index.
+     *
+     * @param index
+     */
     public void set( int index )
     {
         options.set( index );
     }
 
 
+    /**
+     * Clears (sets false) the option at a given index.
+     *
+     * @param index
+     */
     public void clear( int index )
     {
         options.clear( index );
     }
 
 
-    /*
+    /**
      * Byte-reversing methods are an anomaly of the BouncyCastle
      * DERBitString endianness.  Thes methods can be removed if the
      * Apache Directory Snickers codecs operate differently.
+     * 
+     * @return The raw {@link Options} bytes.
      */
     public byte[] getBytes()
     {

Modified: directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationData.java
URL: http://svn.apache.org/viewvc/directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationData.java?view=diff&rev=539987&r1=539986&r2=539987
==============================================================================
--- directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationData.java (original)
+++ directory/apacheds/branches/kerberos-encryption-types/kerberos-shared/src/main/java/org/apache/directory/server/kerberos/shared/messages/value/PreAuthenticationData.java Sun May 20 17:50:01 2007
@@ -30,19 +30,35 @@
     private byte[] dataValue;
 
 
-    public PreAuthenticationData(PreAuthenticationDataType dataType, byte[] dataValue)
+    /**
+     * Creates a new instance of PreAuthenticationData.
+     *
+     * @param dataType
+     * @param dataValue
+     */
+    public PreAuthenticationData( PreAuthenticationDataType dataType, byte[] dataValue )
     {
         this.dataType = dataType;
         this.dataValue = dataValue;
     }
 
 
+    /**
+     * Returns the {@link PreAuthenticationDataType}.
+     *
+     * @return The {@link PreAuthenticationDataType}.
+     */
     public PreAuthenticationDataType getDataType()
     {
         return dataType;
     }
 
 
+    /**
+     * Returns the raw bytes of the {@link PreAuthenticationData}.
+     *
+     * @return The raw bytes of the {@link PreAuthenticationData}.
+     */
     public byte[] getDataValue()
     {
         return dataValue;