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

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


On Mon, 21 Aug 2023 04:48:51 GMT, David Holmes <dholmes 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.
>
> @jdksjolen  The guide lists what is permitted to be used, so if it isn't listed as permitted then it isn't presently permitted.

Thank you David, then I guess I'd need to get the guide changed before my or Axel's proposal is merged.

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

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


More information about the hotspot-dev mailing list