RFR(XS) 8224679: Shenandoah: Make ShenandoahParallelCodeCacheIterator noncopyable

Aleksey Shipilev shade at redhat.com
Thu May 23 19:28:16 UTC 2019


On 5/23/19 5:01 PM, Zhengyu Gu wrote:
> Copies may cause its member '_iters' to be freed multiple times.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8224679

Please link the issue that made these redundant.

> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8224679/webrev.00/

Please match what ShenandoahRegionIterator does, notably, declare variables? I remember the absence
of those tripped some compilers.

  // No implicit copying: iterators should be passed by reference to capture the state
  ShenandoahRegionIterator(const ShenandoahRegionIterator& that);
  ShenandoahRegionIterator& operator=(const ShenandoahRegionIterator& o);

Otherwise good.

-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20190523/ad4a7519/signature.asc>


More information about the hotspot-gc-dev mailing list