RFR: 8240629: argfiles parsing broken for argfiles with comment cross 4096 bytes chunk

Mandy Chung mandy.chung at oracle.com
Mon Mar 9 03:56:23 UTC 2020

On 3/8/20 11:27 AM, Henry Jen wrote:
> 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/

This patch looks okay.


More information about the core-libs-dev mailing list