RFR: 8309847: FrameForm and RegisterForm constructors should initialize all members

Tobias Hartmann thartmann at openjdk.org
Wed Jun 14 11:12:00 UTC 2023


On Tue, 13 Jun 2023 02:26:01 GMT, Vladimir Petko <duke at openjdk.org> wrote:

> This PR fixes missing constructor initialisations in formsopt.cpp.
> This PR does not implement [CppCoreGuidelines#Rc-in-class-initializer](https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Rc-in-class-initializer ) to keep the style consistent and minimise changes.

Looks good to me too. I'll sponsor after some quick sanity testing.

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

Marked as reviewed by thartmann (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14435#pullrequestreview-1479130577


More information about the hotspot-compiler-dev mailing list