I suggest to rename it to jdk.test.lib.LockFreeLogger. It does not use java.util.logging and it could be in the same package as the other utility.
Mandy > On May 17, 2017, at 5:13 PM, Paul Sandoz <paul.san...@oracle.com> wrote: > > > It’s this one :-) > > https://bugs.openjdk.java.net/browse/JDK-8180399 > <https://bugs.openjdk.java.net/browse/JDK-8180399> > http://cr.openjdk.java.net/~iignatyev/8180399/webrev.00/ > <http://cr.openjdk.java.net/~iignatyev/8180399/webrev.00/> > > Does the Locks test need to depend on stuff in /lib/testlibrary ? > > Paul. > >> On 16 May 2017, at 21:21, Igor Ignatyev <igor.ignat...@oracle.com> wrote: >> >> http://cr.openjdk.java.net/~iignatyev//8180399/webrev.00/index.html >>> 186 lines changed: 87 ins; 90 del; 9 mod; >> >> Hi all, >> >> could you please review this small patch which moves LockFreeLogManager >> class from jdk test library to the top level test library? >> this fix is a part of ongoing effort on merging and cleaning up our test >> libraries[1]. >> >> webrev: http://cr.openjdk.java.net/~iignatyev//8180397/webrev.00/index.html >> JBS: https://bugs.openjdk.java.net/browse/JDK-8180397 >> testing: affected tests >> >> [1] https://bugs.openjdk.java.net/browse/JDK-8075327 >> >> Thanks, >> -- Igor >> >> >
