Trivial bug in psScavenge.cpp?
jeremymanson at google.com
Fri Oct 25 19:35:24 PDT 2013
Following my habit of reporting trivial bugs (and perhaps my habit of being
wrong about their being bugs). I've been playing around with the
event-based JVM tracing API, since we had our own version of the same thing
internally (amusingly, we called our timer TraceGCTime instead of
GCTraceTime, but it did almost exactly the same thing). I noticed this in
469 GCTraceTime tm("StringTable", false, false, &_gc_timer);
470 // Unlink any dead interned Strings and process the remaining
471 PSScavengeRootsClosure root_closure(promotion_manager);
That's all well and good, but surely you want curly braces before line 469
and after line 472? Otherwise it credits a lot more time to string table
processing than is appropriate...
More information about the hotspot-dev