Looks fine.

-phil.

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

http://cr.openjdk.java.net/~mbaesken/webrevs/8203355.0/ <http://cr.openjdk.java.net/%7Embaesken/webrevs/8203355.0/>

https://bugs.openjdk.java.net/browse/JDK-8203355

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


Reply via email to