Looks fine to me. But I am not an expert here. And I wonder why the <p>
tag is so specific, can we get the similar issue if some other tags will
be used instead?
On 22.03.16 11:35, Rajeev Chamyal wrote:
Hello All,
Gentle reminder.
Please review the fix.
Bug : https://bugs.openjdk.java.net/browse/JDK-8150225
Webrev: http://cr.openjdk.java.net/~rchamyal/8150225/webrev.00/
Regards,
Rajeev Chamyal
-----Original Message-----
From: Rajeev Chamyal
Sent: 09 March 2016 15:58
To: Sergey Bylokhov; Alexander Scherbatiy; swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> [9] Review request for JDK-8150225
api/javax_swing/text/AbstractWriter/index_indent failed
Hello Sergey,
I have run JCK tests for HTMLWriter and AbstractWriter with this fix and all
passed.
Regards,
Rajeev Chamyal
-----Original Message-----
From: Sergey Bylokhov
Sent: 09 March 2016 15:54
To: Rajeev Chamyal; Alexander Scherbatiy; swing-dev@openjdk.java.net
Subject: Re: <Swing Dev> [9] Review request for JDK-8150225
api/javax_swing/text/AbstractWriter/index_indent failed
Hi, Rajeev.
Please confirm that there are no new issues in the jck after this fix.
On 09.03.16 12:18, Rajeev Chamyal wrote:
Hello All,
Please review the following fix for Jdk9:
Bug : https://bugs.openjdk.java.net/browse/JDK-8150225
Webrev: http://cr.openjdk.java.net/~rchamyal/8150225/webrev.00/
<http://cr.openjdk.java.net/~rchamyal/8146276/webrev.00/>
Issue : JCK conformance test failed due to fix for bug JDK-7104635
Fix: Reverted the fix for JDK-7104635 and added a new method in
HTMLWriter.java to check if P tag is within Pre tag.
Decrement indentation is skipped if P tag is with a Pre tag.
Regards,
Rajeev Chamyal
--
Best regards, Sergey.
--
Best regards, Sergey.