On Thu, 22 Apr 2021 04:19:50 GMT, Ian Graves <igra...@openjdk.org> wrote:
>> 8261168: Convert javadoc tool to use Stream.toList() > > Ian Graves has updated the pull request incrementally with one additional > commit since the last revision: > > Changing Collectors.toList() to an explicit ArrayList collection src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util/VisibleMemberTable.java line 461: > 459: .collect(Collectors.toList()); > 460: > 461: // Prefix local results first 1. Could you incorporate this comment instead of deleting it? 2. Consider refactoring L502:L516 in a similar fashion, i.e. `a.addAll(0, b)` -> `Stream.concat(b, a)`. ------------- PR: https://git.openjdk.java.net/jdk/pull/3612
