RFR: 7903590: Refactor jextract source code generation [v2]
Jorn Vernee
jvernee at openjdk.org
Mon Nov 27 13:59:42 UTC 2023
On Mon, 27 Nov 2023 13:26:21 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
>> Jorn Vernee has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains ten additional commits since the last revision:
>>
>> - Merge branch 'panama' into Untangle
>> - Review comments
>> - don't emit constructor for first header + inline FIB modifiers
>> - remove extra changes
>> - add newSourceFile factory method
>> - use isNested() predicate
>> - fix header comment indentation
>> - reduce inheritance
>> - SourceFileBuilder
>
> src/main/java/org/openjdk/jextract/impl/ClassSourceBuilder.java line 63:
>
>> 61: this.className = className;
>> 62: this.superName = superName;
>> 63: this.enclosing = enclosing;
>
> should we check that the enclosing class has the same source file builder as this one?
Note really important from the `ClassSourceBuilder` perspective. Having builders with different underlying `SourceFileBuilders` would still work AFAICS, the sources would just be emitted to different source files.
-------------
PR Review Comment: https://git.openjdk.org/jextract/pull/141#discussion_r1406199002
More information about the jextract-dev
mailing list