Skip to content

UI, libobs: Fix not handled in switch warnings#8357

Merged
WizardCM merged 1 commit intoobsproject:masterfrom
cg2121:fix-switch-warnings
Mar 25, 2023
Merged

UI, libobs: Fix not handled in switch warnings#8357
WizardCM merged 1 commit intoobsproject:masterfrom
cg2121:fix-switch-warnings

Conversation

@cg2121
Copy link
Copy Markdown
Contributor

@cg2121 cg2121 commented Feb 27, 2023

Description

These warnings were being spit out by GCC on Linux.

Motivation and Context

Less spammy compiling

How Has This Been Tested?

Compiled OBS

Types of changes

  • Code cleanup (non-breaking change which makes code smaller or more readable)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@cg2121 cg2121 added the Code Cleanup Non-breaking change which makes code smaller or more readable label Feb 27, 2023
Comment thread UI/qt-wrappers.cpp
Comment thread UI/qt-wrappers.cpp Outdated
@cg2121 cg2121 force-pushed the fix-switch-warnings branch from ae6e7a6 to 75136f8 Compare February 27, 2023 09:49
@cg2121 cg2121 requested a review from tytan652 February 27, 2023 09:50
Comment thread UI/qt-wrappers.cpp Outdated
@cg2121 cg2121 added this to the OBS Studio 29.1 milestone Mar 21, 2023
These warnings were being spit out by GCC on Linux.
@cg2121 cg2121 force-pushed the fix-switch-warnings branch from 75136f8 to 90e015f Compare March 21, 2023 18:21
@WizardCM WizardCM merged commit 1ac609f into obsproject:master Mar 25, 2023
@cg2121 cg2121 deleted the fix-switch-warnings branch May 1, 2026 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Code Cleanup Non-breaking change which makes code smaller or more readable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants