RFR: 7904079: Fix setting up length parameter for CXUnsavedFiles Struct [v8]

Jorn Vernee jvernee at openjdk.org
Mon Sep 22 15:53:01 UTC 2025


On Mon, 22 Sep 2025 12:46:03 GMT, Varada M <varadam at openjdk.org> wrote:

>> CXUnsavedFile struct contains three members, first two are pointers and last one is unsigned long. When jextract makes downcall to clang_reparseTranslationUnit_Impl it sets up the third parameter in the struct incorrectly. It sets INT instead of LONG so on big endian platform the parameter will be set differently. 
>> 
>> JBS Issue : [CODETOOLS-7904079](https://bugs.openjdk.org/browse/CODETOOLS-7904079)
>
> Varada M has updated the pull request incrementally with one additional commit since the last revision:
> 
>   7904079: Fix setting up length parameter for CXUnsavedFiles Struct

Thanks for the updates! All tests pass here as well. This looks ready.

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

Marked as reviewed by jvernee (Reviewer).

PR Review: https://git.openjdk.org/jextract/pull/289#pullrequestreview-3253679369


More information about the jextract-dev mailing list