+1
Thanks,
Alexander.
On 09/04/2015 05:43 PM, Alexander Scherbatiy wrote:
The fix looks good to me.
Thanks,
Alexandr.
On 9/3/2015 4:21 PM, Semyon Sadetsky wrote:
Alexanders? Any questions?
On 8/5/2015 3:11 PM, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8014725
webrev: http://cr.openjdk.java.net/~ssadetsky/8014725/webrev.00/
The test is moved from closed to open repo. Initially it was a test
issue: new document html flavor parameters introduced in 7075105
were not taken into account.
After the test is fixed and run an issue was found in the code. In
case InputStream is used as transferable object it is converted to a
number of compatible formats but during this conversion InputStream
can be read many times without resetting its position after each
read. As result the InputStream is exhausted during the first flavor
conversion and then all consequent flavors receive empty content.
Stream reset was added as solution to this.
--Semyon
