+1, the indexOf was supposed to be done on getResourceIds(appResources, type, required) not the copy (which is sorted)
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <https://blog-rmannibucau.rhcloud.com> | Old Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | JavaEE Factory <https://javaeefactory-rmannibucau.rhcloud.com> 2017-07-04 9:38 GMT+02:00 Svetlin Zarev <svetlin.angelov.za...@gmail.com>: > 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 implementation. The > implementation of ArrayList, works directly on the inner array, which > violates the assumption of the current comparator that the arraylist' > sbacking array is not modified in real time. > > I'll create a jira bug shortly and attach a reproducible test case. > > Kind regards, > Svetlin >
