CSR review request for: 8187289 NotifyFramePop request is not cleared if JVMTI_EVENT_FRAME_POP is disabled

[David Holmes]

On 17/11/2017 12:00 PM, serguei.spitsyn at oracle.com wrote:
> Hi David,
> 
> I've to leave now for a couple of hours.
> I've created the CSR from the real bug report, but don't know anything 
> about the bugs-stage.
> A link "csr for" was auto-added when the CSR was created.

You must have done that while looking at the bug on bugs-stage. The real 
bug:

https://bugs.openjdk.java.net/browse/JDK-8187289

has no CSR.

David

> Thanks,
> Serguei
> 
> 
> On 11/16/17 17:53, David Holmes wrote:
>> The CSR needs to be created again from the real bug report. Everyone 
>> will need to re-review and re-add any necessary comments.
>>
>> David
>>
>> On 17/11/2017 11:51 AM, David Holmes wrote:
>>> On 17/11/2017 10:25 AM, serguei.spitsyn at oracle.com wrote:
>>>> Dan and Chris
>>>>
>>>> Could one of you, please, review the CSR for 8187289:
>>>> https://bugs-stage.openjdk.java.net/browse/JDK-8191098
>>>>
>>>> Bug is:
>>>> https://bugs-stage.openjdk.java.net/browse/JDK-8187289
>>>
>>> The links above should not be to bugs-stage!
>>>
>>> David
>>>
>>>> Approved webrev:
>>>> http://cr.openjdk.java.net/%7Esspitsyn/webrevs/2017/hotspot/8187289-jvmti-framepop.2/ 
>>>>
>>>>
>>>>
>>>> This CSR covers a change in behavior, not in the JVMTI spec.
>>>> We decided that the JVMTI NotifyFramePop is intuitive enough and 
>>>> does not need an update.
>>>>
>>>> Thanks,
>>>> Serguei
>>>>
>>>>
>>>>
>