RFR: 8255082: HotSpot Style Guide should permit noexcept [v2]
David Holmes
dholmes at openjdk.org
Mon Jun 2 21:10:51 UTC 2025
On Mon, 2 Jun 2025 08:21:34 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
>> Please review this change to permit the use of `noexcept` under certain
>> circumstances in HotSpot code.
>>
>> http://wg21.link/n3050
>>
>> Testing:
>>
>> JDK-8316930 (HotSpot should use noexcept instead of throw()) showed what the
>> conversion would look like. It will need to be brought up to current mainline,
>> possibly with modifications.
>>
>> This is a modification of the Style Guide, so rough consensus among the
>> HotSpot Group members is required to make this change. Only Group members
>> should vote for approval (via the github PR), though reasoned objections or
>> comments from anyone will be considered. A decision on this proposal will not
>> be made before Friday 16-June-2025 at 12h00 UTC.
>>
>> Since we're piggybacking on github PRs here, please use the PR review process
>> to approve (click on Review Changes > Approve), rather than sending a "vote:
>> yes" email reply that would be normal for a CFV.
>
> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
>
> dholmes review
Marked as reviewed by dholmes (Reviewer).
doc/hotspot-style.html line 1121:
> 1119: <li>Only the argument-less form of <code>noexcept</code> exception
> 1120: specifications are permitted. <code>noexcept</code> exception
> 1121: specifications with arguments are forbidden.</li>
I was suggesting dropping the second sentence as it is implied by the first.
-------------
PR Review: https://git.openjdk.org/jdk/pull/25574#pullrequestreview-2889941827
PR Review Comment: https://git.openjdk.org/jdk/pull/25574#discussion_r2122157846
More information about the hotspot-dev
mailing list