james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nor...@apache.org
Subject svn commit: r492539 - in /james/server/sandbox/handlerapi-experiment: ./ src/java/org/apache/james/smtpserver/core/filter/fastfail/ src/java/org/apache/james/util/junkscore/ src/test/org/apache/james/util/junkscore/
Date Thu, 04 Jan 2007 12:11:23 GMT
Author: norman
Date: Thu Jan  4 04:11:22 2007
New Revision: 492539

URL: http://svn.apache.org/viewvc?view=rev&rev=492539
Log:
Remove unused classed from prevois JunkScoreHandler implementation

Removed:
    james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/smtpserver/core/filter/fastfail/JunkHandlerData.java
    james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/JunkScoreConfigUtil.java
    james/server/sandbox/handlerapi-experiment/src/test/org/apache/james/util/junkscore/JunkScoreConfigUtilTest.java
Modified:
    james/server/sandbox/handlerapi-experiment/TODO
    james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/ComposedJunkScore.java

Modified: james/server/sandbox/handlerapi-experiment/TODO
URL: http://svn.apache.org/viewvc/james/server/sandbox/handlerapi-experiment/TODO?view=diff&rev=492539&r1=492538&r2=492539
==============================================================================
--- james/server/sandbox/handlerapi-experiment/TODO (original)
+++ james/server/sandbox/handlerapi-experiment/TODO Thu Jan  4 04:11:22 2007
@@ -9,3 +9,4 @@
 - SenderAuthIdentifyVerificationRcptHook use session.getConfigurationData().isVerifyIdentity()
to decide wether to enable itself or not: isn't it better to add the handler to the config
and commenting/uncommenting the handler already do the trick? If so we shoudl simply remove
that configuration and remove the check in the code.
 - I would like to Merge HeloHook and EhloHook to a single hook. They are alternative and
I think there is no need for both in the high-level api. less is more ;-)
   (currently we have a single handler implementing both hook and doing the same thing for
both).
+- Check if we can delete the ComposedJunkScore class. I think the class is not needed anymore
\ No newline at end of file

Modified: james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/ComposedJunkScore.java
URL: http://svn.apache.org/viewvc/james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/ComposedJunkScore.java?view=diff&rev=492539&r1=492538&r2=492539
==============================================================================
--- james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/ComposedJunkScore.java
(original)
+++ james/server/sandbox/handlerapi-experiment/src/java/org/apache/james/util/junkscore/ComposedJunkScore.java
Thu Jan  4 04:11:22 2007
@@ -69,6 +69,7 @@
         Map m = new HashMap(score1.getStoredScores());
         m.putAll(score2.getStoredScores());
         return m;
+        
     }
 
     /**



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Mime
View raw message