RFR of JDK-8186610,move ModuleUtils to top-level testlibrary

Hamlin Li huaming.li at oracle.com
Thu Oct 11 06:31:56 UTC 2018


Thank you clarifying Igor.

Moving ModuleTargetHelper to local folder has a drawback: it's hard for 
future maintainer to found it if they need to use it in other places, 
that make it an "*invisible*" library class.

Although I don't fully agree with you, I have updated the webrev as you 
suggested: http://cr.openjdk.java.net/~mli/8186610/webrev.01/

Thank you

-Hamlin


On 2018/10/11 11:38 AM, Igor Ignatyev wrote:
> b/c this will make test library modularization[1] somewhat 
> troublesome, also I ain't sure if ModuleTargetHelper really needs to 
> be placed into the top-level library regardless of its dependency on 
> non-public API. "promoting" test library class to the top-level 
> library comes w/ increased maintenance costs, the parent task[2] 
> explains that in more details.
>
> [1] https://bugs.openjdk.java.net/browse/JDK-8211358
> [2] https://bugs.openjdk.java.net/browse/JDK-8211290
>
> -- Igor
>
>> On Oct 10, 2018, at 8:26 PM, Hamlin Li <huaming.li at oracle.com 
>> <mailto:huaming.li at oracle.com>> wrote:
>>
>> Hi Igor,
>>
>> Would you please clarify your concern further? I mean why 
>> ModuleTargetHelper can be put to top level when it use non-public 
>> APIs e.g. jdk.internal.module.*?
>>
>> Thank you
>>
>> -Hamlin
>>
>> On 2018/10/11 11:08 AM, Igor Ignatyev wrote:
>>> Hi Hamlin,
>>>
>>> as ModuleTargetHelper uses non-public API, I'd prefer not to have in 
>>> a common test library, and 8211976 suggests moving it closer to 
>>> tests. could you please explain why you decided to put it into the 
>>> library instead?
>>>
>>> Thanks,
>>> -- Igor
>>>
>>> ----- Original Message -----
>>> From: huaming.li at oracle.com <mailto:huaming.li at oracle.com>
>>> To: core-libs-dev at openjdk.java.net 
>>> <mailto:core-libs-dev at openjdk.java.net>
>>> Sent: Wednesday, October 10, 2018 7:40:34 PM GMT -08:00 US/Canada 
>>> Pacific
>>> Subject: RFR of JDK-8186610,move ModuleUtils to top-level testlibrary
>>>
>>> Would you please review the following patch?
>>>
>>> bug:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8186610
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8211976
>>>
>>> webrev: http://cr.openjdk.java.net/~mli/8186610/ 
>>> <http://cr.openjdk.java.net/%7Emli/8186610/>
>>>
>>>
>>> Thank you
>>>
>>> -Hamlin
>>>
>>
>



More information about the core-libs-dev mailing list