<Swing Dev> [9] Review Request: 8177450: javax.swing.text.html.parser.Parser parseScript ignores a character after commend end

Phil Race philip.race at oracle.com
Tue Apr 4 18:13:56 UTC 2017


2122 if(i > 0) { should be "if (i > 0) {" Can you comment on what tests 
have been run to ensure this does not regress anything else ? -phil.



On 03/31/2017 04:28 AM, Mikhail Cherkasov wrote:
> Hello,
>
> Could you please review the fix for jdk9?
>
> The problem is that Parser reads extra symbol after comment or script tag and missed the symbol.
> I added 'continue' after those tags, so it goes to beginning of Parser while loop and process the symbol.
>    
> Bug:https://bugs.openjdk.java.net/browse/JDK-8177450
> Webrev:http://cr.openjdk.java.net/~mcherkas/8177450/9/webrev/
>
> Thanks,
> Mikhail.
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20170404/20349248/attachment.html>


More information about the swing-dev mailing list