On Sun, 24 Aug 2025 20:39:46 GMT, John Hendrikx <[email protected]> wrote:
>> Nir Lisker has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - review comments 2 >> - review comments 1 > > modules/javafx.base/src/main/java/javafx/util/converter/BaseTemporalStringConverter.java > line 86: > >> 84: .withChronology(chrono) >> 85: .withDecimalStyle(DecimalStyle.of(locale)); >> 86: } > > Weird level of indent, perhaps use 4 or 8? why 8? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1880#discussion_r2535721058
