> 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/
