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