RFR: 8255082: HotSpot Style Guide should permit noexcept [v2]

[David Holmes]

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