On Mon, Jun 29, 2015 at 8:04 AM, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote:
> > > On 27.06.2015 21:54, Martin Buchholz wrote: > > > > On Sat, Jun 27, 2015 at 5:46 AM, Ivan Gerasimov <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 :) > Well, as something that lives in the nether world between code and comment, it *is* commented out code. I moved in your direction by renaming the not-quite-a-method to dataStructureInvariants.
