Re: RFR: 8228394: Cleanup unused java.net SharedSecrets classes

2019-07-19 Thread Claes Redestad 

On 2019-07-19 15:28, Chris Hegarty wrote:

Nice cleanup! Reviewed.


Thanks for reviewing, Chris!

/Claes

Re: RFR: 8228394: Cleanup unused java.net SharedSecrets classes

2019-07-19 Thread Chris Hegarty 



> On 19 Jul 2019, at 13:51, Claes Redestad  wrote:
> 
> Hi,
> 
> please review this cleanup to remove a few java.net access objects that
> are no longer in use.
> 
> Webrev: http://cr.openjdk.java.net/~redestad/8228394/open.00/
> Bug:https://bugs.openjdk.java.net/browse/JDK-8228394

Nice cleanup! Reviewed.

-Chris.

Re: RFR: 8228394: Cleanup unused java.net SharedSecrets classes

2019-07-19 Thread Claes Redestad 

On 2019-07-19 15:01, Alan Bateman wrote:

This looks good to me.


Thanks for reviewing, Alan!

/Claes

Re: RFR: 8228394: Cleanup unused java.net SharedSecrets classes

2019-07-19 Thread Alan Bateman 




On 19/07/2019 13:51, Claes Redestad wrote:

Hi,

please review this cleanup to remove a few java.net access objects that
are no longer in use.

Webrev: http://cr.openjdk.java.net/~redestad/8228394/open.00/
Bug:    https://bugs.openjdk.java.net/browse/JDK-8228394

(One usage was from sun.misc.ClassLoaderUtil, removed by 
https://bugs.openjdk.java.net/browse/JDK-8034853, which seems

to have forgotten to remove a related and now unused test.jar
file. I removed this too and verified all tests pass.)

This looks good to me.