Author: nextgens
Date: 2006-07-11 16:48:23 +0000 (Tue, 11 Jul 2006)
New Revision: 9561

Modified:
   trunk/freenet/src/freenet/client/async/SimpleManifestPutter.java
   trunk/freenet/src/freenet/client/async/SingleFileInserter.java
   trunk/freenet/src/freenet/clients/http/ToadletContextImpl.java
   trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java
   trunk/freenet/src/freenet/config/StringArrOption.java
   trunk/freenet/src/freenet/crypt/SHA1.java
   trunk/freenet/src/freenet/io/comm/UdpSocketManager.java
   trunk/freenet/src/freenet/node/IPDetectorPluginManager.java
   trunk/freenet/src/freenet/node/InsertHandler.java
   trunk/freenet/src/freenet/node/SSKInsertHandler.java
   trunk/freenet/src/freenet/support/NumberedRecentItems.java
   trunk/freenet/src/freenet/support/io/PersistentTempBucketFactory.java
   trunk/freenet/src/freenet/transport/IPAddressDetector.java
Log:
Second part of the potentially controversial patch : it removes more dead-code 
and suppresses many eclipse warning

Modified: trunk/freenet/src/freenet/client/async/SimpleManifestPutter.java
===================================================================
--- trunk/freenet/src/freenet/client/async/SimpleManifestPutter.java    
2006-07-11 16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/client/async/SimpleManifestPutter.java    
2006-07-11 16:48:23 UTC (rev 9561)
@@ -32,44 +32,36 @@

                protected PutHandler(String name, Bucket data, ClientMetadata 
cm, boolean getCHKOnly) throws InserterException {
                        super(SimpleManifestPutter.this.getPriorityClass(), 
SimpleManifestPutter.this.chkScheduler, SimpleManifestPutter.this.sskScheduler, 
SimpleManifestPutter.this.client);
-                       this.name = name;
                        this.cm = cm;
                        this.data = data;
                        InsertBlock block = 
                                new InsertBlock(data, cm, 
FreenetURI.EMPTY_CHK_URI);
                        this.origSFI =
                                new SingleFileInserter(this, this, block, 
false, ctx, false, getCHKOnly, true, null, false);
-                       currentState = origSFI;
                        metadata = null;
                }

                protected PutHandler(String name, FreenetURI target, 
ClientMetadata cm) {
                        super(SimpleManifestPutter.this.getPriorityClass(), 
SimpleManifestPutter.this.chkScheduler, SimpleManifestPutter.this.sskScheduler, 
SimpleManifestPutter.this.client);
-                       this.name = name;
                        this.cm = cm;
                        this.data = null;
                        Metadata m = new Metadata(Metadata.SIMPLE_REDIRECT, 
target, cm);
                        metadata = m;
                        origSFI = null;
-                       currentState = null;
                }

                protected PutHandler(String name, String targetInZip, 
ClientMetadata cm, Bucket data) {
                        super(SimpleManifestPutter.this.getPriorityClass(), 
SimpleManifestPutter.this.chkScheduler, SimpleManifestPutter.this.sskScheduler, 
SimpleManifestPutter.this.client);
-                       this.name = name;
                        this.cm = cm;
                        this.data = data;
                        this.targetInZip = targetInZip;
                        Metadata m = new 
Metadata(Metadata.ZIP_INTERNAL_REDIRECT, targetInZip, cm);
                        metadata = m;
                        origSFI = null;
-                       currentState = null;
                }

                private SingleFileInserter origSFI;
-               private ClientPutState currentState;
                private ClientMetadata cm;
-               private final String name;
                private Metadata metadata;
                private boolean finished;
                private String targetInZip;
@@ -118,9 +110,6 @@

                public void onTransition(ClientPutState oldState, 
ClientPutState newState) {
                        if(oldState == this) {
-                               // We do not need to have a hashtable of state 
-> PutHandler.
-                               // Because we can just pull the parent off the 
state!
-                               this.currentState = newState;
                        }
                }


Modified: trunk/freenet/src/freenet/client/async/SingleFileInserter.java
===================================================================
--- trunk/freenet/src/freenet/client/async/SingleFileInserter.java      
2006-07-11 16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/client/async/SingleFileInserter.java      
2006-07-11 16:48:23 UTC (rev 9561)
@@ -42,7 +42,6 @@
        final boolean insertAsArchiveManifest;
        /** If true, we are not the top level request, and should not
         * update our parent to point to us as current put-stage. */
-       private boolean cancelled;
        private boolean reportMetadataOnly;
        public final Object token;

@@ -396,7 +395,6 @@
        }

        public void cancel() {
-               cancelled = true;
        }

        public void schedule() throws InserterException {

Modified: trunk/freenet/src/freenet/clients/http/ToadletContextImpl.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/ToadletContextImpl.java      
2006-07-11 16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/clients/http/ToadletContextImpl.java      
2006-07-11 16:48:23 UTC (rev 9561)
@@ -28,7 +28,6 @@
  */
 public class ToadletContextImpl implements ToadletContext {

-       private final Socket sock;
        private final MultiValueTable headers;
        private final OutputStream sockOutputStream;
        private final PageMaker pagemaker;
@@ -40,7 +39,6 @@
        private boolean closed;

        public ToadletContextImpl(Socket sock, MultiValueTable headers, String 
CSSName, BucketFactory bf) throws IOException {
-               this.sock = sock;
                this.headers = headers;
                this.closed = false;
                sockOutputStream = sock.getOutputStream();

Modified: trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java
===================================================================
--- trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java       
2006-07-11 16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/clients/http/filter/HTMLFilter.java       
2006-07-11 16:48:23 UTC (rev 9561)
@@ -32,32 +32,18 @@

 public class HTMLFilter implements ContentDataFilter, CharsetExtractor {

-       private boolean debug = false;
        private static boolean deleteWierdStuff = true;
        private static boolean deleteErrors = true;
-       private boolean allowSecurityErrors = false;
-       private boolean allowSecurityWarnings = false;
-       private boolean cssParanoidStringCheck = false;
-
-       private final static String possibleAnonCompromiseMsg =
-               "You have retrieved some content which is not recognised by 
FProxy, and so we "
-                       + "don't know what your web browser might do with it.  
It could be harmless, "
-                       + "but it could make your web browser do something 
which would compromise your "
-                       + "anonymity.";
        public void setParanoidStringCheck(boolean b) {
-               cssParanoidStringCheck = b;
        }

        public void setAllowSecurityWarnings(boolean value) {
-               allowSecurityWarnings = value;
        }

        public void setAllowSecurityErrors(boolean value) {
-               allowSecurityErrors = value;
        }

        public void setDebug(boolean debug) {
-               this.debug = debug;
        }

        public Bucket readFilter(Bucket bucket, BucketFactory bf, String 
charset, HashMap otherParams, FilterCallback cb) throws DataFilterException, 
IOException {
@@ -413,7 +399,6 @@
                if (!pc.killTag) {
                        t = t.sanitize(pc);
                        if (t != null) {
-                               boolean deletedStyle = false;
                                if (pc.writeStyleScriptWithTag) {
                                        pc.writeStyleScriptWithTag = false;
                                        String style = 
pc.currentStyleScriptChunk;
@@ -1385,7 +1370,7 @@
                        ParsedTag p,
                        HTMLParseContext pc) throws DataFilterException {
                        // Call parent so we swallow the scripting
-                       Hashtable h = super.sanitizeHash(hn, p, pc);
+                       super.sanitizeHash(hn, p, pc);
                        return null; // Lose the tags
                }


Modified: trunk/freenet/src/freenet/config/StringArrOption.java
===================================================================
--- trunk/freenet/src/freenet/config/StringArrOption.java       2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/config/StringArrOption.java       2006-07-11 
16:48:23 UTC (rev 9561)
@@ -67,5 +67,9 @@
                        return null;
                }
        }
+
+       public String getDefaultValue() {
+               return defaultValue;
+       }

 }

Modified: trunk/freenet/src/freenet/crypt/SHA1.java
===================================================================
--- trunk/freenet/src/freenet/crypt/SHA1.java   2006-07-11 16:12:37 UTC (rev 
9560)
+++ trunk/freenet/src/freenet/crypt/SHA1.java   2006-07-11 16:48:23 UTC (rev 
9561)
@@ -58,7 +58,6 @@

     private int state[] = new int[5];
     private long count;
-    private boolean digestValid = false;
     private byte[] digestBits;
     private boolean NSA = true;

@@ -83,7 +82,6 @@
      */
     public SHA1(boolean b) {
         count = 0;
-        digestValid = false;
         NSA = b;
         init();
     }
@@ -250,7 +248,6 @@
         state[4] = 0xC3D2E1F0;
         count = 0;
         digestBits = new byte[20];
-        digestValid = false;
         blockIndex = 0;
     }

@@ -336,7 +333,6 @@
             digestBits[i] = (byte)
                 ((state[i>>2] >>> ((3-(i & 3)) << 3) ) & 0xff);
         }
-        digestValid = true;
     }

     /**
@@ -394,8 +390,6 @@
         for (int i = 0; i < count; i++) 
             sha1.update(data);
         long endSameUpdate = System.currentTimeMillis();
-        byte digested[] = sha1.digest();
-        long afterDigested = System.currentTimeMillis();
         long updateTime = endSameUpdate - startSameUpdate;
         long speed = 0;
         long eachTime = 0;
@@ -414,8 +408,6 @@
         for (int i = 0; i < count; i++) 
             sha1.update(data2[i]);
         long endDifferentUpdate = System.currentTimeMillis();
-        digested = sha1.digest();
-        afterDigested = System.currentTimeMillis();
         updateTime = endDifferentUpdate - differentStartTime;
         speed = 0;
         eachTime = 0;
@@ -454,7 +446,7 @@
      * perfoms a self-test - spits out the test vector outputs on System.out
      */
     public static void SHAselfTest(Digest s) {
-        int i, j;
+        int i;
         // This line may be safely deleted, its to make it easy to see
         // the output of the program.
         System.out.println("SHA-1 Test PROGRAM.");

Modified: trunk/freenet/src/freenet/io/comm/UdpSocketManager.java
===================================================================
--- trunk/freenet/src/freenet/io/comm/UdpSocketManager.java     2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/io/comm/UdpSocketManager.java     2006-07-11 
16:48:23 UTC (rev 9561)
@@ -38,14 +38,12 @@
        /** _filters serves as lock for both */
        private LinkedList _filters = new LinkedList();
        private LinkedList _unclaimed = new LinkedList();
-       private int _unclaimedPos;
        private int _dropProbability;
        private LowLevelFilter lowLevelFilter;
        /** RNG for debugging, used with _dropProbability.
         * NOT CRYPTO SAFE. DO NOT USE FOR THINGS THAT NEED CRYPTO SAFE RNG!
         */
        private Random dropRandom;
-       private boolean _active;
        private boolean _isDone;
        private static UdpSocketManager _usm;
        private static final int MAX_UNMATCHED_FIFO_SIZE = 50000;
@@ -546,7 +544,6 @@

     public void close(boolean exit) {
        Logger.error(this, "Closing.", new Exception("error"));
-               _active = false;
                synchronized (this) {
                        while (!_isDone) {
                                try {

Modified: trunk/freenet/src/freenet/node/IPDetectorPluginManager.java
===================================================================
--- trunk/freenet/src/freenet/node/IPDetectorPluginManager.java 2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/node/IPDetectorPluginManager.java 2006-07-11 
16:48:23 UTC (rev 9561)
@@ -246,8 +246,6 @@
                                                // No locally detected IP, only 
one or two connections.
                                                // Have we had more relatively 
recently?
                                                int count = 0;
-                                               long timeref = now;
-                                               if(firstTimeMaybeFakePeers > 0) 
timeref = firstTimeMaybeFakePeers;
                                                for(int i=0;i<peers.length;i++) 
{
                                                        PeerNode p = peers[i];
                                                        if((!p.isConnected()) 
|| (now - p.lastReceivedPacketTime() < 5*60*1000)) {

Modified: trunk/freenet/src/freenet/node/InsertHandler.java
===================================================================
--- trunk/freenet/src/freenet/node/InsertHandler.java   2006-07-11 16:12:37 UTC 
(rev 9560)
+++ trunk/freenet/src/freenet/node/InsertHandler.java   2006-07-11 16:48:23 UTC 
(rev 9561)
@@ -39,8 +39,6 @@
     private byte[] headers;
     private BlockReceiver br;
     private Thread runThread;
-    private boolean sentSuccess;
-    
     PartiallyReceivedBlock prb;

     InsertHandler(Message req, long id, Node node, long startTime) {
@@ -144,7 +142,6 @@
         if(htl == 0) {
             canCommit = true;
                msg = DMT.createFNPInsertReply(uid);
-               sentSuccess = true;
                try {
                                source.send(msg, this);
                        } catch (NotConnectedException e) {
@@ -253,7 +250,6 @@

             if(status == CHKInsertSender.SUCCESS) {
                msg = DMT.createFNPInsertReply(uid);
-               sentSuccess = true;
                try {
                                        source.send(msg, this);
                                } catch (NotConnectedException e) {

Modified: trunk/freenet/src/freenet/node/SSKInsertHandler.java
===================================================================
--- trunk/freenet/src/freenet/node/SSKInsertHandler.java        2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/node/SSKInsertHandler.java        2006-07-11 
16:48:23 UTC (rev 9561)
@@ -8,7 +8,6 @@
 import freenet.io.comm.Message;
 import freenet.io.comm.MessageFilter;
 import freenet.io.comm.NotConnectedException;
-import freenet.io.xfer.BlockReceiver;
 import freenet.keys.NodeSSK;
 import freenet.keys.SSKBlock;
 import freenet.keys.SSKVerifyException;
@@ -37,12 +36,7 @@
     private SSKInsertSender sender;
     private byte[] data;
     private byte[] headers;
-    private BlockReceiver br;
-    private Thread runThread;
-    private boolean sentSuccess;
     private boolean canCommit;
-    private boolean collided;
-
     SSKInsertHandler(Message req, long id, Node node, long startTime) {
         this.req = req;
         this.node = node;
@@ -79,8 +73,6 @@
     }

     private void realRun() {
-        runThread = Thread.currentThread();
-        
         // Send Accepted
         Message accepted = DMT.createFNPSSKAccepted(uid, pubKey == null);

@@ -160,7 +152,6 @@

         if(htl == 0) {
                Message msg = DMT.createFNPInsertReply(uid);
-               sentSuccess = true;
                try {
                                source.send(msg, this);
                        } catch (NotConnectedException e) {
@@ -259,7 +250,6 @@

             if(status == SSKInsertSender.SUCCESS) {
                Message msg = DMT.createFNPInsertReply(uid);
-               sentSuccess = true;
                try {
                                        source.send(msg, null);
                                } catch (NotConnectedException e) {

Modified: trunk/freenet/src/freenet/support/NumberedRecentItems.java
===================================================================
--- trunk/freenet/src/freenet/support/NumberedRecentItems.java  2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/support/NumberedRecentItems.java  2006-07-11 
16:48:23 UTC (rev 9561)
@@ -15,8 +15,6 @@
     private NumberedItem[] items;
     private int count;
     private Comparator myComparator;
-    private boolean wrapAround;
-    
     /**
      * Create a NumberedRecentItems list.
      * @param maxSize The maximum number of NumberedItems to keep.
@@ -27,7 +25,6 @@
     public NumberedRecentItems(int maxSize, boolean wrap) {
         items = new NumberedItem[maxSize];
         count = 0;
-        wrapAround = wrap;
         myComparator = new NumberedItemComparator(wrap);
     }


Modified: trunk/freenet/src/freenet/support/io/PersistentTempBucketFactory.java
===================================================================
--- trunk/freenet/src/freenet/support/io/PersistentTempBucketFactory.java       
2006-07-11 16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/support/io/PersistentTempBucketFactory.java       
2006-07-11 16:48:23 UTC (rev 9561)
@@ -25,9 +25,6 @@
        /** Directory containing persistent temporary files */
        private final File dir;

-       /** Temp file prefix; anything not matching this prefix will be ignored 
*/
-       private final String prefix;
-       
        /** Original contents of directory */
        private HashSet originalFiles;

@@ -40,7 +37,6 @@
        public PersistentTempBucketFactory(File dir, String prefix, 
RandomSource rand) throws IOException {
                this.dir = dir;
                this.rand = rand;
-               this.prefix = prefix;
                this.fg = new FilenameGenerator(rand, false, dir, prefix);
                if(!dir.exists()) {
                        dir.mkdir();

Modified: trunk/freenet/src/freenet/transport/IPAddressDetector.java
===================================================================
--- trunk/freenet/src/freenet/transport/IPAddressDetector.java  2006-07-11 
16:12:37 UTC (rev 9560)
+++ trunk/freenet/src/freenet/transport/IPAddressDetector.java  2006-07-11 
16:48:23 UTC (rev 9561)
@@ -205,7 +205,6 @@
                        Logger.debug(
                                this,
                                "onGetAddresses found " + v.size() + " 
potential addresses)");
-               boolean detectedInetAddress = false;
                InetAddress addrDetected = null;
                if (v.size() == 0) {
                        Logger.error(this, "No addresses found!");
@@ -229,7 +228,6 @@

                                                addrDetected = i;
                                                //Use the last detected valid 
IP as 'detected' IP
-                                               detectedInetAddress = true;
                                                if ((preferedInetAddress != 
null)
                                                        && addrDetected.equals(
                                                                
preferedInetAddress)) { //Prefer the specified address if it is still available 
to us. Do not look for more ones


Reply via email to