Skip to content

[2022.3] Change #ifdef to allow for displayIndex (multi-display) in Unity 2022.3#1686

Merged
jamesmcgill merged 1 commit intodevelopfrom
display-index-for-2022_3
Jun 8, 2023
Merged

[2022.3] Change #ifdef to allow for displayIndex (multi-display) in Unity 2022.3#1686
jamesmcgill merged 1 commit intodevelopfrom
display-index-for-2022_3

Conversation

@reapazor
Copy link
Copy Markdown
Contributor

@reapazor reapazor commented Jun 8, 2023

Description

Alters the #ifdef's that protect the displayIndex work to match supported Unity editor versions. It actually landed in late 2022.2 as well, but it was safer to use the UNITY_2022_3_OR_NEWER condition.

Changes made

#if UNITY_2023_1_OR_NEWER is now #if UNITY_2022_3_OR_NEWER for displayIndex code

@reapazor reapazor requested review from ekcoh, jamesmcgill and jfreire-unity and removed request for ekcoh June 8, 2023 14:04
Copy link
Copy Markdown
Collaborator

@jamesmcgill jamesmcgill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @reapazor. Looks good.

@jamesmcgill jamesmcgill merged commit c55cdd9 into develop Jun 8, 2023
@jamesmcgill jamesmcgill deleted the display-index-for-2022_3 branch June 8, 2023 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants