[9] Review request: 8028616 Htmleditorkit parser doesn't handle leading slash (/) (original) (raw)
Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Fri Jan 10 13:12:26 UTC 2014
- Previous message: [9] Review request: 8028616 Htmleditorkit parser doesn't handle leading slash (/)
- Next message: [9] Review request: 8028616 Htmleditorkit parser doesn't handle leading slash (/)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
It would be good to check the data and position in the handleText callback in the test and pass the test immediately if the check in the handleText is correct. Otherwise, the fix looks good for me.
Thanks, Alexandr.
On 1/10/2014 3:06 PM, dmitry markov wrote:
Hello,
Could you review the fix for jdk9, please? bug: https://bugs.openjdk.java.net/browse/JDK-8028616 webrev: http://cr.openjdk.java.net/~dmarkov/8028616/jdk9/webrev.00/ The method Parser.parseContent() should invoke legalElementContext(), if the parsed text is not wrapped by tags and starts with slash (/). Thanks, Dmitry
- Previous message: [9] Review request: 8028616 Htmleditorkit parser doesn't handle leading slash (/)
- Next message: [9] Review request: 8028616 Htmleditorkit parser doesn't handle leading slash (/)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]