Update of /cvsroot/freenet/freenet/src/freenet/message
In directory sc8-pr-cvs1:/tmp/cvs-serv15810/src/freenet/message

Modified Files:
        Request.java 
Log Message:
6249:
Iakin's fix for Request.java - nasty bug, may have caused requests from recent nodes 
to fail.
Try to fix lock contention in RandomAccessFilePool. Code review would be appreciated. 
Catch Throwables when closing files.
Reduce outLimitCutoff to 80%. Thus the node will reject all queries if it is using 
more than 80% of outbound bandwidth. The previous 90% figure conflicted with the hard 
bandwidth limit of 120%, which (because it is deliberately conservative), was 
preventing it from going over 90%.
Implement removal of PeerHandlers from OCM, when either
a) No connections and no contact details, and not in routing table, regardless of 
whether we have queued messages or
b) No connections, no messages.
- Some tricky concurrency issues, hence the new RemovingPeerHandlerException
Track number and last event time for outbound connection attempts, successes, and 
failures, in PeerHandler.
Add PeerHandler.probablyNotConnectable()
Don't queue messages if we are unlikely to be able to open a connection and we have no 
current connections
Diagnostics - messageSendTime* especially. Added messageSendTime{Request,NonRequest}, 
fixed messageSendTimeNoQR.
Make sending a QR because of a loop in RequestDone asynchronous.
Don't add PeerHandlers with null IDs in OCM (probably not used).
Lots of logging, a few toString()'s, indenting
- remove some spurious logging, for example, Accepted before SendFinished is okay 
unless it's a lot before it, or the SendFinished is unsuccessful


Index: Request.java
===================================================================
RCS file: /cvsroot/freenet/freenet/src/freenet/message/Request.java,v
retrieving revision 1.9
retrieving revision 1.10
diff -u -w -r1.9 -r1.10
--- Request.java        9 Oct 2003 00:23:40 -0000       1.9
+++ Request.java        15 Oct 2003 21:15:48 -0000      1.10
@@ -45,7 +45,7 @@
             // there is no reason to verify this.
            if (nodeRef == null) {
                requestSource = source.targetReference();
-               if(nodeRef == null)
+               if(requestSource == null)
                    throw new InvalidMessageException("Can't find Source field");
            } else {
                requestSource = new NodeReference(nodeRef, false, 

_______________________________________________
cvs mailing list
[EMAIL PROTECTED]
http://dodo.freenetproject.org/cgi-bin/mailman/listinfo/cvs

Reply via email to