Hi Prasanta,

Looks good to me too.

On 18/03/2020 21:59, Sergey Bylokhov wrote:
Looks fine.

On 3/17/20 8:54 pm, Prasanta Sadhukhan wrote:
Hi Sergey,

Test updated

http://cr.openjdk.java.net/~psadhukhan/8241078/webrev.1/

Regards
Prasanta
On 17-Mar-20 11:48 PM, Sergey Bylokhov wrote:
HI, Prasanta.

I suggest to limit memory in the test by the "-mx", otherwise
the test will pass on the systems which has the big amount of memory.
I tested 100G, when it pass.

On 3/17/20 6:54 am, Prasanta Sadhukhan wrote:
Hi All,

Please review a fix for an issue where it is seen that OOM errors generated when parsing HTML files with large <pre> tagtext.

This is because of fix done in JDK-8173123 where the entire data array was copied again and again, insteadof the range specified via offset.

Fix is to use the offset in array copying to only copy the range specified

Bug: https://bugs.openjdk.java.net/browse/JDK-8241078

webrev: http://cr.openjdk.java.net/~psadhukhan/8241078/webrev.0/

All javax/swing/text/html and javax/swing/text/DefaultStyleDocument jtreg test still pass after this change.

Regards
Prasanta
--
Regards,
Alexey

Reply via email to