In the test I only replaced
import java.lang.reflect.*;
with
import java.lang.reflect.Modifier;
and added the new bugid to the @bug tag.
Sincerely yours,
Ivan
On 19.07.2014 2:53, Martin Buchholz wrote:
It looks like the test has a new import, but no actual new code that
uses it.
On Fri, Jul 18, 2014 at 3:39 PM, Ivan Gerasimov
<ivan.gerasi...@oracle.com <mailto:ivan.gerasi...@oracle.com>> wrote:
Hello!
StringBuilder can be replaced with StringJoiner, which will make
the code shorter and, probably, more efficient.
The existing regtest
(java/lang/reflect/Modifier/toStringTest.java) can be used to make
sure nothing is broken.
Would you please help review the fix?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8051382
WEBREV: http://cr.openjdk.java.net/~igerasim/8051382/0/webrev/
<http://cr.openjdk.java.net/%7Eigerasim/8051382/0/webrev/>
Sincerely yours,
Ivan
