Should the same change be applied to the .h files as well? Mandy
> On Oct 26, 2016, at 7:24 PM, Pete Brunet <peter.bru...@oracle.com> wrote: > > Please review the latest update at > http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.03/ > > 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: >> >> The fix looks good to me. >> >> Thanks, >> Alexandr. >> >> On 10/24/2016 1:18 PM, Erik Joelsson wrote: >>> The last change looks good and simple to me. >>> >>> /Erik >>> >>> >>> On 2016-10-21 06:55, Pete Brunet wrote: >>>> Please see the latest update >>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.02/ >>>> >>>> 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 >>>> http://www.oracle.com/technetwork/java/javase/downloads/jab-2-0-2-download-354311.html >>>> >>>> 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: >>>>> I've updated the webrev. Please see >>>>> http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.01/ >>>>> >>>>> 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: >>>>>> On 2016-10-14 17:51, Pete Brunet wrote: >>>>>>> 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: https://bugs.openjdk.java.net/browse/JDK-8167213 >>>>>>> >>>>>>> Webrev: http://cr.openjdk.java.net/~ptbrunet/JDK-8167213/webrev.00/ >>>>>> Build changes looks good to me. >>>>>> >>>>>> /Magnus >>> >> >