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 _______________________________________________ weld-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/weld-dev
