Here's the updated fix with testcase (I took the testcase from the bugreport and transformed it into a proper jtreg test).
http://cr.openjdk.java.net/~rkennke/6896335/webrev.01/ Is this ok to put back? We also need another reviewer, Igor, Dmitri? /Roman Am Montag, den 30.11.2009, 11:56 -0800 schrieb Phil Race: > sorry to sound like a broken record, fix looks fine, > regression test needed. In this case I don't think we > have one .. I suppose in this and the last case too it > could be that adding noreg-jck might be sufficient but > a regression test would be better for various reasons. > > -phil. > > Roman Kennke wrote: > > This is a relatively simple fix for 6896335. When I did the change to > > GraphicsEnvironment initialization, I missed this small line that's only > > executed in headless mode, leading to weird exceptions... > > > > http://cr.openjdk.java.net/~rkennke/6896335/webrev.00/ > > > > Ok to commit? > > > > /Roman > > > >
