On 9/7/2015 6:47 PM, Sergey Bylokhov wrote:
On 07.09.15 16:26, Semyon Sadetsky wrote:
We cannot guarantee that the used way of the creation of the buffer
strategy will not cause any unchecked exceptions.
We can, because the list of exceptions is described in its docs, all
other exceptions will mean a bug in implementation or in documentation.
But the test is essentially synthetic and it induces a lot of hardware
related code. So to state the above you need to investigate all
exception for all possible implementations. Did you do that?
We don't need to
stumble on them.
On 9/7/2015 2:59 PM, Sergey Bylokhov wrote:
On 04.09.15 17:42, Semyon Sadetsky wrote:
Yes, I thought about that. But flip buffer is potentially allowed to
throw other exceptions caused by the platform.
Wouldn't it be excessive to introduce such unspecified expectation?
Actually expectations is clearly specified in xxx.createBufferStrategy
method. There are only two exceptions AWTException and
IllegalArgumentException. It seems that we cannot get
IllegalArgumentException in the test since numBuffers>1 and caps!=
null. All other possible exceptions are unspecified and this is a bug.
No?
On 9/4/2015 5:01 PM, Sergey Bylokhov wrote:
On 04.09.15 15:12, Semyon Sadetsky wrote:
The original bug was about ClastCastException.
Then probably catch AWTException which is only expected from
createBufferStrategy?. this will cover old and new bug.
With the option the test will check nothing if buffering is not
supported on the test server.
On 9/4/2015 2:40 PM, Sergey Bylokhov wrote:
Hi, Semyon.
Is it really necessary to catch ClassCastException? Can you try to
test this functionality via -Dswing.bufferPerWindow. When this
option
is passed the backbuffer should be created automatically if
supported.
On 04.09.15 14:03, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8134732
webrev: http://cr.openjdk.java.net/~ssadetsky/8134732/webrev.00/
It's a test bug: when the flip buffer is not available on the
platform
its creation attempt causes exception.
Solution: ignore the exception.
--Semyon
