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
