Fixing this encompasses expanding the [SPI class|https://github.com/weld/api/blob/master/weld-spi/src/main/java/org/jboss/weld/resources/spi/ClassFileInfo.java] but also the implementations - both in [Weld|https://github.com/weld/core/blob/master/environments/common/src/main/java/org/jboss/weld/environment/deployment/discovery/jandex/JandexClassFileInfo.java] itself and in [WFLY|https://github.com/wildfly/wildfly/blob/master/weld/subsystem/src/main/java/org/jboss/as/weld/discovery/WeldClassFileInfo.java].
To stay consistent, we should also behave the same when not using Jandex, which means changing the behaviour in [{{AnnotatedTypeLoader}}|https://github.com/weld/core/blob/master/impl/src/main/java/org/jboss/weld/bootstrap/AnnotatedTypeLoader.java#L90].
Though I am not sure why we need those two separate Jandex implementations in the first place? They seem almost identical with only minor details which may have originated from doing the changes separately. Might be worth introducing a common abstract class in SPI . [~mkouba] maybe you have some memory of this ? |
|
