looks fine.
-phil.
On 2/2/2010 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,
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, Ig
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 s
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 wro
I will add/open-existing regression test for that, just like I'll do for
the other fixes I sent today.
/Roman
Am Montag, den 30.11.2009, 20:52 +0100 schrieb Roman Kennke:
> This is a relatively simple fix for 6896335. When I did the change to
> GraphicsEnvironment initialization, I missed this s
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