Skip to content

Conversation

@cg2121
Copy link
Contributor

@cg2121 cg2121 commented May 31, 2025

Description

We no longer have to manually destroy obs_properties_t.

Motivation and Context

RAII is better

How Has This Been Tested?

Opened properties in different places of OBS to make sure nothing broke.

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 UI/UX Anything to do with changes or additions to UI/UX elements. Code Cleanup Non-breaking change which makes code smaller or more readable labels May 31, 2025
We no longer have to manually destroy obs_properties_t.
@cg2121 cg2121 force-pushed the obsproperties-obsptr branch from 256ad01 to ea1d38e Compare December 29, 2025 20:20
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 UI/UX Anything to do with changes or additions to UI/UX elements.

Projects

Status: Ready For Review

Development

Successfully merging this pull request may close these issues.

3 participants