RFR: 8240629: argfiles parsing broken for argfiles with comment cross 4096 bytes chunk
Henry Jen
henry.jen at oracle.com
Sun Mar 8 18:27:49 UTC 2020
Thanks for the review, I updated the webrev[1] with simplified test and ensure other cases in boundary won’t be causing trouble by only take meaningful tokens.
This fix is more defensive and allow anchor to be ignored when it’s meaningless.
[1] http://cr.openjdk.java.net/~henryjen/jdk/8240629.1/webrev/
Cheers,
Henry
> On Mar 8, 2020, at 1:54 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>
> On 06/03/2020 22:40, Henry Jen wrote:
>> Hi,
>>
>> Please review the webrev[1] fix JDK-8240629 reported earlier by Robert.
>>
>> http://cr.openjdk.java.net/~henryjen/jdk/8240629.0/webrev/
>>
> The changes and test look okay.
>
> -Alan
More information about the core-libs-dev
mailing list