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

Jorn Vernee jvernee at openjdk.org
Fri Sep 19 15:09:52 UTC 2025


On Fri, 19 Sep 2025 14:06:16 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

The latest version passes all tests on my Windows machine.

Could you please update the README at https://github.com/openjdk/jextract/tree/master/updateclang to reflect the changes that were needed to `CXUnsavedFile`? You can include the patch that I sent in that file.

Other than that, this looks ready to me.

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

PR Comment: https://git.openjdk.org/jextract/pull/289#issuecomment-3312571016


More information about the jextract-dev mailing list