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/

Reply via email to