RFR: 8314820: Unify Screen adapter ordinal logic
Andy Goryachev
angorya at openjdk.org
Tue Dec 16 16:30:05 UTC 2025
On Tue, 16 Dec 2025 12:42:05 GMT, Marius Hanl <mhanl at openjdk.org> wrote:
> This is a small improvement that was noted back in https://github.com/openjdk/jfx/pull/1200#issuecomment-1671954361 by @mstr2, and was on my list since then.
>
> The idea is to have only one place for assigning `adapter ordinals` – within `Screen`. The corresponding setter `setAdapterOrdinal` can therefore be made private. This improves the separation of responsibilities a little. :-)
Looks like a minor cleanup, no functional changes whatsoever.
Note: next time, you may want to sync up with the latest master branch before making it RFR.
-------------
Marked as reviewed by angorya (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/2009#pullrequestreview-3583960539
More information about the openjfx-dev
mailing list