[OpenJDK 2D-Dev] RFR(XS): 8213944: Fix AIX build after the removal of Xrandr.h and add a configure check for it

Aleksey Shipilev shade at redhat.com
Thu Nov 15 16:31:21 UTC 2018


On 11/15/2018 05:06 PM, Volker Simonis wrote:
> can I please have a review for the following small change:
> 
> http://cr.openjdk.java.net/~simonis/webrevs/2018/8213944/

*) I tested it on platform without libxrandr-dev installed, and configure reported the failure
appopriately.

*) Indent looks off at L2207 here:

2205 static char *get_output_screen_name(JNIEnv *env, int screen) {
2206 #ifdef _AIX
2207         return NULL;
2208 #else
2209     if (!awt_XRRGetScreenResources || !awt_XRRGetOutputInfo) {


Thanks,
-Aleksey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20181115/e5917e0d/signature.asc>


More information about the 2d-dev mailing list