LGTM as well… thanks a lot!
> On 5. Nov 2018, at 20:22, Xuelei Fan <xuelei....@oracle.com> wrote: > > It looks fine to me. > > Thanks, > Xuelei > > On 11/5/2018 10:59 AM, Anthony Scarpino wrote: >> Hi, >> I'd like to get a code review of this simple change. It's a simple check to >> make sure the hostname variable is not null and throw a proper exception. >> http://cr.openjdk.java.net/~ascarpino/8211339/webrev.00/ >> Tony
