RFR: 8302816: Refactor sorting-related classes [v2]

Nir Lisker nlisker at openjdk.org
Fri Apr 28 13:11:00 UTC 2023


On Fri, 28 Apr 2023 12:46:38 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> The formatter is set to 120 characters in a line, I think it's fine to leave as is.
>
> If you were already modifying this, then sure. I don't think Ambarish's point was about the line being too long. In this case, the rewrapping is on otherwise unmodified lines of code, so is unrelated to your PR.

Alright. I don't mind reverting, but I think that this form is easier to read than the multiline one.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1041#discussion_r1180373762


More information about the openjfx-dev mailing list