On Friday 04 January 2008 17:22, robert at freenetproject.org wrote:
> Author: robert
> Date: 2008-01-04 17:22:34 +0000 (Fri, 04 Jan 2008)
> New Revision: 16885
>
> Modified:
> trunk/freenet/src/freenet/node/useralerts/UserAlertManager.java
> Log:
> more future-proof and support less-than-minor messages
>
I'm not sure I get it - it's more future proof because we don't need to put a
messageTypes++ in if we add another type of message?
>
> Modified: trunk/freenet/src/freenet/node/useralerts/UserAlertManager.java
> ===================================================================
> --- trunk/freenet/src/freenet/node/useralerts/UserAlertManager.java
2008-01-04 17:07:15 UTC (rev 16884)
> +++ trunk/freenet/src/freenet/node/useralerts/UserAlertManager.java
2008-01-04 17:22:34 UTC (rev 16885)
> @@ -204,8 +204,9 @@
> separatorNeeded = true;
> messageTypes++;
> }
> - if (messageTypes > 1) {
> - alertSummaryString.append(" | ");
> + if (messageTypes != 1) {
> + if (separatorNeeded)
> + alertSummaryString.append(" | ");
>
alertSummaryString.append(l10n("totalLabel")).append(' ').append(totalNumber);
> }
> HTMLNode summaryBox = null;
>
> _______________________________________________
> 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/20080108/7c5fab5f/attachment.pgp>
