Re: [rvm-core] [rvm-commits] jikesrvm: 13 new changesets

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

Sorry for not noticing this change earlier.

I don't understand the rationale for the removal of comments that are duplicated on overridden methods (ie method in subclass carries the same comment as method in superclass).   Sorry if I'm being stupid here, but this duplication seems helpful/necessary (at least with the Eclipse IDE) because the comments are ostensibly useful, and when I'm walking through a class it seems unhelpful to have them elided (even though I can push up through the hierarchy (sometimes deep) to find them.   

To be clear, I'm all for standardization and application good practice, but it's just not immediately obvious to me that removing these comments is a good thing.  (I'm also very appreciative of Eric's substantial effort in pushing these changes through).

Can someone enlighten me?

Thanks,

--Steve

On 03/05/2012, at 2:10 AM, jik...@li... wrote:
> 
> details:   http://jikesrvm.hg.sourceforge.net/hgweb/jikesrvm/jikesrvm/rev/0d21d93c0db1
> changeset: 10446:0d21d93c0db1
> user:      Erik Brangs <eri...@gm...>
> date:      Fri Apr 27 22:26:43 2012 +0200
> description:
> RVM-962: Remove duplicated comments from org.mmtk.plan and org.mmtk.plan.* packages.