[foreign] rename package nicl to foreign

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Fri Jul 6 09:26:38 UTC 2018



On 06/07/18 03:34, Sundararajan Athijegannathan wrote:
> I built and tested on Mac. All fine except that StdLibTest fails 
> intermittently and this may not due to current changes.
>
> PS. Do we have to rename README.nicl to README.foreign? & change 
> branch name?
Right - we should, which reminds me we should probably also update the 
Panama project page, which is still talking about nicl.

Maurizio
>
> -Sundar
>
> On 05/07/18, 7:01 PM, Maurizio Cimadamore wrote:
>> Hi,
>> as part of the code restructuring, here's a webrev which updates the 
>> packages names to the new form:
>>
>> nicl -> foreign
>> nicl.types -> foreign.memory
>> nicl.metadata -> foreign.annotations
>> nicl.layout -> foreign.layout
>>
>> I've also did similar renaming in the test folder, and I've changed 
>> the test group form jdk_nicl to jdk_foreign.
>>
>> Most of the webrev is straightforward and IDE generated - however 
>> there were changes that I had to do manually, in cpp files, makefiles 
>> and other configuration files, as well as in some jextract hard coded 
>> constants.
>>
>> Webrev:
>>
>> http://cr.openjdk.java.net/~mcimadamore/panama/foreign_rename/
>>
>> Cheers
>> Maurizio
>>
>>



More information about the panama-dev mailing list