Hi, Petr,
the fix looks fine. Could you add more details about the failure to the
bug comments, please?
Thanks,
Artem
On 8/26/2013 2:54 PM, Petr Pchelko wrote:
Hello, AWT Team.
This is a reminder. Could somebody please review this fix.
With bet regards. Petr.
On Aug 15, 2013, at 2:22 PM, Petr Pchelko wrote:
Hello, AWT Team.
Please review the following fix:
http://cr.openjdk.java.net/~pchelko/8015453/webrev.00/
The bug:
The test fails because of an IOException.
The problem:
It is a regression of the following fix:
http://hg.openjdk.java.net/jdk8/awt/jdk/rev/96750ebc769b
It that fix the the DataTransfer.translateBytesOrStream was splitted to
translateBytes and translateStream,
however one if clause was from the original method was not moved to the
translateStream method, so we were trying
to translate an object incorrectly.
The solution:
The missing if clause was added to it's place.
With best regards. Petr.
