[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: MutableClassInfo?

[stale.pedersen@jboss.org]

"alesj" wrote : org.jboss.reflect.spi shouldn't contain Javassist specific classes. 
  | 
  | Make it spi.javassist or something similar. 
  | e.g. we sometimes use helpers pckg 
i moved the Body implementations to org.jboss.reflect.plugins.javassist. i was kinda uncertain about where they should be placed since they didnt expose any javassist classes and they are needed to use the mutable classes.
- but the user need to specify javassist specific implementations when he want to create a mutable object, so i guess it fits in the javassist package too.
"alesj" wrote : I guess this is gonna be log.trace or removed? 
its already removed, totally forgot to remove them.
"alesj" wrote : 
  |   | public class DummyClassPoolFactory implements ClassPoolFactory
  |   | {
  |   | 
  |   |    public ClassPool getPoolForLoader(ClassLoader cl)
  |   |    {
  |   |       return ClassPool.getDefault();
  |   |    }
  |   | 
  |   | }
  |   | 
  | Why Dummy if it's default?
i made those in the beginning hoping that kabir would finish his classloading stuff before i added this to trunk. when it didnt happen i kept the dummy just to show that this is something that will be changed later. this isnt something that is exposed to the user, so i felt it was ok to just leave it.
- its the same code that was used earlier though, so it works like before.

View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4218516#4218516

Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=4218516