Re: AutoConfig - IllegalArgumentException: Comparison method violates its general contract!

2017-07-04 Thread Svetlin Zarev
Created TomEE-2085 and https://github.com/apache/tomee/pull/84 What do you think ? 2017-07-04 10:41 GMT+03:00 Romain Manni-Bucau : > +1, the indexOf was supposed to be done on getResourceIds(appResources, > type, required) not the copy (which is sorted) > > > Romain Manni-Bucau > @rmannibucau

Re: AutoConfig - IllegalArgumentException: Comparison method violates its general contract!

2017-07-04 Thread Romain Manni-Bucau
+1, the indexOf was supposed to be done on getResourceIds(appResources, type, required) not the copy (which is sorted) Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github

AutoConfig - IllegalArgumentException: Comparison method violates its general contract!

2017-07-04 Thread Svetlin Zarev
Hi, I found a nasty bug in AutoConfig:2077. The comparator does not work correctly with java 8. In older java versions (older than 8), Collections.sort() always creates an array from the list content, while starting with java 8 -> it delegates to the sort() method implemented in the concrete list