JuliaInterop / JavaCall.jl

Call Java from Julia
http://juliainterop.github.io/JavaCall.jl
Other
118 stars 53 forks source link

WIP: Assert root Task for jcall and jfield #107

Closed mkitti closed 4 years ago

mkitti commented 4 years ago

Work in progress on expanding assertions throughout the API. I targeted exported functions which may return incorrect results or strange errors such as "java.lang.System is not found".

aviks commented 4 years ago

Eventually everything goes thru the two _jcall methods, I think.

mkitti commented 4 years ago

JavaCall might be working better on Windows than Linux. Before proceeding further, we should expand the tests further to see what is working under @async

mkitti commented 4 years ago

JULIA_COPY_STACKS does not seem to work well on Windows and JavaCall seems to work well without it. https://travis-ci.org/mkitti/JavaCall.jl/builds/654273171

It may be necessary to qualify the use of JULIA_COPY_STACKS for Linux and Mac only.

aviks commented 4 years ago

Did you mean to close this?

mkitti commented 4 years ago

Yes, I think the assertions may prevent operation on Windows, so I am expanding testing. I will probably rename the assertion to something more general before resubmitting a PR. Current progress: https://github.com/mkitti/JavaCall.jl/tree/test_async Testing: https://travis-ci.org/mkitti/JavaCall.jl/builds/654326201