Looks good. Nit: why is there an empty line? + jlong fmt8; + + jlong fmt32;
-- Chris On May 15, 2013, at 1:21 PM, Morris Meyer <morris.me...@oracle.com> wrote: > Folks, > > Could I get a review for these two small changes in src/solaris/native. This > is to fix the nightly CTW testing crashes on Solaris caused by a library SEGV > internal to X11 that occurs during class initialization when the display is > NULL. > > I've tested this patch on SfBay with JPRT and with the CTW tests on Solaris > x86 and Sparc. > > Thanks much, > > --morris > > JBS - https://jbs.oracle.com/bugs/browse/JDK-7196866 > WEBREV - http://cr.openjdk.java.net/~morris/7196866
