Hi Alan,
please review the update patch, and reduced checking as suggested:
http://cr.openjdk.java.net/~xiaofeya/8194230/webrev.01/
Thanks,
Felix
On 2018/8/6 18:23, Alan Bateman wrote:
On 31/07/2018 07:16, Felix Yang wrote:
Hi all,
please review a patch to improve the checking on the settings.
Originally it will throw NPE, if specified path is invalid.
Bug:
https://bugs.openjdk.java.net/browse/JDK-8194230
Webrev:
http://cr.openjdk.java.net/~xiaofeya/8194230/webrev.00/
Have you considered merging getJdk8Path and isJdk8 so that it simple
checks for the release file and rejects the value of JDK8_HOME when it
doesn't exist? I think that should reduce the number of checks and
keep it very simple.
-Alan