RFR(XS): 8213944: Fix AIX build after the removal of Xrandr.h and add a configure check for it (original) (raw)

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

More information about the build-dev mailing list