[ https://issues.jboss.org/browse/ROASTER-44?page=com.atlassian.jira.plugin... ] George Gastaldi commented on ROASTER-44: ---------------------------------------- My only concern with addAnnotationValue(Class<?>) is to add the import. I am not 100% sure that setName() should be the overloaded option. This is my implementation of the addAnnotationValue(Class<?>): {code} @Override public AnnotationSource<O> addAnnotationValue(Class<? extends java.lang.annotation.Annotation> type) { if (!getOrigin().hasImport(type)) { getOrigin().addImport(type); } return addAnnotationValue().setName(type.getSimpleName()); } @Override public AnnotationSource<O> addAnnotationValue(String name, Class<? extends java.lang.annotation.Annotation> type) { if (!getOrigin().hasImport(type)) { getOrigin().addImport(type); } return addAnnotationValue(name).setName(type.getSimpleName()); } {code} -- This message was sent by Atlassian JIRA (v6.3.8#6338)