Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dispose on unload #378

Merged
merged 2 commits into from Oct 31, 2014
Merged

Dispose on unload #378

merged 2 commits into from Oct 31, 2014

Conversation

twall
Copy link
Contributor

@twall twall commented Oct 31, 2014

The issue addressed has to do with situations where the JVM is apparently unloading JNA’s native library prior to running Java object finalizers.

  • Invoke Native.dispose() when JNA's native library is unloaded.
  • Added disposal of all Memory, CallbackReference, and registered Direct mappings

@dblock
Copy link
Member

dblock commented Oct 31, 2014

Looks intense, I'll merge it.

dblock added a commit that referenced this pull request Oct 31, 2014
@dblock dblock merged commit fc454ce into master Oct 31, 2014
@twall
Copy link
Contributor Author

twall commented Oct 31, 2014

Need to fix a few unit tests around GC…

On Oct 31, 2014, at 11:11 AM, Daniel Doubrovkine (dB.) @dblockdotorg notifications@github.com wrote:

Looks intense, I'll merge it.


Reply to this email directly or view it on GitHub.

@dblock
Copy link
Member

dblock commented Nov 1, 2014

I thought those were unrelated failures, but I was wrong :(

@twall
Copy link
Contributor Author

twall commented Nov 1, 2014

NP, they’re fixed now. Need to do another release soon, need to find some time to redo some native builds.

On Nov 1, 2014, at 9:49 AM, Daniel Doubrovkine (dB.) @dblockdotorg notifications@github.com wrote:

I thought those were unrelated failures, but I was wrong :(


Reply to this email directly or view it on GitHub.

kiefermat added a commit to 1and1/rados-java that referenced this pull request Apr 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants