Skip to content

Conversation

@RytoEX
Copy link
Member

@RytoEX RytoEX commented Mar 26, 2023

Description

This variable was added in d18b38e (#6799), but it is not used. Let's remove it.

Motivation and Context

Want less warnings.

How Has This Been Tested?

I have not tested this, but I have visually inspected the code of this function and the variable is not used. If CI passes, that should be a sufficient test.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • 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.

This variable was added in d18b38e,
but it is not used. Let's remove it.
@RytoEX RytoEX added Bug Fix Non-breaking change which fixes an issue Code Cleanup Non-breaking change which makes code smaller or more readable labels Mar 26, 2023
@RytoEX RytoEX merged commit 9b270c3 into obsproject:master Mar 26, 2023
@RytoEX RytoEX deleted the fix-unused-variable branch March 26, 2023 04:21
@RytoEX RytoEX added this to the OBS Studio 29.1 milestone Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fix Non-breaking change which fixes an issue 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.

1 participant