On 10/27/16 6:31 PM, Pete Brunet wrote:
> On 10/27/16 1:30 PM, Mandy Chung wrote:
>>> On Oct 27, 2016, at 10:44 AM, Phil Race <philip.r...@oracle.com> wrote:
>>>
>>> No, we are definitely shipping those.
>>> Unless of course you think we should stop shipping JNI headers too …
>>>
>> No. I tried to understand what is external interface. I took it that these
>> header files are external interfaces.
>>
>> I reviewed:
>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.05/index.html
>>
>> Please move AccessBridgeCalls.c to
>> src/jdk.accessibility/windows/native/bridge directory.
> I moved the source to
> src/jdk.accessibility/windows/native/bridge/src
> and the includes to
> src/jdk.accessibility/windows/native/bridge/include
>
> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.06/
I forgot to update the link in readme.html, i.e. include/bridge is now
bridge/include. See
http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.07/
>> Right now, the header files to be packaged in the JMOD file and linked into
>> the image is white-listed in the makefile.
>>
>> Ultimately, we want the entire directory to contain only header files for
>> external interfaces and then the build can simply take all files under
>> src/${MODULE}/share,${OS}/native/include and get rid of the hardcoded list
>> in the makefile. We are not there yet - one open issue is related to the
>> duplicated header files in VM and JDK and some are private and some are
>> external. But we should prepare for that.
>>
>> Mandy
>>
>>
