On Fri, 3 Dec 2021 07:29:17 GMT, Toshio Nakamura <tnakam...@openjdk.org> wrote:

> Could you review this fix?
> 
> Problem:
> Custom JRE generated by jlink cannot find non-ASCII named modules included 
> inside the JRE.
> 
> Cause and fix:
> If module or package name was composed by ASCII then non-ASCII characters, 
> ImageStringsReader:stringFromByteBufferMatches() miscalculated the length of 
> matched string. The first part of ASCII characters was missing. This patch 
> corrected the value.
> 
> Testing:
> tier1 and tier2 on Linux have no regression.
> I wasn't able to create an automate test for this issue. I appreciate any 
> advice.

Thank you so much for the valuable advices, Jim, Alan, and Naoto.

The created test calls `ImageStringsReader.stringFromByteBufferMatches()` 
directly.
It's a package private method, and I tried to use library style.

The upper public method, `BasicImageReader.match()`, is hard to prepare data.
Calling jlink is also complex for me. So, I choose this way.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6693

Reply via email to