[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: UnwrapValueUnitTestCase.testCollectionUnwrap failure on

[adrian@jboss.org]

"alesj" wrote : "adrian at jboss.org" wrote : 
  |   | Clearly, the caching in AbstractBeanInfoFactory shouldn't be using
  |   | classInfo.getName() in its caching.
  |   | 
  | What about if I just change that the key is ClassInfo instead of String?
  | (I've done that locally and all the tests pass)

But it wouldn't be much of a cache since the ClassInfo's can be pretty transient.

Also I think it will cause a memory leak.

You effectively have:
WeakValueHashMap<ClassInfo, BeanInfo>
but the ClassInfo could itself get a hard reference on the BeanInfo
via one of its "attachments", e.g. the caching of SchemaBindings in jbossxb
against the TypeInfo.

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

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