On 27.06.2015 21:54, Martin Buchholz wrote:


On Sat, Jun 27, 2015 at 5:46 AM, Ivan Gerasimov <ivan.gerasi...@oracle.com <mailto:ivan.gerasi...@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

Reply via email to