On Thu, 22 Apr 2021 09:16:40 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

>> Ian Graves has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Tweaking some comments and list/stream concatenation
>
> 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)`.

Done!

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

PR: https://git.openjdk.java.net/jdk/pull/3612

Reply via email to