JDK 9 RFR of 8038163: Build failure on Mac OS 10.9.2 (Mavericks) due to warning treated as error (original) (raw)
Staffan Larsen staffan.larsen at oracle.com
Fri Mar 21 19:44:48 UTC 2014
- Previous message: JDK 9 RFR of 8038163: Build failure on Mac OS 10.9.2 (Mavericks) due to warning treated as error
- Next message: JDK 9 RFR of 8038163: Build failure on Mac OS 10.9.2 (Mavericks) due to warning treated as error
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Looks good to me.
The version with JNI_TRUE/JNI_FALSE is more correct, but I’m fine with either one.
(note: this review request should have been sent to serviceability-dev).
Thanks, /Staffan
On 21 mar 2014, at 20:41, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
Please review at your convenience:
Issue: https://bugs.openjdk.java.net/browse/JDK-8038163 Patch: http://cr.openjdk.java.net/~bpb/8038163/ My only question would be whether the changed line 291 should instead be this: return (major > 1 || (major == 1 && minor >= 2)) ? JNITRUE : JNIFALSE; Thanks, Brian
- Previous message: JDK 9 RFR of 8038163: Build failure on Mac OS 10.9.2 (Mavericks) due to warning treated as error
- Next message: JDK 9 RFR of 8038163: Build failure on Mac OS 10.9.2 (Mavericks) due to warning treated as error
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]