Review request: JDK-8055723 Replace concat String to append in StringBuilder parameters

Ulf Zibis Ulf.Zibis at CoSoCo.de
Tue Oct 21 00:26:30 UTC 2014


Am 21.10.2014 um 01:02 schrieb Otávio Gonçalves de Santana:
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8055723
>
>
> WEBREV: http://cr.openjdk.java.net/~weijun/8055723/client/webrev.02/
> WEBREV: http://cr.openjdk.java.net/~weijun/8055723/core/webrev.03/

I did not look through all sources.
In Scanner.java I discovered:
1307 sb.append("[delimiters=").append(delimPattern).append(']');
1308         sb.append("[position=").append(position).append(']');
...
Maybe better:
1307         sb.append("[delimiters=").append(delimPattern);
1308         sb.append("][position=").append(position);
...

-Ulf




More information about the core-libs-dev mailing list