On Thu, 24 Sep 2020 18:23:54 GMT, Ambarish Rapte <ara...@openjdk.org> wrote:
>> Leon Linhart has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Removed unused import and addressed review comments > > modules/javafx.base/src/main/java/javafx/collections/ModifiableObservableListBase.java > line 32: > >> 30: import java.util.List; >> 31: import java.util.ListIterator; >> 32: import java.util.Objects; > > This import seems unused, please remove. Indeed. Removed it. > modules/javafx.base/src/main/java/javafx/collections/ModifiableObservableListBase.java > line 97: > >> 95: clear(); >> 96: addAll(col); >> 97: return true; > > I think following code would be more suitable here, > boolean res = super.addAll(c); > return res; > This code is already used in two `addAll()` methods of this class. Makes sense to me. I changed it accordingly. ------------- PR: https://git.openjdk.java.net/jfx/pull/284