RFR: 8337241: Shenandoah: Normalize include guards

Aleksey Shipilev shade at openjdk.org
Fri Jul 26 07:56:33 UTC 2024


On Thu, 25 Jul 2024 23:15:31 GMT, William Kemper <wkemper at openjdk.org> wrote:

> Some of the include guards are not like the others.

Looks fine, with a nit.

src/hotspot/share/gc/shenandoah/shenandoahController.hpp line 105:

> 103:   void update_gc_id();
> 104: };
> 105: #endif //SHARE_GC_SHENANDOAH_SHENANDOAHCONTROLLER_HPP

Should have a space after `//`?

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

Marked as reviewed by shade (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/20342#pullrequestreview-2201170905
PR Review Comment: https://git.openjdk.org/jdk/pull/20342#discussion_r1692658169


More information about the hotspot-gc-dev mailing list