> On Dec 6, 2016, at 1:36 AM, Sergei Kovalev <sergei.kova...@oracle.com> wrote:
> 
> Hi Daniel,
> 
> Please take a look at http://cr.openjdk.java.net/~skovalev/8170664/webrev.01/

 109         boolean simleConsoleOnly = 
!Layer.boot().findModule("java.logging").isPresent();

typo: s/simle/simple

 107         Class sploggerType = splogger.getClass();
  :
 123             Class sloggerType = slogger.getClass();
 124             System.out.println("slogger: " + sloggerType);
 125             if (sloggerType.equals(sploggerType)) {

This check is redundant.  Is this the intended check?

Assuming the above check is not needed, you can further simplify something like 
this:

String expectedType = Layer.boot().findModule("java.logging").isPresent()
    ? "SimpleConsoleLogger" : "JdkLazyLogger”;

Mandy

Reply via email to