Integrated: 8264707: HotSpot Style Guide should permit use of lambda

[Kim Barrett]

On Tue, 17 Aug 2021 13:49:43 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this proposal to permit the use of lambda expressions in
> HotSpot code, with some restrictions and suggestions for good usage within
> HotSpot code.  Lambda expressions were added in C++11, and provide a more
> expressive syntax for local functions, with a number of use-cases where they
> can improve readability by eliminating a lot of uninteresting boilerplate.
> 
> Some example uses are included, but are not part of the proposed change.
> They will be removed from the PR before it is pushed.  (In particular, the
> ScopeGuard utility uses move semantics, the use of which hasn't been
> approved or even discussed.)  They are given to show some of the benefits
> that might accrue from permitting the use of lambdas. In particular, they
> highlight some of the code reduction that is possible. Some of these code
> changes might be proposed in the future, using the normal PR process.
> 
> 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 Wednesday 1-Sep-2021 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.

This pull request has now been integrated.

Changeset: 3eca9c36
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/3eca9c36a63595baee0659ac818fd5bedc528db1
Stats:     399 lines in 2 files changed: 386 ins; 11 del; 2 mod

8264707: HotSpot Style Guide should permit use of lambda

Reviewed-by: stefank, dholmes, coleenp, iklam, sjohanss, eosterlund, iveresov, kvn

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

PR: https://git.openjdk.java.net/jdk/pull/5144