Re: svn commit: r1658730 - in /openwebbeans/trunk/webbeans-impl/src: main/java/org/apache/webbeans/util/ test/java/org/apache/webbeans/test/interceptors/factory/ test/java/org/apache/webbeans/test/int

2015-02-12 Thread Joseph Bergmark
I need to write a quick unit test to verify, but I think this may still be missing non-public inherited methods. Previously it only used getDeclaredMethods, which would return all all private, protected, default and public methods for the class, but did not contain any inherited methods. If I'm r

Re: svn commit: r1658730 - in /openwebbeans/trunk/webbeans-impl/src: main/java/org/apache/webbeans/util/ test/java/org/apache/webbeans/test/interceptors/factory/ test/java/org/apache/webbeans/test/int

2015-02-12 Thread Romain Manni-Bucau
sure it should but shouldn't we use annotated type methods instead of pure reflection? Romain Manni-Bucau @rmannibucau http://www.tomitribe.com http://rmannibucau.wordpress.com https://github.com/rmannibucau 2015-02-12 14:41 GMT+01:00 Joseph Bergmark : > I need to write a quick unit test to ver

Re: svn commit: r1658730 - in /openwebbeans/trunk/webbeans-impl/src: main/java/org/apache/webbeans/util/ test/java/org/apache/webbeans/test/interceptors/factory/ test/java/org/apache/webbeans/test/int

2015-02-12 Thread Mark Struberg
I think we originally DID use AnnotatedType. We surely did provide proxies for methods in superclasses. Would need to look at the history if/when we lost this. LieGrue, strub > On Thursday, 12 February 2015, 14:48, Romain Manni-Bucau > wrote: > > sure it should but shouldn't we use annotat