Hi Pavel,

Looks like a good consolidation and resolution.

Roger


On 12/19/2016 10:26 AM, 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/
As far as I can see all related tests are fine.

Thanks,
-Pavel

--------------------------------------------------------------------------------
[1]https://bugs.openjdk.java.net/browse/JDK-8164907


Reply via email to