RFR 8164907: Eliminate dependency on java.naming/com.sun.jndi.toolkit.url

[Chris Hegarty]

On 19/12/16 15:26, Pavel Rappo wrote:
> Hello,
>
> Could you please review the following change for [1]?
>
> For convenience I split the otherwise monolithic change into 4 separate webrevs.
> The first webrev addresses the first part of the task. Which is elimination of
> undesirable inter-modular dependencies. There were dependencies on 3 symbols,
> 2 utility methods and 1 class:
>
>    http://cr.openjdk.java.net/~prappo/8164907/webrev.00
>
> The remaining 3 webrevs address the second part of the task. Which is coalescing
> the CORBA code in a single repo:
>
>    http://cr.openjdk.java.net/~prappo/8164907/dev/
>    http://cr.openjdk.java.net/~prappo/8164907/corba/
>    http://cr.openjdk.java.net/~prappo/8164907/jdk/

These changes look good to me. Thanks for generating an
easy-to-see version of the diffs too. Makes it easier to
review.

-Chris.


> As far as I can see all related tests are fine.
>
> Thanks,
> -Pavel
>
> --------------------------------------------------------------------------------
> [1] https://bugs.openjdk.java.net/browse/JDK-8164907
>