Need reviewer for warning error fix on hprof_init.c

Alan Bateman Alan.Bateman at oracle.com
Sat Sep 15 03:15:36 PDT 2012


On 15/09/2012 02:52, Kelly O'Hair wrote:
>
>  7198327: Fix mac warning error in hprof_init.c
>
> http://cr.openjdk.java.net/~ohair/openjdk8/repo-jdk/webrev/src/share/demo/jvmti/hprof/hprof_init.c.sdiff.html 
> <http://cr.openjdk.java.net/%7Eohair/openjdk8/repo-jdk/webrev/src/share/demo/jvmti/hprof/hprof_init.c.sdiff.html>
>
> The Mac warns on the if test for the port number being > 65535
> made more sense for the port to be a simple int.
>
> -kto
>
Looks okay to me too.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20120915/7fedbcc7/attachment.html 


More information about the serviceability-dev mailing list