RFR (s): 8026849 - Fix typos in the GC code, part 2

[Kirk Pepperdine]

On Jan 29, 2014, at 12:29 AM, Jesper Wilhelmsson <jesper.wilhelmsson at oracle.com> wrote:

> Hi,
> 
> Could I have a couple of reviews for this small patch?
> 
> These are fixes for a couple of typos that I didn't include in the big fix recently pushed. They were not included since they do not reside in comments but rather assert messages, flag descriptions and some verbose messages.
> 
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8026849
> 
> Webrev:
> http://cr.openjdk.java.net/~jwilhelm/8026849/webrev/
> 
> 
> This patch also contains a fix for 8023899 - Typo in TraceCPUTime message.
> https://bugs.openjdk.java.net/browse/JDK-8023899

-        _logfile->print(" [Times: user=%3.2f sys=%3.2f, real=%3.2f secs] ",
+        _logfile->print(" [Times: user=%3.2f sys=%3.2f real=%3.2f secs] ",
           user_secs, system_secs, real_secs);


caveat, not a official reviewer. I looked at all the typo fixes and they all look good. The only thing I can comment on is that I appreciate what you’re trying to do here but I don’t really care if there is a comma or a space between the entries as I don’t read these files but my parser does read them and it cares. Yeah it’s a trivial fix to the parser for this one but when I look at all the spaces, commas, ’s’es and other trivial formatting changes that creep in over time it starts to get really messy. Can we please just leave this one out?

Kind regards,
Kirk

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20140129/dc7adb87/attachment.htm>