RFR: JDK-8198751 Refactor SetupNativeCompilation to take NAME and TYPE

Tim Bell tim.bell at oracle.com
Wed Feb 28 16:38:14 UTC 2018


Magnus:

> 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

Looks good.

/Tim




More information about the build-dev mailing list