Yes, it's now as simple as it can/should be. Also, it's pretty similar
to what Jozef did in 2.0.
ML
On 6.8.2012 12:03, Ales Justin wrote:
Is this OK now?
I mean, are you happy with the fix. :-)
> You can merge this Pull Request by running:
>
> git pull
https://github.com/luksa/weld-core weld1144
>
> Or you can view, comment on it, or merge it online at:
>
>
https://github.com/weld/core/pull/218
>
> -- Commit Summary --
>
> * Remove unnecessary code
> * Extract methods from WeldClassImpl constructor to make it readable
> * WELD-1144 Superclass of processed AnnotationType is ignored
>
> -- File Changes --
>
> M impl/src/main/java/org/jboss/weld/bean/AbstractClassBean.java (2)
> M impl/src/main/java/org/jboss/weld/introspector/ForwardingWeldClass.java (4)
> M impl/src/main/java/org/jboss/weld/introspector/WeldClass.java (2)
> M impl/src/main/java/org/jboss/weld/introspector/jlr/WeldClassImpl.java (256)
> M impl/src/main/java/org/jboss/weld/manager/SimpleInjectionTarget.java (2)
> M impl/src/main/java/org/jboss/weld/util/Beans.java (167)
> M pom.xml (2)
> M
tests-arquillian/src/test/java/org/jboss/weld/tests/annotatedType/weld1144/CdiTest1.java
(1)
> M
tests-arquillian/src/test/java/org/jboss/weld/tests/annotatedType/weld1144/CdiTest2.java
(1)
> M
tests-arquillian/src/test/java/org/jboss/weld/tests/annotatedType/weld1144/SomeInjectedBean.java
(1)
> M
tests-arquillian/src/test/java/org/jboss/weld/tests/annotatedType/weld1144/Weld1144Test.java
(6)
>
> -- Patch Links --
>
>
https://github.com/weld/core/pull/218.patch
>
https://github.com/weld/core/pull/218.diff
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/weld/core/pull/218