On Thursday 16 April 2009 23:42:32 nextgens at freenetproject.org wrote:
> Author: nextgens
> Date: 2009-04-16 22:42:32 +0000 (Thu, 16 Apr 2009)
> New Revision: 26911
>
> Modified:
> branches/nextgens-stuffs/freenet/src/freenet/io/comm/Message.java
> branches/nextgens-stuffs/freenet/src/freenet/io/comm/MessageFilter.java
> Log:
> Cheap optimization
>
> Modified: branches/nextgens-stuffs/freenet/src/freenet/io/comm/Message.java
> ===================================================================
> --- branches/nextgens-stuffs/freenet/src/freenet/io/comm/Message.java
2009-04-16 22:38:46 UTC (rev 26910)
> +++ branches/nextgens-stuffs/freenet/src/freenet/io/comm/Message.java
2009-04-16 22:42:32 UTC (rev 26911)
> @@ -285,6 +285,10 @@
> public MessageType getSpec() {
> return _spec;
> }
> +
> + public int fieldCount() {
> + return _payload.size();
> + }
>
> public boolean isSet(String fieldName) {
> return _payload.containsKey(fieldName);
>
> Modified:
branches/nextgens-stuffs/freenet/src/freenet/io/comm/MessageFilter.java
> ===================================================================
> --- branches/nextgens-stuffs/freenet/src/freenet/io/comm/MessageFilter.java
2009-04-16 22:38:46 UTC (rev 26910)
> +++ branches/nextgens-stuffs/freenet/src/freenet/io/comm/MessageFilter.java
2009-04-16 22:42:32 UTC (rev 26911)
> @@ -186,6 +186,9 @@
> return false;
> }
> synchronized (_fields) {
> + if(_fieldList.size() > m.fieldCount())
> + return false;
Looks bogus to me.
> +
> for (String fieldName : _fieldList) {
> if (!m.isSet(fieldName)) {
> return false;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 835 bytes
Desc: This is a digitally signed message part.
URL:
<https://emu.freenetproject.org/pipermail/devl/attachments/20090428/b7af20a9/attachment.pgp>
