duchien85 / kryo

Automatically exported from code.google.com/p/kryo
BSD 3-Clause "New" or "Revised" License
0 stars 0 forks source link

Weak References to Class files #17

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This is a very preliminary rough patch.  I have some other ideas that I'd
also like to experiment, I've posted it here so others may experiment with
it also.

New BSD License as per Kryo.

This builds and all the tests pass, note that this patch is set to weak
references in Kryo's constructor by default.

You've got some performance testing code also I believe?

Cheers,

Peter.

Original issue reported on code.google.com by pfirm...@bigpond.net.au on 13 Apr 2010 at 10:42

Attachments:

GoogleCodeExporter commented 9 years ago
This is an alternative implementation that adds a WeakReference to 
RegisteredClass
and stores additional class information in RegisteredClass, such as the Package
Implementation version.

See patch for discussion comments.

All tests are passing.  Need a test that confirms garbage collection.

This patch is against svn r102, also under new BSD License.

Original comment by pfirm...@bigpond.net.au on 14 Apr 2010 at 3:07

Attachments:

GoogleCodeExporter commented 9 years ago
Still not sure on this, but will leave it for later.

Original comment by nathan.s...@gmail.com on 10 Oct 2010 at 2:52

GoogleCodeExporter commented 9 years ago
It has been some time and no one else has requested this feature. Possibly it 
could be more easily solved by explicit deregistration? Closing the issue 
unless someone can come up with a use case and a test showing when the current 
mechanism prevents class unloading.

Original comment by nathan.s...@gmail.com on 26 Apr 2012 at 2:32