+1 (at least for the jdk code)
Thanks Erik
On 02/03/2016 08:50 AM, Erik Joelsson wrote:
The library libjimage.so consists of C++ source files, but is
currently setup to compile using CFLAGS_JDKLIB instead of
CXXFLAGS_JDKLIB. On Solaris, these variables contain quite a few
differences due to the compilers taking very different arguments. This
makes the build process very noisy in the build log.
When correcting the flags, the proper warnings were also enabled. I
took the liberty of fixing the warning complaining about the variable
"i" hiding a variable in the outer scope.
Bug: https://bugs.openjdk.java.net/browse/JDK-8148955
Webrev: http://cr.openjdk.java.net/~erikj/8148955/webrev.jdk.01/
/Erik