Looks fine shura > On Jul 24, 2018, at 6:48 PM, Alexandre (Shura) Iline > <[email protected]> wrote: > > Hi, > > Could you please tale a quick look on this simple fix? > > diff --git a/test/jtreg-ext/requires/VMProps.java > b/test/jtreg-ext/requires/VMProps.java > --- a/test/jtreg-ext/requires/VMProps.java > +++ b/test/jtreg-ext/requires/VMProps.java > @@ -432,7 +432,8 @@ > System.getProperty("java.home") + "/release"))) { > Properties properties = new Properties(); > properties.load(in); > - return properties.getProperty("IMPLEMENTOR").replace("\"", ""); > + String implementorProperty = > properties.getProperty("IMPLEMENTOR"); > + return (implementorProperty == null) ? "null" : > implementorProperty.replace("\"", ""); > } catch (IOException e) { > e.printStackTrace(); > } > > > Thank you. > > Shura
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 [email protected] <mailto:[email protected]>
