RFR [9] 8148117: Move sun.misc.Cleaner to jdk.internal.ref

[Roger Riggs]

Hi Chris,

Looks good, thanks for updating the test.

One typo:

"Unexpected *exist *code" Roger


On 1/26/2016 11:27 AM, Chris Hegarty wrote:
> Latest webrev updated in-place:
>    http://cr.openjdk.java.net/~chegar/8148117/
>
>   * to execute the run method requires an appropriate permission
>   * reverted any copyright changes ( leave to a bulk update )
>   * updated the test to remove the script
>
> -Chris.
>
>
> On 26 Jan 2016, at 15:23, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>
>> On 26/01/2016 13:55, Chris Hegarty wrote:
>>> It is wonderful to see the various ideas on this thread about the longer
>>> term solution to the prompt releasing of direct buffer native memory. I
>>> do not want to obstruct that ( it is very informative ), but I’d like to warp up
>>> the review on the actual moving of Cleaner. To that end, I’ve update the
>>> webrev as per Alan’s comments and suggestion ( to extend Runnable ).
>>>
>>> http://cr.openjdk.java.net/~chegar/8148117/
>>>
>>> -Chris.
>>>
>>>
>> This looks okay. As a defensive-in-depth then Cleaner::run can do a permission check and should ease concerns about leakage.
>
>