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

Kevin Rushforth kcr at openjdk.org
Fri Apr 28 13:45:57 UTC 2023


On Fri, 28 Apr 2023 12:58:24 GMT, Nir Lisker <nlisker at openjdk.org> wrote:

>> 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.

I don't mind either way. I was just giving a little context for Ambarish's comment.

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

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


More information about the openjfx-dev mailing list