RFR: 7903933: Move sharable items from different generations to a common file [v7]

Nizar Benalla nbenalla at openjdk.org
Fri May 2 14:30:36 UTC 2025


> 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:

  feedback: rename option and consolidate code

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

Changes:
  - all: https://git.openjdk.org/jextract/pull/278/files
  - new: https://git.openjdk.org/jextract/pull/278/files/532ee1a6..5c8802a6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jextract&pr=278&range=06
 - incr: https://webrevs.openjdk.org/?repo=jextract&pr=278&range=05-06

  Stats: 83 lines in 5 files changed: 44 ins; 16 del; 23 mod
  Patch: https://git.openjdk.org/jextract/pull/278.diff
  Fetch: git fetch https://git.openjdk.org/jextract.git pull/278/head:pull/278

PR: https://git.openjdk.org/jextract/pull/278


More information about the jextract-dev mailing list