8202076: test/jdk/java/io/File/WinSpecialFiles.java on windows with VS2017

Brian Burkhalter brian.burkhalter at oracle.com
Fri May 18 23:55:09 UTC 2018


Hi Ivan,

All good ideas for next week!

Thanks,

Brian

On May 18, 2018, at 4:49 PM, Ivan Gerasimov <ivan.gerasimov at oracle.com> wrote:

> Thanks Brian, the last webrev looks good to me!
> 
> I minor suggestion is that I would recommend to use L'\\' to explicitly indicate it is a wide char literal.
> 
> And the last comment:  Maybe it makes sense to update the test java/io/File/WinSpecialFiles.java to make sure we deal correctly with wildcards in a file name (and that nobody will accidentally optimize away the check of the filename).



More information about the core-libs-dev mailing list