RFR: 7903933: Move sharable items from different generations to a common file [v6]
Maurizio Cimadamore
mcimadamore at openjdk.org
Mon Apr 28 15:55:58 UTC 2025
On Fri, 18 Apr 2025 15:35:08 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:
>> Please review this patch to move the `C_*` layouts and the static utility methods into separate classes: `LayoutUtils.java` and `FFMUtils.java`, respectively.
>>
>> - The names could later be personalized through a JSON configuration.
>> - We can use static imports if the `-t` option is no used and the files are generated into the default package, in that case we use the classname to call the static methods or use the `C_*` constants.
>>
>> Some tests had to be modified slightly, either by adding new static imports or replacing classnames.
>
> Nizar Benalla has updated the pull request incrementally with one additional commit since the last revision:
>
> make utility classes final with a private constructor
The option for changing header class name is called `--header-class-name <name>`. So I'd suggest something like `--symbols-class-name`.
-------------
PR Comment: https://git.openjdk.org/jextract/pull/278#issuecomment-2835721076
More information about the jextract-dev
mailing list