RFR: JDK-8189607 Remove duplicated jvmticmlr.h

mandy chung mandy.chung at oracle.com
Wed Oct 18 17:57:26 UTC 2017



On 10/18/17 1:26 AM, Erik Joelsson wrote:
> On 2017-10-18 10:04, Magnus Ihse Bursie wrote:
>> The file jvmticmlr.h is stored twice in the repo, both in hotspot and 
>> in java.base. They are both identical, and only the java.base version 
>> is included in the final product. This might arguably have been 
>> useful in a pre-consolidated world, but makes absolutely no sense now.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8189607
>> WebRev: 
>> http://cr.openjdk.java.net/~ihse/JDK-8189607-remove-duplicated-jvmticmlr/webrev.01
>>
> The question is, which file location makes the most sense. I think 
> your pick of java.base/share/native/include probably makes more sense 
> as that makes it much clearer that this is an exported header file.
>

jvmticmlr.h is an exported header file and 
java.base/share/native/include is a proper location as described in JEP 
201 about the modular source layout.

The change looks good to me too.

Mandy




More information about the build-dev mailing list