Add default case to facebook::react::displayModeToInt()
#48711
+2
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
In the
displayModeToInt()
function, there is no default case defined which is causing the following warning in React Native Windows when trying to build on the New Architecture:Adding the default case removes the warning and resolves the issue. Not sure if using the -1 value in this case is appropriate.
Changelog:
[General] [Fixed] - Add default case to
displayModeToInt()
functionTest Plan:
Tested on React Native Windows New Arch application and was able to build successfully.