RFR [9] 8044773: Refactor jdk.net API so that it can be moved out of the base module
Chris Hegarty
chris.hegarty at oracle.com
Tue Apr 26 10:02:21 UTC 2016
On 26 Apr 2016, at 10:57, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>
>
> On 2016-04-26 11:51, Chris Hegarty wrote:
>> On 26 Apr 2016, at 10:35, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>>
>>> Hello Chris,
>>>
>>> In general it looks good.
>> Thanks for the review Erik.
>>
>>> Just a couple style [1] nits that I would like to get sorted. In Lib-jdk.net.gmk, the arguments to SetupNativeCompilation should be indented 4 spaces relative to the call (continuation). Also line 32 and 45 needs a space after comma.
>> Got it. I updated the webrev in-place:
>> http://cr.openjdk.java.net/~chegar/8044773/jdk/make/lib/Lib-jdk.net.gmk.html
> Now if you just move the )) back again on line 43, I'm happy.
Oh sorry, I missed this. Done.
http://cr.openjdk.java.net/~chegar/8044773/jdk/make/lib/Lib-jdk.net.gmk.html
-Chris.
More information about the build-dev
mailing list