RFR: 8191870: Remove badJNIHandle
Kim Barrett
kim.barrett at oracle.com
Tue Nov 28 19:00:35 UTC 2017
> On Nov 28, 2017, at 9:16 AM, Erik Österlund <erik.osterlund at oracle.com> wrote:
>
> Hi Kim,
>
> Still looks good.
Thanks, Erik.
>
> Thanks,
> /Erik
>
> On 2017-11-28 00:48, 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/
>> incr: http://cr.openjdk.java.net/~kbarrett/8191870/open.01.inc/
More information about the hotspot-dev
mailing list