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