Author: zothar
Date: 2008-01-09 00:00:39 +0000 (Wed, 09 Jan 2008)
New Revision: 16973
Modified:
trunk/freenet/src/freenet/node/DarknetPeerNode.java
trunk/freenet/src/freenet/node/Node.java
Log:
Remove some support for old-style N2NTMs. Rename some methods and variables
related to N2NTM -> N2NM generalizations.
Modified: trunk/freenet/src/freenet/node/DarknetPeerNode.java
===================================================================
--- trunk/freenet/src/freenet/node/DarknetPeerNode.java 2008-01-08 23:31:25 UTC
(rev 16972)
+++ trunk/freenet/src/freenet/node/DarknetPeerNode.java 2008-01-09 00:00:39 UTC
(rev 16973)
@@ -76,8 +76,8 @@
/** Private comment on the peer for /friends/ page's extra peer data file
number */
private int privateDarknetCommentFileNumber;
- /** Queued-to-send N2NTM extra peer data file numbers */
- private LinkedHashSet queuedToSendN2NTMExtraPeerDataFileNumbers;
+ /** Queued-to-send N2NM extra peer data file numbers */
+ private LinkedHashSet queuedToSendN2NMExtraPeerDataFileNumbers;
private static boolean logMINOR;
@@ -113,8 +113,8 @@
// Setup the extraPeerDataFileNumbers
extraPeerDataFileNumbers = new LinkedHashSet();
- // Setup the queuedToSendN2NTMExtraPeerDataFileNumbers
- queuedToSendN2NTMExtraPeerDataFileNumbers = new LinkedHashSet();
+ // Setup the queuedToSendN2NMExtraPeerDataFileNumbers
+ queuedToSendN2NMExtraPeerDataFileNumbers = new LinkedHashSet();
}
@@ -483,12 +483,11 @@
}
Logger.error(this, "Read unknown peer note type
'"+peerNoteType+"' from file "+extraPeerDataFile.getPath());
return false;
- } else if(extraPeerDataType ==
Node.EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NTM) {
+ } else if(extraPeerDataType ==
Node.EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NM) {
boolean sendSuccess = false;
- int type = fs.getInt("n2nType", 1); // FIXME remove
default
- fs.putOverwrite("n2nType", Integer.toString(type));
+ int type = fs.getInt("n2nType");
if(isConnected()) {
- Message n2ntm;
+ Message n2nm;
if(fs.get("extraPeerDataType") != null) {
fs.removeValue("extraPeerDataType");
}
@@ -502,18 +501,18 @@
fs.putOverwrite("sentTime",
Long.toString(System.currentTimeMillis()));
try {
- n2ntm =
DMT.createNodeToNodeMessage(type, fs.toString().getBytes("UTF-8"));
+ n2nm =
DMT.createNodeToNodeMessage(type, fs.toString().getBytes("UTF-8"));
} catch (UnsupportedEncodingException e) {
Logger.error(this,
"UnsupportedEncodingException processing extraPeerDataType
("+extraPeerDataTypeString+") in file "+extraPeerDataFile.getPath(), e);
return false;
}
try {
-
synchronized(queuedToSendN2NTMExtraPeerDataFileNumbers) {
- node.usm.send(this, n2ntm,
null);
- Logger.normal(this, "Sent
queued ("+fileNumber+") N2NTM to '"+getName()+"': "+n2ntm);
+
synchronized(queuedToSendN2NMExtraPeerDataFileNumbers) {
+ node.usm.send(this, n2nm, null);
+ Logger.normal(this, "Sent
queued ("+fileNumber+") N2NM to '"+getName()+"': "+n2nm);
sendSuccess = true;
-
queuedToSendN2NTMExtraPeerDataFileNumbers.remove(new Integer(fileNumber));
+
queuedToSendN2NMExtraPeerDataFileNumbers.remove(new Integer(fileNumber));
}
deleteExtraPeerDataFile(fileNumber);
} catch (NotConnectedException e) {
@@ -521,10 +520,10 @@
}
}
if(!sendSuccess) {
-
synchronized(queuedToSendN2NTMExtraPeerDataFileNumbers) {
+
synchronized(queuedToSendN2NMExtraPeerDataFileNumbers) {
fs.putOverwrite("extraPeerDataType",
Integer.toString(extraPeerDataType));
fs.removeValue("sentTime");
-
queuedToSendN2NTMExtraPeerDataFileNumbers.add(new Integer(fileNumber));
+
queuedToSendN2NMExtraPeerDataFileNumbers.add(new Integer(fileNumber));
}
}
return true;
@@ -723,19 +722,19 @@
}
}
- public void queueN2NTM(SimpleFieldSet fs) {
- int fileNumber = writeNewExtraPeerDataFile( fs,
Node.EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NTM);
- synchronized(queuedToSendN2NTMExtraPeerDataFileNumbers) {
- queuedToSendN2NTMExtraPeerDataFileNumbers.add(new
Integer(fileNumber));
+ public void queueN2NM(SimpleFieldSet fs) {
+ int fileNumber = writeNewExtraPeerDataFile( fs,
Node.EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NM);
+ synchronized(queuedToSendN2NMExtraPeerDataFileNumbers) {
+ queuedToSendN2NMExtraPeerDataFileNumbers.add(new
Integer(fileNumber));
}
}
- public void sendQueuedN2NTMs() {
+ public void sendQueuedN2NMs() {
if(logMINOR)
- Logger.minor(this, "Sending queued N2NTMs for
"+shortToString());
+ Logger.minor(this, "Sending queued N2NMs for
"+shortToString());
Integer[] localFileNumbers = null;
- synchronized(queuedToSendN2NTMExtraPeerDataFileNumbers) {
- localFileNumbers = (Integer[])
queuedToSendN2NTMExtraPeerDataFileNumbers.toArray(new
Integer[queuedToSendN2NTMExtraPeerDataFileNumbers.size()]);
+ synchronized(queuedToSendN2NMExtraPeerDataFileNumbers) {
+ localFileNumbers = (Integer[])
queuedToSendN2NMExtraPeerDataFileNumbers.toArray(new
Integer[queuedToSendN2NMExtraPeerDataFileNumbers.size()]);
}
Arrays.sort(localFileNumbers);
for (int i = 0; i < localFileNumbers.length; i++) {
@@ -761,11 +760,11 @@
* A method to be called once at the beginning of every time
isConnected() is true
*/
protected void onConnect() {
- sendQueuedN2NTMs();
+ sendQueuedN2NMs();
}
// File transfer offers
- // FIXME this should probably be somewhere else, along with the N2NTM
stuff... but where?
+ // FIXME this should probably be somewhere else, along with the N2NM
stuff... but where?
// FIXME this should be persistent across node restarts
/** Files I have offered to this peer */
@@ -1273,7 +1272,7 @@
if(includeSentTime) {
fs.removeValue("sentTime");
}
- queueN2NTM(fs);
+ queueN2NM(fs);
}
} catch (UnsupportedEncodingException e) {
throw new Error("Impossible: "+e, e);
@@ -1302,7 +1301,7 @@
sendAsync(n2ntm, null, 0, null);
} catch (NotConnectedException e) {
fs.removeValue("sentTime");
- queueN2NTM(fs);
+ queueN2NM(fs);
setPeerNodeStatus(System.currentTimeMillis());
return getPeerNodeStatus();
}
@@ -1335,7 +1334,7 @@
sendAsync(n2ntm, null, 0, null);
} catch (NotConnectedException e) {
fs.removeValue("sentTime");
- queueN2NTM(fs);
+ queueN2NM(fs);
setPeerNodeStatus(System.currentTimeMillis());
return getPeerNodeStatus();
}
@@ -1377,7 +1376,7 @@
sendAsync(n2ntm, null, 0, null);
} catch (NotConnectedException e) {
fs.removeValue("sentTime");
- queueN2NTM(fs);
+ queueN2NM(fs);
setPeerNodeStatus(System.currentTimeMillis());
return getPeerNodeStatus();
}
Modified: trunk/freenet/src/freenet/node/Node.java
===================================================================
--- trunk/freenet/src/freenet/node/Node.java 2008-01-08 23:31:25 UTC (rev
16972)
+++ trunk/freenet/src/freenet/node/Node.java 2008-01-09 00:00:39 UTC (rev
16973)
@@ -371,7 +371,7 @@
public static final int N2N_TEXT_MESSAGE_TYPE_FILE_OFFER_REJECTED = 4;
public static final int EXTRA_PEER_DATA_TYPE_N2NTM = 1;
public static final int EXTRA_PEER_DATA_TYPE_PEER_NOTE = 2;
- public static final int EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NTM = 3;
+ public static final int EXTRA_PEER_DATA_TYPE_QUEUED_TO_SEND_N2NM = 3;
public static final int PEER_NOTE_TYPE_PRIVATE_DARKNET_COMMENT = 1;
/** The bootID of the last time the node booted up. Or -1 if we don't
know due to
@@ -2602,7 +2602,7 @@
public void handleNodeToNodeTextMessageSimpleFieldSet(SimpleFieldSet
fs, DarknetPeerNode source, int fileNumber) throws FSParseException {
if(logMINOR)
Logger.minor(this, "Got node to node message: \n"+fs);
- int overallType = fs.getInt("n2nType", 1); // FIXME remove
default
+ int overallType = fs.getInt("n2nType");
fs.removeValue("n2nType");
if(overallType == Node.N2N_MESSAGE_TYPE_FPROXY) {
handleFproxyNodeToNodeTextMessageSimpleFieldSet(fs,
source, fileNumber);