Need reviewer - warnings in serviceability code
Kelly O'Hair
Kelly.Ohair at Sun.COM
Mon Jul 21 11:05:57 PDT 2008
Just some fixes to remove compiler warnings:
6725543: Compiler warnings in serviceability native code
http://bugs.sun.com/view_bug.do?bug_id=6725543
Webrev: http://javaweb.sfbay.sun.com/~ohair/webrevs/jdk7/fix6725543-serviceability/webrev/
or see attached patch file, changes are relatively localized, easy to understand.
Wish I had a way of posting a webrev :^(
Changes include removing some unused vars, adding some casts (mostly on
on strlen()), and renaming INT32 and UINT32 typedefs to avoid conflicts
on windows, which also defines these typedefs.
Changed use of INT32 and UINT32 in transport/socket code to NETLONG and NETULONG
since these are really "network longs" as far as I can tell.
Just using INT32 and UINT32 on Windows may make the most sense, but these files are
in the 'share' section and should be platform independent.
-kto
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: jdk.patch
Url: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20080721/0e531010/attachment.ksh
More information about the serviceability-dev
mailing list