RFR: 8314820: Unify Screen adapter ordinal logic [v2]

Andy Goryachev angorya at openjdk.org
Thu Dec 18 20:37:16 UTC 2025


On Wed, 17 Dec 2025 10:54:34 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. :-)
>
> Marius Hanl has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge branch 'master' of https://github.com/openjdk/jfx into 8314820-unify-screen-adapter
>  - 8314820: Unify Screen adapter ordinal logic

let's see if another comment wakes up the SKARA bot

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

PR Comment: https://git.openjdk.org/jfx/pull/2009#issuecomment-3672121080


More information about the openjfx-dev mailing list