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

Nizar Benalla nbenalla at openjdk.org
Tue Feb 4 13:07:42 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:

  replace hardcoded strings I missed

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

Changes:
  - all: https://git.openjdk.org/jextract/pull/278/files
  - new: https://git.openjdk.org/jextract/pull/278/files/583bcf61..fa41fe8b

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

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 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