RFR: 8050091: LinkedList has incorrect implementation comment

Ivan Gerasimov ivan.gerasimov at oracle.com
Mon Jun 29 15:04:25 UTC 2015



On 27.06.2015 21:54, Martin Buchholz wrote:
>
>
> On Sat, Jun 27, 2015 at 5:46 AM, Ivan Gerasimov 
> <ivan.gerasimov at oracle.com <mailto:ivan.gerasimov at oracle.com>> wrote:
>
>     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?
>
>
> It could be used for debugging, but it's primarily the (corrected!) 
> documentation of the invariants of the fields, expressed in executable 
> code.

Ah, alright, thanks,

A short comment would help avoid confusion, as currently it does look 
just like a commented out code :)

Sincerely yours,
Ivan




More information about the core-libs-dev mailing list