Looking more into this it seems that AbstractCallbackMetaData.describeVisit() and AbstractInjectionMetaData.describeVisit() both need access to MetaDataVisitor.getControllerContext(), so we'll get NPEs unless there is a hack. Can we not just add | void MetaDataVisitor.structureVisit(MetaDataVisitorNode node); | void MetaDataVisitorNode.structureVisit(MetaDataVisitor visitor); | where the MetaDataVisitorNode.structureVisit() impls don't ever do anything clever, just delegate to the visitor | void structureVisit(MetaDataVisitor visitor) | { | visitor.structureVisit(this); | } | Would that not be more in line with a "normal" visitor implementation? View the original post : http://www.jboss.org/index.html?module=bb&op=viewtopic&p=4209038#... Reply to the post : http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&a...