RFR: 8171855: Move package name transformations during module bootstrap into native code
claes.redestad at oracle.com
Fri Jan 6 14:23:39 UTC 2017
On 2017-01-06 15:07, Alan Bateman wrote:
> On 06/01/2017 13:18, Claes Redestad wrote:
>> Updated jdk webrev in-place:
> I looked through the latest webrev and it looks quite good. Part of me
> regrets introducing JNI code of course but this is startup motivated.
> In Module.c then I assume GetInternalPackageName should be static
> getInternalPackageName as the scope is just this file (no need for
Done, will update in place.
> Do we have tests that use package names > 128 to test cases where the
> buffer on the stack not large enough?
I'm not sure; the malloc'ing path is exercised by DefineModule, but it
would make sense to add a sanity test to each method I guess.
More information about the jigsaw-dev