ok with me.
-igor
On 2/2/10 1:40 PM, Roman Kennke wrote:
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