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



More information about the build-dev mailing list