May 2022 - jextract-dev

RFR: 7903202: enable macOS CI checks
by Denys Makogon 08 Jun '22

08 Jun '22

This adjustment modifies the build config by introducing strategy matrix in order to enable builds from both Ubuntu and macOS. ------------- Commit messages: - renaming a job based on the given feedback - 7903202: enable macOS CI checks Changes: https://git.openjdk.java.net/jextract/pull/45/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=45&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903202 Stats: 34 lines in 1 file changed: 22 ins; 0 del; 12 mod Patch: https://git.openjdk.java.net/jextract/pull/45.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/45/head:pull/45 PR: https://git.openjdk.java.net/jextract/pull/45

4 8

RFR: 7903201: README.md referes old generated code pattern
by Athijegannathan Sundararajan 08 Jun '22

08 Jun '22

backporting applicable parts of the same fix from jdk19. * using latest generated code pattern in README.md * using module-version in --version output ------------- Commit messages: - 7903201: README.md referes old generated code pattern Changes: https://git.openjdk.java.net/jextract/pull/43/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=43&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903201 Stats: 11 lines in 2 files changed: 1 ins; 1 del; 9 mod Patch: https://git.openjdk.java.net/jextract/pull/43.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/43/head:pull/43 PR: https://git.openjdk.java.net/jextract/pull/43

2 2

RFR: Fixes needed for mach5
by Jorn Vernee 31 May '22

31 May '22

This patch adds the wrapper Makefile from the jdk repo to the jextract repo, and moves the existing Makefile contents to `make/Init.gmk` (mirroring jdk). The wrapper will set the `topdir` variable, which is then used in `Init.gmk` to include the common makefile, as well as cd into the right working directory before running the `Build.gmk` and `RunTestsPrebuilt.gmk` makefiles ------------- Commit messages: - fixes needed for mach5 Changes: https://git.openjdk.java.net/jextract/pull/46/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=46&range=00 Stats: 117 lines in 2 files changed: 65 ins; 0 del; 52 mod Patch: https://git.openjdk.java.net/jextract/pull/46.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/46/head:pull/46 PR: https://git.openjdk.java.net/jextract/pull/46

2 5

RFR: 7903201: README.md referes to old API and old generated code pattern
by Athijegannathan Sundararajan 27 May '22

27 May '22

* fixed README.md for new API usage and new generated code pattern * --version uses module version rather than java.version * fixed makefile --module-version to be 19 ------------- Commit messages: - 7903201: README.md referes to old API and old generated code pattern Changes: https://git.openjdk.java.net/jextract/pull/42/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=42&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903201 Stats: 17 lines in 3 files changed: 1 ins; 2 del; 14 mod Patch: https://git.openjdk.java.net/jextract/pull/42.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/42/head:pull/42 PR: https://git.openjdk.java.net/jextract/pull/42

2 3

RFR: 7903198: Refine filtering for enums
by Athijegannathan Sundararajan 25 May '22

25 May '22

backporting the same fix from jdk19 ------------- Commit messages: - 7903198: Refine filtering for enums Changes: https://git.openjdk.java.net/jextract/pull/41/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=41&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903198 Stats: 47 lines in 6 files changed: 24 ins; 5 del; 18 mod Patch: https://git.openjdk.java.net/jextract/pull/41.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/41/head:pull/41 PR: https://git.openjdk.java.net/jextract/pull/41

2 2

RFR: 7903200: sync makefiles to build jmod for jextract
by Athijegannathan Sundararajan 25 May '22

25 May '22

sync makefiles for jmod creation (and update for jdk19). renamed jmod file name ------------- Commit messages: - 7903200: sync makefiles to build jmod for jextract Changes: https://git.openjdk.java.net/jextract/pull/39/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=39&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903200 Stats: 34 lines in 2 files changed: 16 ins; 10 del; 8 mod Patch: https://git.openjdk.java.net/jextract/pull/39.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/39/head:pull/39 PR: https://git.openjdk.java.net/jextract/pull/39

2 2

RFR: 7903200: sync makefiles to build jmod for jextract
by Athijegannathan Sundararajan 25 May '22

25 May '22

Fixing makefiles for jmod creation. Changed the name of the jmod file. ------------- Commit messages: - 7903200: sync makefiles to build jmod for jextract Changes: https://git.openjdk.java.net/jextract/pull/38/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=38&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903200 Stats: 31 lines in 2 files changed: 15 ins; 10 del; 6 mod Patch: https://git.openjdk.java.net/jextract/pull/38.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/38/head:pull/38 PR: https://git.openjdk.java.net/jextract/pull/38

2 2

RFR: 7903198: Refine filtering for enums
by Maurizio Cimadamore 25 May '22

25 May '22

This patch fixes an issue with the logic for filtering constants. Since both enum constants and `#define` macros are represented as `Declaration.Constant` nodes, `OutputFactory` is confused as to whether the constant being generated is a macro or not. This patch adds a new filtering option `--include-enum`; this option can be used to include an enum (and all its constants). `OutputFactory` has been rectified so that: * `IncludeHelper` is also called on the enum (so that filtering happens) * `visitConstant` does not call `IncludingHelper` filtering routine on things that are non toplevel constants In other words, if we get to `visitConstant` and the parent node is an enum - we should not filter: if we're here it means that the parent enum is included in the filtering. ------------- Commit messages: - Initial push Changes: https://git.openjdk.java.net/jextract/pull/36/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=36&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903198 Stats: 46 lines in 6 files changed: 24 ins; 4 del; 18 mod Patch: https://git.openjdk.java.net/jextract/pull/36.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/36/head:pull/36 PR: https://git.openjdk.java.net/jextract/pull/36

2 2

RFR: 7903181: port jextract for foreign-preview changes
by Athijegannathan Sundararajan 20 May '22

20 May '22

7903181: port jextract for foreign-preview changes ------------- Commit messages: - removed trailing whitespace in TestBlas.java, TestLapack.java and GitClone.java - Merge branch 'master' into foreign-preview - 7903197: add a build step to create jmod file for jextract module - fixed compilesource.sh of libffmpeg sample with correct include path - fixed libffmpeg for foreign-preview. - 7903190: jextract runtime helper issues - 7903187: jextract should support --version option - Adding jdk.zipfs to root modules of jextract jdk image so that --enable-preview - simplified libjimage sample. deleted dlfcn extract as it ia not used. - removed unnecessary 'allocator' variable in many places. Using session instead for allocation. - ... and 5 more: https://git.openjdk.java.net/jextract/compare/e9906418...4f0d0b13 Changes: https://git.openjdk.java.net/jextract/pull/35/files Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=35&range=00 Issue: https://bugs.openjdk.java.net/browse/CODETOOLS-7903181 Stats: 2245 lines in 178 files changed: 70 ins; 1409 del; 766 mod Patch: https://git.openjdk.java.net/jextract/pull/35.diff Fetch: git fetch https://git.openjdk.java.net/jextract pull/35/head:pull/35 PR: https://git.openjdk.java.net/jextract/pull/35

2 14

git: openjdk/jextract: 7903181: port jextract for foreign-preview changes
by Maurizio Cimadamore 20 May '22

20 May '22

Changeset: ac00cb56 Author: Athijegannathan Sundararajan <sundar(a)openjdk.org> Committer: Maurizio Cimadamore <mcimadamore(a)openjdk.org> Date: 2022-05-20 15:11:47 +0000 URL: https://git.openjdk.java.net/jextract/commit/ac00cb56bfe4ca4fb4fd3bc5e1d374… 7903181: port jextract for foreign-preview changes Reviewed-by: mcimadamore ! .github/workflows/test.yml ! README.md ! build.gradle ! samples/cblas/TestBlas.java ! samples/cblas/compile_windows.ps1 ! samples/cblas/compilesource.sh ! samples/cblas/run.sh ! samples/cblas/run_windows.ps1 ! samples/dlopen/Dlopen.java ! samples/dlopen/compilesource.sh ! samples/dlopen/run.sh ! samples/go/compilesource.sh ! samples/go/run.sh ! samples/helloworld/compilesource.sh ! samples/helloworld/run.sh ! samples/lapack/TestLapack.java ! samples/lapack/compile_windows.ps1 ! samples/lapack/compilesource.sh ! samples/lapack/run.sh ! samples/lapack/run_windows.ps1 ! samples/libclang/ASTPrinter.java ! samples/libclang/compile_windows.ps1 ! samples/libclang/compilesource.sh ! samples/libclang/run.sh ! samples/libclang/run_windows.ps1 ! samples/libcurl/CurlMain.java ! samples/libcurl/compile_windows.ps1 ! samples/libcurl/compilesource.sh ! samples/libcurl/run.sh ! samples/libcurl/run_windows.ps1 ! samples/libffmpeg/LibffmpegMain.java ! samples/libffmpeg/compile.sh ! samples/libffmpeg/compilesource.sh ! samples/libffmpeg/run.sh ! samples/libgit2/GitClone.java ! samples/libgit2/compile_windows.ps1 ! samples/libgit2/compilesource.sh ! samples/libgit2/run.sh ! samples/libgit2/run_windows.ps1 ! samples/libjimage/JImageFile.java ! samples/libjimage/compile.sh ! samples/libjimage/compilesource.sh ! samples/libjimage/extract.sh + samples/libjimage/org/openjdk/Constants$root.java ! samples/libjimage/org/openjdk/JImageClose_t.java ! samples/libjimage/org/openjdk/JImageFindResource_t.java ! samples/libjimage/org/openjdk/JImageGetResource_t.java ! samples/libjimage/org/openjdk/JImageOpen_t.java ! samples/libjimage/org/openjdk/JImagePackageToModule_t.java ! samples/libjimage/org/openjdk/JImageResourceIterator_t.java ! samples/libjimage/org/openjdk/JImageResourceVisitor_t.java ! samples/libjimage/org/openjdk/RuntimeHelper.java ! samples/libjimage/org/openjdk/constants$0.java ! samples/libjimage/org/openjdk/constants$1.java ! samples/libjimage/org/openjdk/constants$2.java ! samples/libjimage/org/openjdk/constants$3.java ! samples/libjimage/org/openjdk/jimage_h.java - samples/libjimage/org/unix/Dl_info$0.java - samples/libjimage/org/unix/RuntimeHelper.java - samples/libjimage/org/unix/constants$0.java - samples/libjimage/org/unix/constants$1.java - samples/libjimage/org/unix/constants$2.java - samples/libjimage/org/unix/dl_info.java - samples/libjimage/org/unix/dlfcn_h.java ! samples/libjimage/run.sh ! samples/libproc/LibprocMain.java ! samples/libproc/compilesource.sh ! samples/libproc/run.sh ! samples/lp_solve/LpSolveDemo.java ! samples/lp_solve/compilesource.sh ! samples/lp_solve/run.sh ! samples/opengl/Teapot.java ! samples/opengl/compile_windows.ps1 ! samples/opengl/compilesource.sh ! samples/opengl/run.sh ! samples/opengl/run_windows.ps1 ! samples/python3/PythonMain.java + samples/python3/compile_windows.ps1 ! samples/python3/compilesource.sh ! samples/python3/run.sh + samples/python3/run_windows.ps1 ! samples/readline/Readline.java ! samples/readline/compilesource.sh ! samples/readline/run.sh - samples/shared_windows.ps1 ! samples/sqlite/SqliteMain.java ! samples/sqlite/compilesource.sh ! samples/sqlite/run.sh ! samples/tcl/TCLMain.java ! samples/tcl/compilesource.sh ! samples/tcl/run.sh ! samples/tensorflow/TensorflowLoadSavedModel.java ! samples/tensorflow/compile_windows.ps1 ! samples/tensorflow/compilesource.sh ! samples/tensorflow/loadsavedmodel.sh ! samples/tensorflow/run_windows.ps1 ! samples/time/PanamaTime.java ! samples/time/compilesource.sh ! samples/time/run.sh ! src/main/java/module-info.java ! src/main/java/org/openjdk/jextract/Declaration.java ! src/main/java/org/openjdk/jextract/Type.java ! src/main/java/org/openjdk/jextract/clang/Cursor.java ! src/main/java/org/openjdk/jextract/clang/Diagnostic.java ! src/main/java/org/openjdk/jextract/clang/EvalResult.java ! src/main/java/org/openjdk/jextract/clang/Index.java ! src/main/java/org/openjdk/jextract/clang/LibClang.java ! src/main/java/org/openjdk/jextract/clang/PrintingPolicy.java ! src/main/java/org/openjdk/jextract/clang/SourceLocation.java ! src/main/java/org/openjdk/jextract/clang/SourceRange.java ! src/main/java/org/openjdk/jextract/clang/TranslationUnit.java ! src/main/java/org/openjdk/jextract/clang/Type.java ! src/main/java/org/openjdk/jextract/clang/libclang/CXCursorVisitor.java ! src/main/java/org/openjdk/jextract/clang/libclang/CXString.java ! src/main/java/org/openjdk/jextract/clang/libclang/CXToken.java ! src/main/java/org/openjdk/jextract/clang/libclang/CXType.java ! src/main/java/org/openjdk/jextract/clang/libclang/CXUnsavedFile.java ! src/main/java/org/openjdk/jextract/clang/libclang/Constants$root.java ! src/main/java/org/openjdk/jextract/clang/libclang/Index_h.java ! src/main/java/org/openjdk/jextract/clang/libclang/RuntimeHelper.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$0.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$1.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$10.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$11.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$12.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$13.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$14.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$15.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$16.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$17.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$18.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$2.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$3.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$4.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$5.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$6.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$7.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$8.java ! src/main/java/org/openjdk/jextract/clang/libclang/constants$9.java ! src/main/java/org/openjdk/jextract/impl/ClassSourceBuilder.java ! src/main/java/org/openjdk/jextract/impl/ConstantBuilder.java ! src/main/java/org/openjdk/jextract/impl/DeclarationImpl.java ! src/main/java/org/openjdk/jextract/impl/FunctionalInterfaceBuilder.java ! src/main/java/org/openjdk/jextract/impl/FunctionalInterfaceScanner.java ! src/main/java/org/openjdk/jextract/impl/HeaderFileBuilder.java ! src/main/java/org/openjdk/jextract/impl/JavaSourceBuilder.java ! src/main/java/org/openjdk/jextract/impl/OutputFactory.java ! src/main/java/org/openjdk/jextract/impl/PrettyPrinter.java ! src/main/java/org/openjdk/jextract/impl/RecordLayoutComputer.java ! src/main/java/org/openjdk/jextract/impl/StructBuilder.java ! src/main/java/org/openjdk/jextract/impl/StructLayoutComputer.java ! src/main/java/org/openjdk/jextract/impl/ToplevelBuilder.java ! src/main/java/org/openjdk/jextract/impl/TreeMaker.java ! src/main/java/org/openjdk/jextract/impl/TypeImpl.java ! src/main/java/org/openjdk/jextract/impl/TypeMaker.java ! src/main/java/org/openjdk/jextract/impl/UnionLayoutComputer.java ! src/main/java/org/openjdk/jextract/impl/UnsupportedLayouts.java ! src/main/java/org/openjdk/jextract/impl/Utils.java ! src/main/java/org/openjdk/jextract/impl/Writer.java ! src/main/resources/org/openjdk/jextract/impl/resources/C-X.java.template ! src/main/resources/org/openjdk/jextract/impl/resources/RuntimeHelper.java.template ! test/TEST.ROOT ! test/jtreg/generator/funcPointerInvokers/TestFuncPointerInvokers.java ! test/jtreg/generator/test8244412/LibTest8244412Test.java ! test/jtreg/generator/test8244938/Test8244938.java ! test/jtreg/generator/test8244959/Test8244959.java ! test/jtreg/generator/test8245003/Test8245003.java ! test/jtreg/generator/test8246341/LibTest8246341Test.java ! test/jtreg/generator/test8246400/LibTest8246400Test.java ! test/jtreg/generator/test8252016/Test8252016.java ! test/jtreg/generator/test8252121/Test8252121.java ! test/jtreg/generator/test8252465/LibTest8252465Test.java ! test/jtreg/generator/test8253102/LibTest8253102Test.java ! test/jtreg/generator/test8254983/LibTest8254983Test.java ! test/jtreg/generator/test8257892/LibUnsupportedTest.java ! test/jtreg/generator/test8258605/LibTest8258605Test.java ! test/jtreg/generator/test8259473/LibTest8259473Test.java ! test/jtreg/generator/test8261511/Test8261511.java ! test/jtreg/generator/testFunctionPointer/LibFuncPtrTest.java ! test/jtreg/generator/testStruct/LibStructTest.java ! test/lib/JtregJextractSources.java ! test/lib/testlib/JextractToolRunner.java ! test/testng/org/openjdk/jextract/test/toolprovider/ConstantsTest.java ! test/testng/org/openjdk/jextract/test/toolprovider/IncompleteArrayTest.java ! test/testng/org/openjdk/jextract/test/toolprovider/JextractToolProviderTest.java ! test/testng/org/openjdk/jextract/test/toolprovider/RepeatedDeclsTest.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8240811.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8248415.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8249290.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8249300.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8251943.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8260705.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8260717.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8260929.java ! test/testng/org/openjdk/jextract/test/toolprovider/Test8261578.java ! test/testng/org/openjdk/jextract/test/toolprovider/TestClassGeneration.java ! test/testng/org/openjdk/jextract/test/toolprovider/TestNested.java ! test/testng/org/openjdk/jextract/test/toolprovider/UniondeclTest.java

1 0