On 15 July 2013 23:26, <aherit...@apache.org> wrote: > Code cleanup - Maven requires Java 5+ : Remove unnecessary boxing
Not sure that's a good idea. I've found quite a few bugs related to boxing in other projects. For example, auto-unboxing a field that can sometimes be null may cause an unexpected NPE; it's not always obvious what is causing the NPE if it's implicit. Or code that is using Integer when it should be using int - or vice versa. If you remove all the explicit boxing there are usually too many compiler warnings to wade through. And using @SuppressWarning where the auto-boxing is justified is more work than keeping the explicit boxing/unboxing. --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org
