Hi Steven, On 07/03/2013 03:19 PM, Steven R. Loomis wrote: > Omair, > I didn't hear back from you on this list, but I will reply here with > more details.
Sorry about the delay, I got busy with unrelated work. > See diffs below. I would be very suspicious of how this > could run without crashing, That's very interesting. Looking at the diff, I am not sure how my patch even compiles. And just to be sure that I was not making a mistake, I tried building today with my patch after removing the jdk/src/share/native/sun/font/layout/. It built fine. > if the tests I mentioned were run (anything > with 'kerning' in its name, using a font that actually uses kerning pairs). I ran through all the jtreg font tests I could see in the JDK. I also ran jdk/test/java/awt/font/TextLayout/KernCrash.java and jdk/test/java/awt/font/TextLayout/KerningLeak.java manually and they both passed. (KernCrash just opened up a window with some text displayed and no errors. I closed it after a few minutes). But maybe I just don't have the right fonts? KernCrash wanted Bistream Vera (Vera.ttf); I passed it the path to the system-install Vera.tff. Thanks, Omair -- PGP Key: 66484681 (http://pgp.mit.edu/) Fingerprint = F072 555B 0A17 3957 4E95 0056 F286 F14F 6648 4681
