RFR: 8299953: Merge ContiguousSpaceDCTOC into DirtyCardToOopClosure

Thomas Schatzl tschatzl at openjdk.org
Wed Jan 25 16:41:13 UTC 2023


On Wed, 11 Jan 2023 10:09:20 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

> Simple change of merging a class into its super class.
> 
> Test: hotspot_gc
> 
> PS: `git diff --color-moved=dimmed_zebra` might be useful to see the diff.

Lgtm, please fix the indentation before pushing.

src/hotspot/share/gc/shared/space.cpp line 61:

> 59:       // later card.  Figure out where the object ends.
> 60:       assert(_sp->block_size(top_obj) == cast_to_oop(top_obj)->size(),
> 61:         "Block size and object size mismatch");

Pre-existing: Indentation for the assert message is off.

Suggestion:

      assert(_sp->block_size(top_obj) == cast_to_oop(top_obj)->size(),
             "Block size and object size mismatch");

src/hotspot/share/gc/shared/space.cpp line 149:

> 147:                                                    HeapWord* bottom,    \
> 148:                                                    HeapWord* top,       \
> 149:                                                    ClosureType* cl) {   \

Parameter indentation is off now with the change, also the macro continuation `` characters do not line up any more.

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

Marked as reviewed by tschatzl (Reviewer).

PR: https://git.openjdk.org/jdk/pull/11938


More information about the hotspot-gc-dev mailing list