RFR (S): 8189797:Fix initializer lists in G1ConcurrentMark and G1CMTask

Thomas Schatzl thomas.schatzl at oracle.com
Tue Oct 24 08:14:59 UTC 2017


Hi Kim,

  thanks for your review.

On Mon, 2017-10-23 at 15:06 -0400, Kim Barrett wrote:
> I forgot that for G1CMTask you moved the colon indicating the start
> of
> the initializer list, but then left the indentation of the
> initializer
> list unchanged, making it now inconsistent with others, including
> G1ConcurrentMark in the same file.

All fixed I think. Since this is such a trivial change, assuming I do
not get more feedback, I will just push it in the next few days.

Webrevs:
http://cr.openjdk.java.net/~tschatzl/8189797/webrev.1 (full)
http://cr.openjdk.java.net/~tschatzl/8189797/webrev.0_to_1 (diff)

Thanks,
  Thomas




More information about the hotspot-gc-dev mailing list