Erik-
Looks good to me as well.
/Tim
On 06/18/18 13:58, Phil Race wrote:
+1
-phil.
On 06/18/2018 01:34 PM, Erik Joelsson wrote:
When using VS2017, configure prints a warning about the toolchain
being unsupported. This needs to change now that this is the main
toolchain version for OpenJDK on Windows (at least for Oracle).
This patch removes the warning for VS2017 and adds it back for all
other versions of Visual Studio. I've made it possible to toggle this
individually for each VS version so if someone wants to claim support
for another VS version, they can flip it back.
Bug: https://bugs.openjdk.java.net/browse/JDK-8205183
Webrev: http://cr.openjdk.java.net/~erikj/8205183/webrev.01/
/Erik