RFR: 8050091: LinkedList has incorrect implementation comment

Ivan Gerasimov ivan.gerasimov at oracle.com
Sat Jun 27 12:46:51 UTC 2015


Hi Martin!

Thank you for this cleanup!
Removal of wrong comments looks fine.

But your webrev contains commented checkInvariants() method.
Is it a leftover from debugging or something?

Sincerely yours,
Ivan

On 26.06.2015 23:04, Martin Buchholz wrote:
> Hi Ivan, I'd like you to do a code review.
>
> https://bugs.openjdk.java.net/browse/JDK-8050091
> http://cr.openjdk.java.net/~martin/webrevs/openjdk9/LinkedList-invariant/ 
> <http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk9/LinkedList-invariant/>




More information about the core-libs-dev mailing list