The changes look good to me. Thanks, Dmitry
> On 18 Sep 2018, at 22:49, Phil Race <philip.r...@oracle.com> wrote: > > Thanks, > > However I also need to remove the now obsolete > comment which explained why we had our own copy .. > http://cr.openjdk.java.net/~prr/8210866.1 > > -phil. > > On 09/18/2018 01:14 PM, Sergey Bylokhov wrote: >> Looks fine. >> >> On 18/09/2018 12:50, Phil Race wrote: >>> Bug: https://bugs.openjdk.java.net/browse/JDK-8210880 >>> Webrev : http://cr.openjdk.java.net/~prr/8210866/ >>> >>> As described in the bug, we can rely on the build platform for this file, >>> as we do for most X11 include files. Builds pass on our core platforms. >>> Since this is an X11 protocol file, I expect very few people will find they >>> are missing this and have to install it .. >>> >>> -phil. >> >> >
