RFR: JDK-8004842 Unify values of boolean make variables set in configure to true/false

David Holmes david.holmes at oracle.com
Mon Feb 6 03:47:25 UTC 2017


Hi Magnus,

On 3/02/2017 11:14 PM, Magnus Ihse Bursie wrote:
> Currently there are three different ways of expressing a boolean value
> in spec.gmk, true/false, yes/no and 1/0. We should only have one and
> since true/false is the most common one, we should stick to that.
>
> Since this bug was opened, most cases has been fixed but two remains:
> USE_PRECOMPILED_HEADER=1

One nit:

hotspot/src/share/vm/precompiled/precompiled.hpp

I don't think we normally document configure args in the source code, 
and I would expect that we can still pass 
USE_PRECOMPILED_HEADERS=true/false as a make arg. So I think this should 
also just change 0 to false.

Thanks,
David

> ENABLE_INTREE_EC=yes
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8004842
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8004842-standardize-booleans-in-build/webrev.01
>
>
> /Magnus
>



More information about the build-dev mailing list