RFR: 8038628: Remove unused Closure::abort()

Jon Masamitsu jon.masamitsu at oracle.com
Fri Mar 28 16:49:06 UTC 2014


Stefan,

I could not tell from the CR why the use of Cosure::abort()
was removed.

      }
-    if (cl->abort()) return cur;
-    // The check above must occur before the operation below, since an
-    // abort might invalidate the "size" operation.
      cur += obj->size();

Can you explain a bit more?  Thanks.

Jon


On 3/28/2014 6:13 AM, Stefan Karlsson wrote:
> Please, review this small patch to remove Closure::abort().
>
> webrev: http://cr.openjdk.java.net/~stefank/8038628/webrev.00/
> RFE: https://bugs.openjdk.java.net/browse/JDK-8038628
>
> thanks,
> StefanK



More information about the hotspot-dev mailing list