[jboss-dev-forums] [Design the new POJO MicroContainer] - Re: Annotation processing

[alesj]

"adrian at jboss.org" wrote : 
  | It's only real use at the moment is in AOP to find out if there are instance annotations.
  | That processing is wrong anyway. There should be a
  | MetaData.hasInstanceMetaData() instead.
  | 

What's the difference between local and non-local retrievers (from MetaDataContext):

  |    /**
  |     * Get the retrievals
  |     * 
  |     * @return the retrievals
  |     */
  |    List<MetaDataRetrieval> getRetrievals();
  | 
  |    /**
  |     * Get the local retrievals
  |     * 
  |     * @return the local retrievals
  |     */
  |    List<MetaDataRetrieval> getLocalRetrievals();
  | 

I guess this needs removal as well (from KernelControllerContextAction):

  |             if (joinpoint instanceof KernelControllerContextAware)
  |                ((KernelControllerContextAware) joinpoint).setKernelControllerContext(context);
  | ...
  |             if (joinpoint instanceof KernelControllerContextAware)
  |                ((KernelControllerContextAware) joinpoint).unsetKernelControllerContext(null);
  | 

Since all information for AOPConstructorJoinpoint should then be in MetaData instance taken from the stack?

Any quick tips on how MetaData.hasInstanceMetaData() should look like?
Should it still use MetaDataContext as current impl in AOPConstructorJoinpoint does?

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

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