On Mon, 5 Jul 2021 13:06:48 GMT, Jeanette Winzenburg <faste...@openjdk.org> 
wrote:

>> Marius Hanl has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - removed now unused imports
>>  - Review adjustments
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/ChoiceBoxTest.java
>  line 176:
> 
>> 174:             fail("ChoiceBox.setValue() should not throw an exception.");
>> 175:         }
>> 176: 
> 
> why the try? let the exception bubble up as-thrown (just call the formerly 
> misbehaving method) - otherwise we are loosing information as to the origin 
> of the error :)

That was probably to simple to think of for me 😄  
done

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

PR: https://git.openjdk.java.net/jfx/pull/557

Reply via email to