[OpenJDK 2D-Dev] RFR [XS] - 8203355 : MITSHM define guarding missing

Baesken, Matthias matthias.baesken at sap.com
Tue May 22 06:33:42 UTC 2018

Hi Phil, thanks for looking into it.

Can I get a second review ?

Best regards, Matthias

From: Phil Race [mailto:philip.race at oracle.com]
Sent: Freitag, 18. Mai 2018 17:37
To: Baesken, Matthias <matthias.baesken at sap.com>; 2d-dev <2d-dev at openjdk.java.net>
Subject: Re: [OpenJDK 2D-Dev] RFR [XS] - 8203355 : MITSHM define guarding missing

Looks fine.

On 05/17/2018 11:49 PM, Baesken, Matthias wrote:
Please review this minor change  that adds  a missing  #ifdef MITSHM
Compile time guarding  to  X11SurfaceData.c



Usually  one would not run into  any problems  because of the  double-guarding by "HEADLESS", but  in  case  that  MITSHM  in not defined (we do this for a special build here locally)
  we run into issues .  So  the added ifdef check cleans up this minor issue .

Thanks and regards, Matthias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20180522/6d83e0fe/attachment.html>

More information about the 2d-dev mailing list