RFR : [XS] 8214373: adjust usage of ReleaseLongArrayElements in MacosxDebuggerLocal

JC Beyler jcbeyler at google.com
Tue Nov 27 15:47:42 UTC 2018


Hi Matthias,

Not a reviewer but looks good to me :). I checked if other releases were
not in the right place and they all seemed good as well.
Jc

On Tue, Nov 27, 2018 at 7:09 AM Baesken, Matthias <matthias.baesken at sap.com>
wrote:

> Hello, please review this small fix .
>
> In  MacosxDebuggerLocal.m ,  at a few places  the  functions
>  GetLongArrayElements  and  ReleaseLongArrayElements  are used.
>
> However  there is one place  where in an early return  the
> ReleaseLongArrayElements call has been forgotten .
>
> Additionally at another place  ReleaseLongArrayElements   is at the wrong
> place (although this is probably not a big issue  currently).
>
>
> Bug/webrev:
>
> https://bugs.openjdk.java.net/browse/JDK-8214373
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8214373.0/
>
>
>
> Thanks, Matthias
>


-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20181127/a612b33a/attachment-0001.html>


More information about the serviceability-dev mailing list