<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Thanks for noticing that Phil. Updated at<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.05/">http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.05/</a><br>
<br>
<br>
<div class="moz-cite-prefix">On 10/27/16 9:20 AM, Philip Race wrote:<br>
</div>
<blockquote cite="mid:58120D2F.1070704@oracle.com" type="cite">
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
But it still needs to say "jdk9/jdk9" not jdk9/client or jdk9/dev.<br>
<br>
-phil.<br>
<br>
On 10/26/16, 9:27 PM, Anirvan Sarkar wrote:
<blockquote
cite="mid:CAEQxcvGtbgKgjuCV5sa5Ub_qSdp7PZ4rYEv=V78e=GfcqAp9Mg@mail.gmail.com"
type="cite">Hi,
<div><br>
</div>
<div>If you replace the hex number with 'tip' then it will
always point to the latest version. </div>
<div><br>
</div>
<div>Something like <a moz-do-not-send="true"
href="http://hg.openjdk.java.net/jdk9/client/jdk/file/tip/src/jdk.accessibility/windows/native/include/bridge/AccessBridgeCalls.c"
target="_blank">http://hg.openjdk.java.<wbr>net/jdk9/client/jdk/file/tip/<wbr>src/jdk.accessibility/windows/<wbr>native/include/bridge/<wbr>AccessBridgeCalls.c</a></div>
<div><br>
</div>
<div>Regards,</div>
<div>Anirvan Sarkar<span></span><br>
<br>
On Thursday 27 October 2016, Pete Brunet <<a
moz-do-not-send="true"
href="javascript:_e(%7B%7D,'cvml','peter.brunet@oracle.com');"
target="_blank">peter.brunet@oracle.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"> <br>
<br>
<div>On 10/26/16 10:44 PM, Philip Race wrote:<br>
</div>
<blockquote type="cite"> >
<pre> 15 <a href=<a moz-do-not-send="true" href="http://hg.openjdk.java.net/jdk9/client/jdk/file/544828ab2a9b/src/jdk.accessibility/windows/native/include/bridge/AccessBridgeCalls.c" target="_blank">"http://hg.openjdk.java.n<wbr>et/jdk9/client/jdk/file/544828<wbr>ab2a9b/src/jdk.accessibility/<wbr>windows/native/include/bridge/<wbr>AccessBridgeCalls.c"</a>>
That URL is definitely not authoritative.
I think you need to give a pointer to something more like
<a moz-do-not-send="true" href="http://hg.openjdk.java.net/jdk9/jdk9/jdk/src/jdk.accessibility/windows/native/include/bridge/AccessBridgeCalls.c" target="_blank">http://hg.openjdk.java.net/jdk<wbr>9/jdk9/jdk/src/jdk.accessibili<wbr>ty/windows/native/include/<wbr>bridge/AccessBridgeCalls.c</a></pre>
</blockquote>
Looks like that hex number in there is the Mercurial long
revision number of the tip so that's going to keep
changing. I'm not aware of a "latest" link. Maybe some
other reader will know.
<blockquote type="cite">
<pre>But I am not sure about that either .. it may need to be split between the main URL and the location in the repo.
-phil
</pre>
On 10/26/16, 7:24 PM, Pete Brunet wrote:
<blockquote type="cite">
<pre>Please review the latest update at
<a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Eptbrunet/JDK-8167213/webrev.03/" target="_blank">http://cr.openjdk.java.net/~pt<wbr>brunet/JDK-8167213/webrev.03/</a>
The change is to AccessBridgeCalls.c. The license has been changed from
GPL2 to BSD. This is because the file was originally unlicensed prior
to being bundled into the JDK and the compiled .obj is linked to by
vendors creating proprietary code. Vendors will be instructed to
download AccessBridgeCalls.c from the OpenJDK repository. Also the
include/use of AccessBridgeDebug.h/cpp has been removed.
Please also review readme.html which has been added to
.../jdk/include/win32/bridge.
Pete
On 10/25/16 6:48 AM, Alexandr Scherbatiy wrote:
</pre>
<blockquote type="cite">
<pre>The fix looks good to me.
Thanks,
Alexandr.
On 10/24/2016 1:18 PM, Erik Joelsson wrote:
</pre>
<blockquote type="cite">
<pre>The last change looks good and simple to me.
/Erik
On 2016-10-21 06:55, Pete Brunet wrote:
</pre>
<blockquote type="cite">
<pre>Please see the latest update
<a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Eptbrunet/JDK-8167213/webrev.02/" target="_blank">http://cr.openjdk.java.net/~pt<wbr>brunet/JDK-8167213/webrev.02/</a>
The fix now is to simply remove the copy of the AccessBridgeCalls.c
file
into the JDK.
AccessBridgeCalls.c is the implementation of the documented Java Access
Bridge API and is a set of wrapper functions that hides the
complications related to interfacing to JAB's WindowsAccessBridge*.dll.
In the past users of the API would compile and link to
AccessBridgeCalls.c/obj.
Since the interface implementation of AccessBridgeCalls.c will no
longer
be provided the JAB API documentation will be updated to instruct a
user
how to create an equivalent of AccessBridgeCalls.c. The documentation
will also contain a reference to the JAB 2.0.2 download
<a moz-do-not-send="true" href="http://www.oracle.com/technetwork/java/javase/downloads/jab-2-0-2-download-354311.html" target="_blank">http://www.oracle.com/technetw<wbr>ork/java/javase/downloads/jab-<wbr>2-0-2-download-354311.html</a>
which does contain AccessBridgeCalls.c and which is compatible with the
current API and related calls into WindowsAccessBridge*.dll.
Pete
On 10/18/16 12:28 PM, Pete Brunet wrote:
</pre>
<blockquote type="cite">
<pre>I've updated the webrev. Please see
<a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Eptbrunet/JDK-8167213/webrev.01/" target="_blank">http://cr.openjdk.java.net/~pt<wbr>brunet/JDK-8167213/webrev.01/</a>
Rather than removing the files needed by Assistive Technology
developers
we have to provide them in JDK. However since there is a .c file
in the
group of files the files were moved from the include directory to a
new
javaaccessbridge directory.
On 10/17/16 2:43 AM, Magnus Ihse Bursie wrote:
</pre>
<blockquote type="cite">
<pre>On 2016-10-14 17:51, Pete Brunet wrote:
</pre>
<blockquote type="cite">
<pre>Please review the following.
The .h files and .c file provided to allow Assistive Technology to
interface to the Java Access Bridge API are being removed from
the built
JRE/JDK images. They are not used much and they can be obtained
online
via the OpenJDK web site. The pubs will be updated to mention the
location of the files.
Since there is a .c file in this group of files the directory
structure
has been changed slightly to remove the include directory.
There was one file missing from the group of files needed by
developers
and that was moved from the common to the bridge directory.
The make was updated in response to the above.
Bug: <a moz-do-not-send="true" href="https://bugs.openjdk.java.net/browse/JDK-8167213" target="_blank">https://bugs.openjdk.java.net/<wbr>browse/JDK-8167213</a>
Webrev: <a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Eptbrunet/JDK-8167213/webrev.00/" target="_blank">http://cr.openjdk.java.net/~pt<wbr>brunet/JDK-8167213/webrev.00/</a>
</pre>
</blockquote>
<pre>Build changes looks good to me.
/Magnus
</pre>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</blockquote>
</div>
</blockquote>
</div>
--
Sent from Gmail Mobile
</blockquote>
</blockquote>
</body></html>