The build change looks fine, thanks for fixing this!
/Erik
On 2015-05-22 02:42, Roger Riggs wrote:
Oops, got the wrong host:
Webrev:
http://cr.openjdk.java.net/~rriggs/webrev-fix-all-warnings-8074818/
Issues:
8074818: Resolve disabled warnings for libjava
8080007: Stop ignoring warnings for libjava
Thanks, Roger
On 5/21/15 8:34 PM, Ivan Gerasimov wrote:
Hi Roger
On 22.05.2015 0:09, Roger Riggs wrote:
Please review these native code and build changes to clear
compilation warnings.
Most are due to mixing unsigned types with signed types or providing
the correct type to an invoked function.
The code changes look good!
Webrev:
http://bussund0416.us.oracle.com/~rriggs/webrev/webrev-fix-all-warnings-8074818/
I guess, this link may not be accessible from outside Oracle.
Sincerely yours,
Ivan
Issues:
8074818: Resolve disabled warnings for libjava
8080007: Stop ignoring warnings for libjava
JPRT in progress for product and fastdebug builds on solaris, linux,
macosx, windows,
and embedded.
Thanks, Roger