The biggest annoyance is the way @Override is handled. They made the mistake of disallowing it on Interfaces in Java 5. This is fixed in Java 6. But, of course, this is just a nit.
On Mar 14, 2013, at 12:33 PM, Simone Tripodi <simonetrip...@apache.org> wrote: >> Yeah - the big problem is that this generates a warning: >> >> Arrays.asList( WarmUp.class, PostConstruct.class ) >> > > personally I am not so worried about that warnings, they just make > some noise - feel free to include them or not. > > I'd suggest anyway to support Iterable<E> rather than just List<E> for > accepting annotation types, like Guice.createInjector()[1] does > >> I'm glad that this is fixed in Java 7. BTW - why are we targeting Java 5? >> Can we at least move to Java 6? > > do you need some JVM feature that require JDK6? if yes - please > specify the reasons - you can override the ${javac.src.version} and > ${javac.target.version} properties in the component pom. > if not, ... which is the sense to upgrade to 6? :) having APIs still > be backward compatible with JDK5 is still fine since Guice targets it > - and immagine I still have > customers which run their application on JDK1.4! :D > > best, > -Simo > > [1] > http://google-guice.googlecode.com/git/javadoc/com/google/inject/Guice.html#createInjector(java.lang.Iterable<? > extends com.google.inject.Module>) > > http://people.apache.org/~simonetripodi/ > http://simonetripodi.livejournal.com/ > http://twitter.com/simonetripodi > http://www.99soft.org/