My concern with this is that we will want to modify things other than the obvious priority class etc. So the message will get bigger and bigger and bigger, and eventually be as big as Persistent* was in the first place. I dunno if this is a serious problem though.
On Tue, Feb 13, 2007 at 04:35:30PM +0000, bback at freenetproject.org wrote: > Author: bback > Date: 2007-02-13 16:35:29 +0000 (Tue, 13 Feb 2007) > New Revision: 11766 > > Modified: > trunk/freenet/src/freenet/node/fcp/ClientRequest.java > trunk/freenet/src/freenet/node/fcp/ModifyPersistentRequest.java > Log: > corrected methodname > > Modified: trunk/freenet/src/freenet/node/fcp/ClientRequest.java > =================================================================== > --- trunk/freenet/src/freenet/node/fcp/ClientRequest.java 2007-02-12 > 22:19:30 UTC (rev 11765) > +++ trunk/freenet/src/freenet/node/fcp/ClientRequest.java 2007-02-13 > 16:35:29 UTC (rev 11766) > @@ -307,7 +307,7 @@ > * Called after a ModifyPersistentRequest. > * Sends a PersistentRequestModified message to clients if any value > changed. > */ > - public void requestWasModified(String newClientToken, short > newPriorityClass) { > + public void modifyRequest(String newClientToken, short newPriorityClass) > { > > boolean clientTokenChanged = false; > boolean priorityClassChanged = false; > > Modified: trunk/freenet/src/freenet/node/fcp/ModifyPersistentRequest.java > =================================================================== > --- trunk/freenet/src/freenet/node/fcp/ModifyPersistentRequest.java > 2007-02-12 22:19:30 UTC (rev 11765) > +++ trunk/freenet/src/freenet/node/fcp/ModifyPersistentRequest.java > 2007-02-13 16:35:29 UTC (rev 11766) > @@ -73,6 +73,6 @@ > return; > } > > - req.requestWasModified(clientToken, priorityClass); > + req.modifyRequest(clientToken, priorityClass); > } > } > > _______________________________________________ > 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: signature.asc Type: application/pgp-signature Size: 189 bytes Desc: Digital signature URL: <https://emu.freenetproject.org/pipermail/devl/attachments/20070215/536ecb29/attachment.pgp>