RFR: 8191870: Remove badJNIHandle

Kim Barrett kim.barrett at oracle.com
Mon Nov 27 23:48:18 UTC 2017


> On Nov 27, 2017, at 3:43 PM, coleen.phillimore at oracle.com wrote:
> 
> 
> 
> On 11/27/17 3:21 PM, Kim Barrett wrote:
>>> On Nov 27, 2017, at 10:19 AM, coleen.phillimore at oracle.com wrote:
>>> 
>>> 
>>> My question was why there's a command line option, which seems like nobody on earth should use it.  Yes, we don't need to zap for product, only for debug.
>> So remove the CLA and instead #ifdef ASSERT the zapping.
> 
> Yes.
>> 
>> I’d be okay with that.  Another CLA bites the dust.
> 
> What looks like a pointless one at that.
> 
> Coleen

New webrev with ZapJNIHandleArea removed.

full: http://cr.openjdk.java.net/~kbarrett/8191870/open.01/
incr: http://cr.openjdk.java.net/~kbarrett/8191870/open.01.inc/



More information about the hotspot-dev mailing list