RFR: 8200072: Remove un-needed qualified export from java.base to java.desktop

[Phil Race]

I already pushed 8200072 but I'll file a new bug.

-phil.

On 03/22/2018 11:28 AM, mandy chung wrote:
> Yes it needs to run all platforms because some qualified exports are 
> platform-specific.
>
>
> On 3/22/18 11:22 AM, Phil Race wrote:
>> Seems you need to run it on all platforms to be sure.
>>
>> This one is used on Windows :
>>     exports jdk.internal.perf to java.desktop
>>
>> This one is used on Linux + Solaris :
>>     exports sun.nio.cs to java.desktop
>>
>
> These qualified exports should be moved to 
> src/java.base/$OS/classes/module-info.java.extra.  Maybe you can fix 
> this as part of JDK-8200072?
>
> Mandy
>
>> -phil.
>>
>> On 03/22/2018 11:01 AM, mandy chung wrote:
>>>
>>>
>>> On 3/22/18 10:44 AM, Alan Bateman wrote:
>>>>
>>>>
>>>> On 22/03/2018 17:41, Phil Race wrote:
>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8200072
>>>>> webrev: http://cr.openjdk.java.net/~prr/8200072/
>>>>>
>>>>> jdk.internal.math is exported to java.desktop but it has never 
>>>>> been needed,
>>>>> except briefly, in the jake forest during JDK 9 development.
>>>>> History is in the bug.
>>>> Looks good. We should run jdeps periodically to catch these.
>>>>
>>>
>>> Yes we should set up such automated task.  jdeps --check $MODULE 
>>> will find unused qualified exports.
>>>
>>> $ jdeps --check java.base
>>> java.base (jrt:/java.base)
>>>   [Unused qualified exports in java.base]
>>>     exports jdk.internal.math to java.desktop
>>>     exports jdk.internal.misc to java.sql,jdk.jdeps
>>>     exports jdk.internal.org.objectweb.asm to jdk.internal.vm.ci
>>>     exports jdk.internal.perf to java.desktop
>>>     exports jdk.internal.vm.annotation to jdk.internal.vm.ci
>>>     exports sun.net.www to jdk.incubator.httpclient
>>>     exports sun.nio.ch to jdk.sctp
>>>     exports sun.nio.cs to java.desktop
>>>     exports sun.reflect.misc to java.xml
>>>     exports sun.security.x509 to jdk.security.auth
>>>     exports sun.text.resources to jdk.localedata
>>>
>>> We will have to go through them if any package is accessed through 
>>> reflection.   I create JDK-8200121 to track this.
>>>
>>> Mandy
>>
>