On Fri, 28 Apr 2023 00:50:54 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> We have two suggestions:
>>> - "or a function on a thread cannot be performed at the thread's current 
>>> frame".
>>> - "the function cannot be performed on the thread's current frame."
>> 
>> So, we need to pick one. The second one looks simpler to me but
>> I'm not completely sure that it reflects the full meaning correctly.
>> I wonder about a mix of the two suggestions above:
>> 
>>> "the function cannot be performed at the thread's current frame."
>
> We need to account for the `SetLocalXXX` functions with the `depth` parameter 
> which also return `OPAQUE_FRAME` error code for virtual frames. My concern is 
> if the "current frame" part is fully correct.

I've pushed variant from Chris which is a rephrase of what Alan suggested.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13546#discussion_r1182047387

Reply via email to