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/


> 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