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