Re: [Jikesrvm-commits] [Jikesrvm-core] SF.net SVN: jikesrvm: [12046] rvmroot/trunk

[Steve B. <Ste...@an...>]

Hi Peter,

A secondary comment...

One of the things you removed (write(int value, boolean hexToo)) and  
one of the things you made private (write(VM_Atom value)) are, in my  
experience, primarily used for debugging.

Non-existence in the current code base is generally a good reason to  
remove something, but debugging hooks in general may not necessarily  
be in use except when someone is actually debugging...  I'm not  
justifying this state of affairs or suggesting this is good practice,  
but just pointing out that this is current reality, and so urge you  
not to be too gung-ho about deleting stuff that appears to be unused  
but which may have a reasonable debugging role.

Some of us seem to spend a frighteningly large fraction of our  
"research" time debugging ;-)

...and to be clear: I for one really appreciate the enormous amount  
of cleanup work you've been doing.

Cheers,

--steve