Author: norman
Date: Thu Jul 27 08:10:07 2006
New Revision: 426098

URL: http://svn.apache.org/viewvc?rev=426098&view=rev
Log:
Code cleanup
Add javadocs

Modified:
    
james/server/trunk/src/java/org/apache/james/transport/matchers/AbstractQuotaMatcher.java
    james/server/trunk/src/java/org/apache/james/transport/matchers/All.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/AttachmentFileNameIs.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/CommandForListserv.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/CompareNumericHeaderValue.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/FetchedFrom.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/FileRegexMatcher.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/GenericRegexMatcher.java
    
james/server/trunk/src/java/org/apache/james/transport/matchers/IsInWhiteList.java

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/AbstractQuotaMatcher.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/AbstractQuotaMatcher.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/AbstractQuotaMatcher.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/AbstractQuotaMatcher.java
 Thu Jul 27 08:10:07 2006
@@ -72,7 +72,6 @@
      * @return true if over quota
      */
     protected boolean isOverQuota(MailAddress address, Mail mail) {
-        String user = address.getUser();
         try {
             boolean over = getQuota(address, mail) < getUsed(address, mail);
             if (over) log(address + " is over quota.");

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/All.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/All.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- james/server/trunk/src/java/org/apache/james/transport/matchers/All.java 
(original)
+++ james/server/trunk/src/java/org/apache/james/transport/matchers/All.java 
Thu Jul 27 08:10:07 2006
@@ -16,9 +16,6 @@
  * specific language governing permissions and limitations      *
  * under the License.                                           *
  ****************************************************************/
-
-
-
 package org.apache.james.transport.matchers;
 
 import org.apache.mailet.GenericMatcher;
@@ -30,6 +27,10 @@
  * @version 1.0.0, 24/04/1999
  */
 public class All extends GenericMatcher {
+    
+    /**
+     * @see org.apache.mailet.GenericMatcher#match(Mail)
+     */
     public Collection match(Mail mail) {
         return mail.getRecipients();
     }

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/AttachmentFileNameIs.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/AttachmentFileNameIs.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/AttachmentFileNameIs.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/AttachmentFileNameIs.java
 Thu Jul 27 08:10:07 2006
@@ -85,6 +85,9 @@
     /** True if unzip is requested. */
     protected boolean unzipIsRequested;
     
+    /**
+     * @see org.apache.mailet.GenericMatcher#init()
+     */
     public void init() throws MessagingException {
         /* sets up fileNameMasks variable by parsing the condition */
         

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/CommandForListserv.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/CommandForListserv.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/CommandForListserv.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/CommandForListserv.java
 Thu Jul 27 08:10:07 2006
@@ -36,10 +36,16 @@
 
     private MailAddress listservAddress;
 
+    /**
+     * @see org.apache.mailet.GenericRecipientMatcher#init()
+     */
     public void init() throws MessagingException {
         listservAddress = new MailAddress(getCondition());
     }
 
+    /**
+     * @see 
org.apache.mailet.GenericRecipientMatcher#matchRecipient(MailAddress)
+     */
     public boolean matchRecipient(MailAddress recipient) {
         if (recipient.getHost().equals(listservAddress.getHost())) {
             if (recipient.getUser().equals(listservAddress.getUser() + "-on")

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/CompareNumericHeaderValue.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/CompareNumericHeaderValue.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/CompareNumericHeaderValue.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/CompareNumericHeaderValue.java
 Thu Jul 27 08:10:07 2006
@@ -63,6 +63,9 @@
     
     private Double headerValue;
 
+    /**
+     * @see org.apache.mailet.GenericMatcher#init()
+     */
     public void init() throws MessagingException {
         StringTokenizer st = new StringTokenizer(getCondition(), " \t", false);
         if (st.hasMoreTokens()) {
@@ -118,6 +121,9 @@
         }
     }
 
+    /**
+     * @see org.apache.mailet.GenericMatcher#match(Mail)
+     */
     public Collection match(Mail mail) throws MessagingException {
         if (headerName == null) {
             // should never get here

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/FetchedFrom.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/FetchedFrom.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/FetchedFrom.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/FetchedFrom.java
 Thu Jul 27 08:10:07 2006
@@ -37,6 +37,10 @@
  */
 
 public class FetchedFrom extends GenericMatcher {
+    
+    /**
+     * @see org.apache.mailet.GenericMatcher#match(Mail)
+     */
     public Collection match(Mail mail) throws javax.mail.MessagingException {
         MimeMessage message = mail.getMessage();
         String fetch = message.getHeader("X-fetched-from", null);

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/FileRegexMatcher.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/FileRegexMatcher.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/FileRegexMatcher.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/FileRegexMatcher.java
 Thu Jul 27 08:10:07 2006
@@ -29,6 +29,10 @@
  *
  */
 public class FileRegexMatcher extends GenericRegexMatcher {
+    
+    /**
+     * @see org.apache.james.transport.matchers.GenericRegexMatcher#init()
+     */
     public void init() throws MessagingException {
         try {
             java.io.RandomAccessFile patternSource = new 
java.io.RandomAccessFile(getCondition(), "r");

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/GenericRegexMatcher.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/GenericRegexMatcher.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/GenericRegexMatcher.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/GenericRegexMatcher.java
 Thu Jul 27 08:10:07 2006
@@ -56,8 +56,14 @@
         }
     }
 
+    /**
+     * @see org.apache.mailet.GenericMatcher#GenericMatcher()
+     */
     abstract public void init() throws MessagingException;
 
+    /**
+     * @see org.apache.mailet.GenericMatcher#match(Mail)
+     */
     public Collection match(Mail mail) throws MessagingException {
         MimeMessage message = mail.getMessage();
         Perl5Matcher matcher = new Perl5Matcher();

Modified: 
james/server/trunk/src/java/org/apache/james/transport/matchers/IsInWhiteList.java
URL: 
http://svn.apache.org/viewvc/james/server/trunk/src/java/org/apache/james/transport/matchers/IsInWhiteList.java?rev=426098&r1=426097&r2=426098&view=diff
==============================================================================
--- 
james/server/trunk/src/java/org/apache/james/transport/matchers/IsInWhiteList.java
 (original)
+++ 
james/server/trunk/src/java/org/apache/james/transport/matchers/IsInWhiteList.java
 Thu Jul 27 08:10:07 2006
@@ -102,14 +102,8 @@
     }
 
     /**
-     * Setter for property sqlParameters.
-     * @param sqlParameters New value of property sqlParameters.
+     * @see org.apache.mailet.GenericMailet#init()
      */
-    private void setSqlParameters(Map sqlParameters) {
-
-        this.sqlParameters = sqlParameters;
-    }
-
     public void init() throws javax.mail.MessagingException {
         String repositoryPath = null;
         StringTokenizer st = new StringTokenizer(getCondition(), ", \t", 
false);
@@ -153,6 +147,9 @@
         selectByPK = sqlQueries.getSqlString("selectByPK", true);
     }
 
+    /**
+     * @see org.apache.mailet.GenericMatcher#match(Mail)
+     */
     public Collection match(Mail mail) throws MessagingException {
         // check if it's a local sender
         MailAddress senderMailAddress = mail.getSender();
@@ -253,7 +250,7 @@
      * for finding the conf/sqlResources.xml file
      * @throws Exception If any error occurs
      */
-    public void initSqlQueries(Connection conn, 
org.apache.mailet.MailetContext mailetContext) throws Exception {
+    private void initSqlQueries(Connection conn, 
org.apache.mailet.MailetContext mailetContext) throws Exception {
         try {
             if (conn.getAutoCommit()) {
                 conn.setAutoCommit(false);



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to