On Tue, 12 Sep 2023 21:32:58 GMT, Harshitha Onkar <[email protected]> wrote:

> Open sourcing the following tests -
> 
> 1. javax/swing/text/CompositeView/4398059/bug4398059.java
> 2. javax/swing/text/DefaultCaret/4197894/bug4197894.java
> 3. javax/swing/text/DefaultCaret/4203175/bug4203175.java
> 4. javax/swing/text/DefaultEditorKit/4265242/bug4265242.java
> 5. javax/swing/text/DefaultStyledDocument/4472852/bug4472852.java

Marked as reviewed by psadhukhan (Reviewer).

test/jdk/javax/swing/text/DefaultStyledDocument/bug4472852.java line 46:

> 44:         try {
> 45:             doc.insertString(0, "this", null);
> 46:         } catch (BadLocationException e) {

I guess we can do away with this try-catch block and let original exception be 
propagated by throwing exception in main method signature

-------------

PR Review: https://git.openjdk.org/jdk/pull/15693#pullrequestreview-1623776019
PR Review Comment: https://git.openjdk.org/jdk/pull/15693#discussion_r1324026207

Reply via email to