RFR: 7903590: Refactor jextract source code generation [v2]

Jorn Vernee jvernee at openjdk.org
Mon Nov 27 13:00:25 UTC 2023


On Mon, 27 Nov 2023 12:00:09 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/JavaSourceBuilder.java line 38:
> 
>> 36: import java.util.Optional;
>> 37: 
>> 38: interface JavaSourceBuilder {
> 
> I suggest to rename that to something like OutputHelper, OutputBuilder or something like that - or even nest this interface inside OutputFactory, to make it clear that the two are related (e.g. OutputFactory.Builder).

Moved it to an inner `OuputFactory.Builder` interface

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

PR Review Comment: https://git.openjdk.org/jextract/pull/141#discussion_r1406120511


More information about the jextract-dev mailing list