You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This commit was created on GitHub.com and signed with GitHub’s verified signature.
The key has expired.
Changed
The source file djinni/jni/djinni_jni_main.cpp is no longer included if building as static library, because it is only really required when building as shared library. This doesn't change anything for the end user. (#50)
The C wrapper code does not refer to "Python" in it's names any more. The long term goal is to make the C wrapper a general purpose interface that can be used for bridging to other languages as well.
Removed
the paths djinni/<lang>/ are no longer include directories in the CMake target. They were provided to maintain compatibility with the original dropbox Djinni generator and are not needed any more with the cross-language-cpp generator. (#53)
Fixed
The JNI build on Android did not work with the (no so experimental anymore) EXPERIMENTAL_AUTO_CPP_THREAD_ATTACH flag enabled (cmake option JNI_CPP_THREAD_ATTACH=ON) . This is now resolved and JNI compiles for both Android NDK and OpenJDK if you really want multithread support enabled. (#46)
With newer NDKs and Android release, an assert on a JNI pointer stared causing problems under some circumstances. The assert has been adopted to make less assumptions on the internals of the Android target platform. (#47,#30)