RFR: 7903608: Cyclic initialization leads to NPE in header class with global variable [v2]

[Jorn Vernee]

On Thu, 14 Dec 2023 11:02:11 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Maurizio Cimadamore has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Drop spurious option in tests
>
> src/main/java/org/openjdk/jextract/impl/ToplevelBuilder.java line 89:
> 
>> 87:         List<JavaFileObject> files = new ArrayList<>();
>> 88: 
>> 89:         if (headerBuilders.size() == 1) {
> 
> The changes here are the trickiest. Basically, we can't figure out in advance which name header classes should have. So we end up generating something like:
> 
> 
> <headerName>_h#{SUFFIX} extends {PREV_SUFFIX}
> 
> 
> And then leave it to the code here to fixup all the suffixes. For this reason, the name of the runtime helper class cannot just be the current "class name" as that would include the `#{SUFFIX}` bit - but we know that there is going to be _some_ `<headerName>_h` generated at some point, so we can just use that as a runtime helper name.
> 
> Overall, I'm not too proud of this code, but at the same time, after evaluating a couple of alternatives they all seemed too complicated. And, we were doing some magic name adaptation even before.

> but we know that there is going to be some <headerName>_h generated at some point, so we can just use that as a runtime helper name.

So, just to clarify: we use the last header class we generate as the runtime helper (having no name suffix), but the runtime helper methods and primitive layouts are actually in the first header, and they are visible through static inheritance?

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

PR Review Comment: https://git.openjdk.org/jextract/pull/160#discussion_r1426718489