A minor point: the usual idiom we use for rethrowing RuntimeException is

} catch(final RuntimeException e) {
  throw e;
} catch(final Exception e) {
  throw new RuntimeException(e);
}

instead of what you did in Global.java with an "if" in the "catch"

+1 other than that.

Attila.


On Thu, Jun 23, 2016 at 6:28 AM, Sundararajan Athijegannathan <
sundararajan.athijegannat...@oracle.com> wrote:

> Please review http://cr.openjdk.java.net/~sundar/8160141/webrev.00/ for
> https://bugs.openjdk.java.net/browse/JDK-8160141
>
> Thanks,
>
> Sundar
>
>

Reply via email to