On Thursday 24 January 2008 00:12, zothar at freenetproject.org wrote:
> Author: zothar
> Date: 2008-01-24 00:12:10 +0000 (Thu, 24 Jan 2008)
> New Revision: 17218
>
> Modified:
> trunk/freenet/src/freenet/node/DarknetPeerNode.java
> trunk/freenet/src/freenet/node/OpennetPeerNode.java
> trunk/freenet/src/freenet/node/PeerNode.java
> trunk/freenet/src/freenet/node/SeedClientPeerNode.java
> trunk/freenet/src/freenet/node/SeedServerPeerNode.java
> Log:
> Factor getLocalNoderef() out into the separate PeerNode classes, adding it
to SeedClientPeerNode and SeedServerPeerNode in the process
Since they all do the same thing, I suggest you just have one method in
PeerNode. :)
>
> Modified: trunk/freenet/src/freenet/node/DarknetPeerNode.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/DarknetPeerNode.java 2008-01-24
> 00:01:53
UTC (rev 17217)
> +++ trunk/freenet/src/freenet/node/DarknetPeerNode.java 2008-01-24
> 00:12:10
UTC (rev 17218)
> @@ -1531,4 +1531,11 @@
> super.sendInitialMessages();
> sendConnectedDiffNoderef();
> }
> +
> + /**
> + * Return the relevant local node reference related to this peer's type
> + */
> + protected SimpleFieldSet getLocalNoderef() {
> + return crypto.exportPublicFieldSet();
> + }
> }
>
> Modified: trunk/freenet/src/freenet/node/OpennetPeerNode.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/OpennetPeerNode.java 2008-01-24
> 00:01:53
UTC (rev 17217)
> +++ trunk/freenet/src/freenet/node/OpennetPeerNode.java 2008-01-24
> 00:12:10
UTC (rev 17218)
> @@ -101,4 +101,11 @@
> super.sendInitialMessages();
> sendConnectedDiffNoderef();
> }
> +
> + /**
> + * Return the relevant local node reference related to this peer's type
> + */
> + protected SimpleFieldSet getLocalNoderef() {
> + return crypto.exportPublicFieldSet();
> + }
> }
>
> Modified: trunk/freenet/src/freenet/node/PeerNode.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/PeerNode.java 2008-01-24 00:01:53 UTC
(rev 17217)
> +++ trunk/freenet/src/freenet/node/PeerNode.java 2008-01-24 00:12:10 UTC
(rev 17218)
> @@ -294,6 +294,9 @@
> /** Total number of handshake attempts (while in ListenOnly mode) to be
> in
this burst */
> private int listeningHandshakeBurstSize;
>
> + // NodeCrypto for the relevant node reference for this peer's type
(Darknet or Opennet at this time))
> + protected NodeCrypto crypto;
> +
> /**
> * For FNP link setup:
> * The initiator has to ensure that nonces send back by the
> @@ -334,6 +337,7 @@
> myRef = new WeakReference(this);
> this.outgoingMangler = mangler;
> this.node = node2;
> + this.crypto = crypto;
> this.peers = peers;
> this.backedOffPercent = new TimeDecayingRunningAverage(0.0,
> 180000, 0.0,
1.0, node);
> version = fs.get("version");
> @@ -3489,13 +3493,7 @@
> * Return the relevant local node reference related to this peer's type
> */
> protected SimpleFieldSet getLocalNoderef() {
> - if(isDarknet()) {
> - return node.exportDarknetPublicFieldSet();
> - } else if(isOpennet()) {
> - return node.exportOpennetPublicFieldSet();
> - }
> - // What else is there that a differential node reference would
> care
about? Add it here if needed
> - return null;
> + // Do nothing in the default impl
> }
>
> /**
>
> Modified: trunk/freenet/src/freenet/node/SeedClientPeerNode.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/SeedClientPeerNode.java 2008-01-24
00:01:53 UTC (rev 17217)
> +++ trunk/freenet/src/freenet/node/SeedClientPeerNode.java 2008-01-24
00:12:10 UTC (rev 17218)
> @@ -87,4 +87,11 @@
> void startARKFetcher() {
> // Do not start an ARK fetcher.
> }
> +
> + /**
> + * Return the relevant local node reference related to this peer's type
> + */
> + protected SimpleFieldSet getLocalNoderef() {
> + return crypto.exportPublicFieldSet();
> + }
> }
>
> Modified: trunk/freenet/src/freenet/node/SeedServerPeerNode.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/SeedServerPeerNode.java 2008-01-24
00:01:53 UTC (rev 17217)
> +++ trunk/freenet/src/freenet/node/SeedServerPeerNode.java 2008-01-24
00:12:10 UTC (rev 17218)
> @@ -98,4 +98,11 @@
> protected boolean generateIdentityFromPubkey() {
> return false;
> }
> +
> + /**
> + * Return the relevant local node reference related to this peer's type
> + */
> + protected SimpleFieldSet getLocalNoderef() {
> + return crypto.exportPublicFieldSet();
> + }
> }
>
> _______________________________________________
> cvs mailing list
> cvs at freenetproject.org
> http://emu.freenetproject.org/cgi-bin/mailman/listinfo/cvs
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL:
<https://emu.freenetproject.org/pipermail/devl/attachments/20080124/b2fe71c4/attachment.pgp>