On Wed, 1 Jun 2022 18:32:09 GMT, Phil Race <p...@openjdk.org> wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Removed print statements.
>
> src/java.desktop/share/classes/javax/swing/text/html/HTMLEditorKit.java line
> 1123:
>
>> 1121: */
>> 1122: public void handleComment(char[] data, int pos) {
>> 1123: System.out.println("HTMLEditorKit");
>
> And this one ?
Fixed. Had some branching issues that I recently resolved. I lost track of when
this was committed and had to revert back to an older commit to see this change.
> src/java.desktop/share/classes/javax/swing/text/html/parser/DocumentParser.java
> line 34:
>
>> 32: import javax.swing.text.SimpleAttributeSet;
>> 33: import javax.swing.text.html.HTML;
>> 34: import javax.swing.text.html.HTMLEditorKit;
>
> Then there's these changes. OK .. too much .. let's just revert all the files
> that are import only changes.
Reverted all import changes.
> test/jdk/javax/swing/text/html/parser/HtmlTagParserTest.java line 1:
>
>> 1: import java.awt.Dimension;
>
> where's the GPL header on this test ?
Added the header and moved the test to the HTML folder instead since I believe
the process involves classes beyond just the Parser. For example, I found that
the tags and unknown tags are located in the HTML class. I can move it back if
it seems more appropriate since the issue requests changes to the Parser's doc
only.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7446
