Skip to content

UI: Fix crash when closing missing files window#4544

Merged
jp9000 merged 1 commit into
obsproject:masterfrom
cg2121:fix-missing-files-crash
Apr 19, 2021
Merged

UI: Fix crash when closing missing files window#4544
jp9000 merged 1 commit into
obsproject:masterfrom
cg2121:fix-missing-files-crash

Conversation

@cg2121

@cg2121 cg2121 commented Apr 17, 2021

Copy link
Copy Markdown
Contributor

Description

A crash would occur when clicking the 'x' button in the missing files
dialog. This seemed to only happen in debug mode and using QT 5.15.0.

Closes #4363
Fixes #4359

Motivation and Context

Fixes crash that I found.

How Has This Been Tested?

Closed the dialog and made sure it didn't crash.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

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.

@WizardCM WizardCM added the kind/bug Categorizes issue or PR as related to a bug. label Apr 17, 2021
@WizardCM

Copy link
Copy Markdown
Member

To confirm, this fixes #4359?

@cg2121

cg2121 commented Apr 17, 2021

Copy link
Copy Markdown
Contributor Author

It seems to be the same issue, so yes it should fix it.

@cg2121 cg2121 force-pushed the fix-missing-files-crash branch from 8b899ba to 39336cd Compare April 18, 2021 02:40
cg2121 added a commit to cg2121/obs-studio that referenced this pull request Apr 18, 2021
A crash would occur when clicking the 'x' button in the missing files
dialog. This seemed to only happen in debug mode and using QT 5.15.0.

Closes obsproject#4363
Fixes obsproject#4544
@cg2121 cg2121 force-pushed the fix-missing-files-crash branch from 39336cd to 03f2b95 Compare April 18, 2021 06:50
A crash would occur when clicking the 'x' button in the missing files
dialog. This seemed to only happen in debug mode and using QT 5.15.0.

Closes obsproject#4363
Fixes obsproject#4359
@cg2121 cg2121 force-pushed the fix-missing-files-crash branch from 03f2b95 to 806380d Compare April 18, 2021 06:53
@cg2121

cg2121 commented Apr 18, 2021

Copy link
Copy Markdown
Contributor Author

Updated to remove the lambda, as it is not needed.

@jp9000 jp9000 merged commit 44ee911 into obsproject:master Apr 19, 2021
@WizardCM WizardCM added this to the OBS Studio 27.0 milestone Apr 19, 2021
@cg2121 cg2121 deleted the fix-missing-files-crash branch November 8, 2021 15:54
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI: Crash on missing files close button press

3 participants