"adrian(a)jboss.org" wrote : It looks redundant to me, but it does no harm. ;-) | So you're saying this is the same - for AbstractMetaDataLoader: | if (getScope().getScopeLevel(level) != null) | return new AbstractMetaDataContext(this); | and | if (getScope().getScopeLevel(level) != null) | return this; | OK, no real harm, but the second one leaves AbstractMetaDataLoader clean from importing un-spi, un-retrieval packages. :-) View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4105271#... Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&a...