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

Johan Sjölen jsjolen at openjdk.org
Fri Aug 18 10:45:38 UTC 2023


Given some `GrowableArray<E>` where `E` is non-copyable with a move constructor then currently this will fail to compile. This is because `GrowableArray`'s expand and shrink calls the copy constructor. We cast the values to rvalues (akin to `std::move`) to instead call the move constructor if available. If there is no move constructor but there is a copy constructor, then that will be called instead.

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

Commit messages:
 - Move - do not copy

Changes: https://git.openjdk.org/jdk/pull/15344/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15344&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8314571
  Stats: 17 lines in 2 files changed: 15 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/15344.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15344/head:pull/15344

PR: https://git.openjdk.org/jdk/pull/15344


More information about the hotspot-dev mailing list