RFR: 8198317: Enhance JavacTool.getTask for flexibility [v3]
Jonathan Gibbons
jjg at openjdk.java.net
Tue Dec 29 03:45:58 UTC 2020
On Tue, 29 Dec 2020 03:40:52 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Revise test.
>
> Any time that tests split a string into lines it is really important to ensure that it works on Windows as well as Linux/Mac platforms, because of the different line endings.
>
> The code looks OK. Alternatively, if you are not concerned to check for the exact line ending sequence, you can use `split("\\R")` to split on any line ending sequence. Either way, it's worth checking the test runs on platforms with both kinds of typical line ending.
Given that the `langtools/tier1` pre-submit tests worked for Linux and Windows, you're OK on the "newline" front.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1896
More information about the compiler-dev
mailing list