There is actually a test for the previous bug - called bug4496801.java,
but it is in closed.
I suggest it be opened as part of this fix.
Also run any (all) related Swing regression tests that might cover this
area.
I am a little nervous that since the original fix was 13 years ago that some
code out there may rely on the 'incorrect' fix by now ..
-phil.
On 03/21/2016 07:47 AM, Sergey Bylokhov wrote:
Should the test also cover the html based string(and cover the changes
in HTMLDocument)?
On 16.03.16 21:06, Alexander Scherbatiy wrote:
The fix looks good to me.
Just a small comment: if it is a new test it probably should not contain
1998 year in the copyright.
Thanks,
Alexandr.
On 14/03/16 20:48, Semyon Sadetsky wrote:
Hello,
Please review fix for JDK9:
bug: https://bugs.openjdk.java.net/browse/JDK-8151015
webrev: http://cr.openjdk.java.net/~ssadetsky/8151015/webrev.00/
It is regression of 4496801. The fix for 4496801 was wrong and the
correct fix should be taking into account the implied character at the
end of the document. Reverting 4496801 fixes 8151015. Also the correct
fix for 4496801 is provided.
--Semyon
