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