<Swing Dev> RfR JDK-8167213 Move include/bridge/AccessBridgeCalls.c to the source directory

Philip Race philip.race at oracle.com
Tue Nov 1 23:29:30 UTC 2016


This is +1 from me.
Honestly I don't think it needs a re-review by Mandy too just to that 
comment.
So go ahead and push.

-phil.

On 11/1/16, 4:21 PM, Pete Brunet wrote:
> JPRT job ran OK.  Just need +1s from Phil/Mandy for the comment change
> shown below.
>
> On 11/1/16 9:56 AM, Pete Brunet wrote:
>> Mandy and Phil, I thought it would be helpful to add this to the comment
>> in AccesssBridgeCalls.h:
>>
>>   *
>>   * Also note that the API is used in the jaccessinspector and
>> jaccesswalker tools.
>>   * The source for those tools is available in the OpenJDK repository at
>> these links:
>>   *
>>   *
>> http://hg.openjdk.java.net/jdk9/jdk9/jdk/file/tip/src/jdk.accessibility/windows/native/jaccessinspector/jaccessinspector.cpp
>>   *
>> http://hg.openjdk.java.net/jdk9/jdk9/jdk/file/tip/src/jdk.accessibility/windows/native/jaccesswalker/jaccesswalker.cpp
>>   *
>>
>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.10/
>>
>> I'll start the JPRT job today and then once I get your approval for that
>> comment I will push this into 9.
>>
>> Pete
>>
>> On 11/1/16 4:27 AM, Erik Joelsson wrote:
>>> Looks good.
>>>
>>> /Erik
>>>
>>> On 2016-10-31 15:36, Pete Brunet wrote:
>>>> On 10/28/16 8:14 PM, Mandy Chung wrote:
>>>>>> On Oct 28, 2016, at 1:59 PM, Philip Race<philip.race at oracle.com>
>>>>>> wrote:
>>>>>>
>>>>>> If it is not in the image then there is no point in the file existing.
>>>>>> Maybe this could just be a comment at the top of the include file.
>>>>>>
>>>>> This works for me.
>>>> Updated:
>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.09/
>>>>> Mandy
>>>>>
>>>>>> -phil.
>>>>>>
>>>>>> On 10/28/16, 12:42 PM, Mandy Chung wrote:
>>>>>>>> On Oct 28, 2016, at 11:32 AM, Pete
>>>>>>>> Brunet<peter.brunet at oracle.com>   wrote:
>>>>>>>>
>>>>>>>> Hi Mandy, That simplifies things.  The new patch is at:
>>>>>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.08/
>>>>>>> Looks better.
>>>>>>>
>>>>>>> I only notice now that the readme.html is in the include
>>>>>>> directory.  That should be in the documentation as you proposed
>>>>>>> earlier.  I don’t think it should be copied to the image.
>>>>>>>
>>>>>>> Mandy
>>>>>>>



More information about the swing-dev mailing list