September 2022 - jextract-dev

RFR: 7903239: ofAddress factory of function pointer type is wrong for struct returns
by Marko 02 Jan '24

02 Jan '24

When generating the lambda inside the `ofAddress` factory, it is missing the `SegmentAllocator` needed when the function returns a struct. This is [implemented for regular functions](https://github.com/openjdk/jextract/blob/0582eaf1b4cdba95f0ee8c2…, but not for function pointers. ------------- Commit messages: - 7903239: ofAddress factory of function pointer type is wrong for struct returns Changes: https://git.openjdk.org/jextract/pull/58/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=58&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903239 Stats: 47 lines in 3 files changed: 44 ins; 2 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/58.diff Fetch: git fetch https://git.openjdk.org/jextract pull/58/head:pull/58 PR: https://git.openjdk.org/jextract/pull/58

3 11

RFR: Add path independence
by Per Minborg 02 Jan '24

02 Jan '24

This small PR incorporated feedback from the original PR: https://github.com/openjdk/jextract/pull/66 made after it was integrated. ------------- Commit messages: - Add path independence Changes: https://git.openjdk.org/jextract/pull/68/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=68&range=00 Stats: 10 lines in 4 files changed: 2 ins; 0 del; 8 mod Patch: https://git.openjdk.org/jextract/pull/68.diff Fetch: git fetch https://git.openjdk.org/jextract pull/68/head:pull/68 PR: https://git.openjdk.org/jextract/pull/68

5 7

RFR: Add Dockerfile, zero-dep reproducible builds
by Gavin Ray 02 Jan '24

02 Jan '24

As title says, adds a Dockerfile that will fetch all necessary deps and build `jextract` for you on Ubuntu 22.10 It would be nice if some official JDK organization could put this on their Dockerhub, so that users could just do: $ docker run --rm -it openjdk/jextract:latest I've pushed it to my personal Dockerhub, so if anyone would like to test it out, you can use the README directions but prepend `gavinray/` before the image name: $ echo "struct Foo { int a; long b; }; int use_foo(struct Foo foo);" >> sample.h $ docker run --rm -it -v ${PWD}:/tmp/jextract \ gavinray/jextract:latest \ -I /tmp/jextract \ --source \ -l libfoo \ --output /tmp/jextract/generated \ --target-package com.example \ ------------- Commit messages: - Add Dockerfile, zero-dep reproducible builds Changes: https://git.openjdk.org/jextract/pull/74/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=74&range=00 Stats: 75 lines in 2 files changed: 75 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jextract/pull/74.diff Fetch: git fetch https://git.openjdk.org/jextract pull/74/head:pull/74 PR: https://git.openjdk.org/jextract/pull/74

3 6

RFR: Fix typo in README.md
by Per Minborg 17 Oct '22

17 Oct '22

This PR removes a redundant `=` character. ------------- Commit messages: - Fix typo in README.md Changes: https://git.openjdk.org/jextract/pull/78/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=78&range=00 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/78.diff Fetch: git fetch https://git.openjdk.org/jextract pull/78/head:pull/78 PR: https://git.openjdk.org/jextract/pull/78

2 2

RFR: 7903339: unsigned typedefs are not extracted
by Athijegannathan Sundararajan 03 Oct '22

03 Oct '22

unsigned, signed primitive types were not handled in OutputFactory.visitTypedef. ------------- Commit messages: - 7903339: unsigned typedefs are not extracted Changes: https://git.openjdk.org/jextract/pull/80/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=80&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903339 Stats: 107 lines in 3 files changed: 106 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/80.diff Fetch: git fetch https://git.openjdk.org/jextract pull/80/head:pull/80 PR: https://git.openjdk.org/jextract/pull/80

2 2

RFR: 7903335: jextract test TestSplit.java fails
by Athijegannathan Sundararajan 27 Sep '22

27 Sep '22

making class final and constructors private breaks header split classes. Turning off these for split header classes. ------------- Commit messages: - 7903335: jextract test TestSplit.java fails Changes: https://git.openjdk.org/jextract/pull/79/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=79&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903335 Stats: 12 lines in 1 file changed: 5 ins; 5 del; 2 mod Patch: https://git.openjdk.org/jextract/pull/79.diff Fetch: git fetch https://git.openjdk.org/jextract pull/79/head:pull/79 PR: https://git.openjdk.org/jextract/pull/79

2 2

RFR: added a note on JDK version to use for Gradle build.
by Athijegannathan Sundararajan 23 Sep '22

23 Sep '22

Same change from 'master' to 'panama' branch ------------- Commit messages: - added a note on JDK version to use for Gradle build. Changes: https://git.openjdk.org/jextract/pull/77/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=77&range=00 Stats: 6 lines in 1 file changed: 5 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/77.diff Fetch: git fetch https://git.openjdk.org/jextract pull/77/head:pull/77 PR: https://git.openjdk.org/jextract/pull/77

2 2

RFR: added a note on JDK version to use for Gradle build.
by Athijegannathan Sundararajan 22 Sep '22

22 Sep '22

Added missing note to README.md on JDK version required for Gradle version. ------------- Commit messages: - spelling - added a note on JDK version to use for Gradle build. Changes: https://git.openjdk.org/jextract/pull/75/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=75&range=00 Stats: 7 lines in 1 file changed: 6 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/75.diff Fetch: git fetch https://git.openjdk.org/jextract pull/75/head:pull/75 PR: https://git.openjdk.org/jextract/pull/75

2 4

RFR: Fix formatting in sample/time
by Per Minborg 20 Sep '22

20 Sep '22

This PR formats the time so that single-digit minutes will look better. ------------- Commit messages: - Fix formatting in sample/time Changes: https://git.openjdk.org/jextract/pull/73/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=73&range=00 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/73.diff Fetch: git fetch https://git.openjdk.org/jextract pull/73/head:pull/73 PR: https://git.openjdk.org/jextract/pull/73

1 1

RFR: 7903272: Make method handle methods private
by Per Minborg 20 Sep '22

20 Sep '22

xxxx$MH methods should be declared private to reduce cluttering in end users' IDEs and elsewhere. This PR also solves esthetic issues in the generated code. ------------- Commit messages: - Make method handle methods private Changes: https://git.openjdk.org/jextract/pull/67/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=67&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903272 Stats: 3 lines in 1 file changed: 0 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jextract/pull/67.diff Fetch: git fetch https://git.openjdk.org/jextract pull/67/head:pull/67 PR: https://git.openjdk.org/jextract/pull/67

2 5