Skip to content

Fix C++20 issues#13320

Merged
RytoEX merged 2 commits into
obsproject:masterfrom
RytoEX:fix-cpp-20-issues
May 1, 2026
Merged

Fix C++20 issues#13320
RytoEX merged 2 commits into
obsproject:masterfrom
RytoEX:fix-cpp-20-issues

Conversation

@RytoEX
Copy link
Copy Markdown
Member

@RytoEX RytoEX commented Apr 13, 2026

Description

Fix an include issue for C++20.

Change lambdas from implicitly capturing this to explicitly capturing this as needed.

Motivation and Context

Want to keep OBS Studio ready for C++20 and eventually build with C++20.

How Has This Been Tested?

Tested locally on Windows 11 with:

See test builds with the original messy branch:

As written on the obs-websocket PR, we may have to globally switch obs-studio to C++20 while forcing obs-websocket to C++17 until we can resolve the websocketpp issue or replace it.

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:

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

@RytoEX RytoEX requested review from PatTheMav and Warchamp7 April 13, 2026 19:30
@RytoEX RytoEX self-assigned this Apr 13, 2026
@RytoEX RytoEX added kind/bug Categorizes issue or PR as related to a bug. kind/cleanup Non-breaking change which makes code smaller or more readable labels Apr 13, 2026
@RytoEX RytoEX merged commit 085a51a into obsproject:master May 1, 2026
15 checks passed
@RytoEX RytoEX deleted the fix-cpp-20-issues branch May 1, 2026 18:09
@RytoEX RytoEX added this to the OBS Studio 32.2 milestone May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug. kind/cleanup Non-breaking change which makes code smaller or more readable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants