Skip to content

frontend: Update invokeMethod to use functions#13344

Open
Warchamp7 wants to merge 1 commit intoobsproject:masterfrom
Warchamp7:remove-string-invokes
Open

frontend: Update invokeMethod to use functions#13344
Warchamp7 wants to merge 1 commit intoobsproject:masterfrom
Warchamp7:remove-string-invokes

Conversation

@Warchamp7
Copy link
Copy Markdown
Member

Description

Updates all frontend uses of invokeMethod with function pointers or lambdas.

Important

This PR is blocked until we drop support for Ubuntu 24.04 as this style of calling invokeMethod is not supported on Qt 6.5

Motivation and Context

Makes it easier to update or refactor code without having to worry about string references to slot functions.

How Has This Been Tested?

Compiled OBS and performed a few basic tests. I have not tested thoroughly yet.

Types of changes

  • 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.

@Warchamp7 Warchamp7 force-pushed the remove-string-invokes branch from bc279ba to 1abf29d Compare April 20, 2026 20:58
@Warchamp7 Warchamp7 added the kind/cleanup Non-breaking change which makes code smaller or more readable label Apr 21, 2026
@Warchamp7 Warchamp7 added this to the OBS Studio 33.0 milestone May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

1 participant