Integrated: 8254285: G1: Remove "What is this about" comment in G1CollectedHeap.cpp

Thomas Schatzl tschatzl at openjdk.java.net
Fri Oct 9 10:00:22 UTC 2020


On Fri, 9 Oct 2020 09:42:13 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   can I get quick reviews for this imho trivial change that removes some strange comment leftover from initial G1 import
>   reading:
> 
>    // FIXME: what is this about?
>   // I'm ignoring the "fill_newgen()" call if "alloc_event_enabled"
>   // is set.
> 
> None of the referenced identifiers are in any kind of recent code base (looking back to some pre-g1 7u changes). In its
> current context it does not make sense either and apparently whatever omission the comment suggests has not hurt G1 for
> years.  Testing: local compilation
> 
> Thanks,
>   Thomas

This pull request has now been integrated.

Changeset: a95590d0
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/a95590d0
Stats:     3 lines in 1 file changed: 0 ins; 3 del; 0 mod

8254285: G1: Remove "What is this about" comment in G1CollectedHeap.cpp

Reviewed-by: shade, sjohanss

-------------

PR: https://git.openjdk.java.net/jdk/pull/573



More information about the hotspot-gc-dev mailing list