RFR: 8025856 - Fix typos in the GC code (original) (raw)

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Mon Jan 13 14:37:06 UTC 2014

Hi Thomas,

Thanks for looking at this!

I have fixed the things you pointed out. The webrev is updated (same link). I mainly fixed spelling errors in the runtime code to minimize the diff. I have fixed a few capitalizations but I didn't go all-in as I did in some GC files :-) /Jesper

Thomas Schatzl skrev 13/1/14 2:42 PM:

On Wed, 2014-01-08 at 16:37 +0100, Jesper Wilhelmsson wrote:

Hi,

Anyone up for a really tedious review? This change fixes about 300 typos in comments in the GC code. There should only be changes in comments in this change. I had this out for review a while ago but it was a bit late in 8u20 to push a large change like this. I got a few comments last time that have been fixed in this new version. This new webrev is based on jdk9/hs-gc and once approved it will be backported to jdk8 (to minimize the diff between jdk8 and jdk9). Let me know if you have some change out that you want to push before I push this change. The risk of merge conflicts is high since the change touches a lot of code. Bug: https://bugs.openjdk.java.net/browse/JDK-8025856 Webrev: http://cr.openjdk.java.net/~jwilhelm/8025856/webrev.1/ Looks good. A few additional nitpicks: concurrentMark.hpp, line 443: // true: -> // True: g1CollectorPolicy.cpp, line 1078: // do that -> // Do that adaptiveSizePolicy.hpp: there are a lot of initial lower case letters in descriptions beginning from around line 150 arguments.cpp, line 181: // following are JVMTI a -> // Following are... arguments.cpp, line 3679: // which are subtlety different from each other but neither works with "Subtlety" is the noun, I think the adjective is required here (subtly) mutex.cpp, line 966: I think it is okay to remove the [sic] comment - [sic] comments on the wrong spelling of some word, and since now the spelling of the word the [sic] refers to is gone, this should be removed as well. Or I just did not get the joke :) It should be "succeeds" too (thread is third person singular) mutexLocker.hpp, line 348: // Only too be used -> // Only to be used safepoint.hpp, line 158: voluntary -> voluntarily I saw that in runtime/compiler code there were many cases where the First letter after the comment uses lowercase (I mentioned a few above though). Feel free to apply them or not depending on what the runtime/compiler team thinks. Otherwise looks good. Thomas

More information about the hotspot-gc-dev mailing list