RFR: 8307091: A few client tests intermittently throw ConcurrentModificationException [v2]

[Tejesh R]

On Thu, 4 May 2023 18:20:58 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Tejesh R has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Updated based on review comments
>
> src/java.desktop/share/classes/javax/swing/plaf/basic/BasicDirectoryModel.java line 365:
> 
>> 363: 
>> 364:                         if (start >= 0 && end > start) {
>> 365:                             List<File> listStart_OldSize = new Vector<>(fileCache.subList(start, oldSize));
> 
> Why Vector was used instead of ArrayList ?

Nothing specific reason why I choose vector apart from the main object is of Vector type and since in this context (where both the objects are not shared) both Vector/Arraylist comparison remains same. Does Arraylist comparison has any advantage here?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13783#discussion_r1185710608