jpy-consortium / jpy

Apache License 2.0
71 stars 17 forks source link

Use Py_ssize_t when calculate buffer len #145

Closed jmao-denver closed 5 months ago

jmao-denver commented 5 months ago

Also fixed all the pointer to long conversions that clang 15 considers error by default on MacOS M2

Fixes #143

jmao-denver commented 5 months ago

I think there's an issue associated with this PR?

Yes, 5403, I manually tested it and confirmed the fix. But I had to increase the max JVM heap size to 8G, I am not sure if we want to include the test in the CI.