RFR: 8314571: GrowableArray should move its old data and not copy it [v2]

Johan Sjölen jsjolen at openjdk.org
Mon Aug 21 09:54:07 UTC 2023


On Sun, 20 Aug 2023 06:11:42 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:

>> Hi, in short: Yes, I believe that I can switch and that `std::move` is better.
>> 
>> In long: I can't find any mention of `std::move` not being allowed in the style guide, using it seems to be in the same spirit as other std library usage (`#include <new>` for example) and since it is just a better version of what I wrote here let's do the switch.
>
> I believe "Rvalue references and move semantics" falls under the "Undecided Features" category.

@merykitty is correct, at the bottom of the style guide under undecided features:

> Rvalue references and move semantics

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15344#discussion_r1299880031


More information about the hotspot-dev mailing list