"adrian(a)jboss.org" wrote : It should be done on the calls to (un)getTarget() | | | | // Mixin interface for service tracking | | public interface ServiceTracking | | { | | ... | | | | if (me instanceOf ServiceTracking) | | { | | ServiceTracker otherTracker = me.getServiceTracker(); | | if (otherTracker != null) | | otherTracker.incrementUsing(this); | | } | | } | | return result; | | } | | | | The instanceOf ServiceTracking is for backwards compatibilty in case | some old ControllerContext doesn't know about ServiceTracking. | I don't get this "instance of"? Wouldn't ControllerContext implement this ServiceTracking mixin? OK, I guess since you called it mixin, it should be just AbstractControllerContext that impls it. But this makes lookup code ugly ... checking every context in metadata if it's ServiceTracking interface. Oh well, some copy pasting won't kill me ... :-) View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4267033#... Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&a...