RFR: JDK-8191203 Remove duplicated jimage.hpp

David Holmes david.holmes at oracle.com
Tue Nov 21 11:44:38 UTC 2017


Hi Magnus,

This seems fine to me.

Do you know what the story is with the copyright header?

Thanks,
David

On 21/11/2017 9:13 PM, Magnus Ihse Bursie wrote:
> I ran a duplicate name check on the source base. Then I discovered that 
> there is a jimage.hpp in both src/hotspot/share/classfile/jimage.hpp and 
> src/java.base/share/native/libjimage/jimage.hpp. They are identical 
> (apart from copyright headers), and once again, we shouldn't have both. 
> David Holmes suggested removing the hotspot version, so this I've done.
> 
> I intend to push this to jdk/hs.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8191203
> WebRev: 
> http://cr.openjdk.java.net/~ihse/JDK-8191203-remove-duplicated-jimage-hpp/webrev.01 
> 
> 
> /Magnus



More information about the build-dev mailing list