RFR: 8191870: Remove badJNIHandle

[Kim Barrett]

> On Nov 28, 2017, at 8:45 AM, Daniel D. Daugherty <daniel.daugherty at oracle.com> wrote:
> 
> On 11/27/17 6:48 PM, Kim Barrett wrote:
>>> 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/
> 
> src/hotspot/share/runtime/globals.hpp
>     No comments.
> 
> src/hotspot/share/runtime/jniHandles.cpp
>     No comments.
> 
> src/hotspot/share/runtime/jniHandles.hpp
>     No comments.
> 
> src/hotspot/share/runtime/sharedRuntime.cpp
>     No comments.
> 
> src/hotspot/share/utilities/globalDefinitions.hpp
>     No comments.
> 
> Thumbs up.

Thanks, Dan.

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