RFR: JDK-8198751 Refactor SetupNativeCompilation to take NAME and TYPE
Erik Joelsson
erik.joelsson at oracle.com
Thu Mar 1 23:11:19 UTC 2018
Looks good.
/Erik
On 2018-02-27 03:15, Magnus Ihse Bursie wrote:
> Currently, SetupNativeCompilation takes as "main" argument either
> LIBRARY := $name, PROGRAM := $name or STATIC_LIBRARY := $name.
>
> This confounds two distinct concepts: the name of the generated
> binary, and the type of binary to create. Instead, we should have a
> NAME argument and a TYPE argument.
>
> I also took the liberty of updating the code to more modern standards
> wrt whitespace, and fixed the "Yoda style" if conditions.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8198751
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8198751-fix-NativeCompilation-NAME-and-TYPE/webrev.01
>
> /Magnus
>
More information about the build-dev
mailing list