On Wed, 19 Apr 2023 02:43:53 GMT, Stuart Marks <[email protected]> wrote:
>> src/java.base/share/classes/java/util/LinkedHashSet.java line 297:
>>
>>> 295: */
>>> 296: public SequencedSet<E> reversed() {
>>> 297: class ReverseLinkedHashSetView extends AbstractSet<E>
>>> implements SequencedSet<E> {
>>
>> This class should be declared `static` (and private) which means it should
>> not be declared inside reversed.
>
> Can't be static because it uses type variable `E` and also refers to `this`.
It can still be a `private` inner class, as opposed to a method class.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/7387#discussion_r1174349916
