xiaodududu / google-guice

Automatically exported from code.google.com/p/google-guice
Apache License 2.0
0 stars 0 forks source link

Promote class loader code to cglib? #217

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
We do a lot of legwork to decide which classloader to load generated classes 
in. None of this code 
is particularly related to dependency-injection. But it's just necessary to 
solve imposed by 
garbage collection and OSGi.

It would be nice if we contributed our BytecodeGen code to the cglib project, 
so that other 
libraries that use cglib wouldn't have to reinvent this (complicated) code. It 
would also be nice to 
simplify Guice!

The code to contribute:
  http://code.google.com/p/google-
guice/source/browse/trunk/src/com/google/inject/internal/BytecodeGen.java

And documentation (much of which is repeated in the Javadoc):
  http://code.google.com/p/google-guice/wiki/BytecodeGeneration

Chris, what do you think? Does it fit?

Original issue reported on code.google.com by limpbizkit on 3 Jul 2008 at 11:49

GoogleCodeExporter commented 9 years ago
Ping?

Original comment by cgruber@google.com on 16 Jul 2012 at 6:21

GoogleCodeExporter commented 9 years ago
I don't think there's any way this will happen in practice, given who needs to 
work on it and what they have going on. 

Original comment by cgruber@google.com on 18 Nov 2013 at 8:08