Looks OK to me although I just realised there's no bug ID here
FWIW I develop on WIndows, Mac & Linux and I've noticed widely divergent
things that the compilers on these platforms warn about. Warning
free on Linux might not mean warning free on Windows.
Also, assuming you develop on Linux might want to check if any of these
make
the VS compiler less happy about anything.
* src/solaris/native/sun/awt/awt_Font.c
Comparisons with string literals is undefined behavior - keep track of whether
the string should be freed explicitly with a boolean instead.
Gosh .. that code must be from 1996 or thereabouts.
-phil.
On 2/15/14 8:37 AM, Mikael Vidstedt wrote:
Corrected link - this webrev is based on jdk9/client:
http://cr.openjdk.java.net/~mikael/webrevs/jdk-warnings/webrev.01/webrev/
Cheers,
Mikael
On Feb 14, 2014, at 17:54, Mikael Vidstedt <mikael.vidst...@oracle.com> wrote:
All,
A drive-by set of warning fixes:
http://cr.openjdk.java.net/~mikael/webrevs/jdk-warnings/jdk-warnings/webrev.00/
Highlights:
* src/share/native/com/sun/java/util/jar/pack/bands.cpp
Set the size of the array explicitly to increase likelihood of enum and struct array being
in sync. Arguably this should be changed to use the (new) [<enumName>] =
<value> instead.
Initialize all the fields in the (redundant) terminator struct explicitly.
Remove unused macro.
* src/share/native/sun/java2d/opengl/OGLContext.c
Get the prototype for jio_snprintf from jvm.h to address an implicit
declaration.
* src/solaris/native/sun/awt/awt_Font.c
Comparisons with string literals is undefined behavior - keep track of whether
the string should be freed explicitly with a boolean instead.
* src/solaris/native/sun/awt/awt_LoadLibrary.c
The macro is supposed to expand to a void function declaration, but forgets to actually
add the "void".
Cheers,
Mikael