[lworld] RFR: 8373706: [lworld] Layout names should be self-explanatory [v2]
Frederic Parain
fparain at openjdk.org
Tue Dec 16 14:52:57 UTC 2025
On Mon, 15 Dec 2025 22:32:09 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:
>> Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix ending blank line
>
> looks reasonable to me
Thanks @DanHeidinga @MrSimms for your reviews.
> src/hotspot/share/oops/layoutKind.cpp line 45:
>
>> 43: ShouldNotReachHere();
>> 44: }
>> 45: }
>
> Do we need a blank line at the end of the file?
Fixed
-------------
PR Comment: https://git.openjdk.org/valhalla/pull/1801#issuecomment-3660934478
PR Review Comment: https://git.openjdk.org/valhalla/pull/1801#discussion_r2623590406
More information about the valhalla-dev
mailing list