AcademySoftwareFoundation / OpenRV

Open source version of RV, the Sci-Tech award-winning media review and playback software.
Other
550 stars 127 forks source link

463: Unanonymised QMutexLocker usage in OCIOIPNode.cpp #476

Closed kenmcgaugh closed 1 month ago

kenmcgaugh commented 1 month ago

Linked issues

Should fix #463

Summarize your change.

Added "lock" variable name to all anonymous QMutexLocker instances to extend their lifespan to the end of their scope. This matches the QMutexLocker usage in other parts of the app (RVConsoleWindow.cpp and QTAudioRenderer.cpp).

Describe the reason for the change

Enabling OCIO was causing frequent crashing when reader threads were set greater than 1, in particular under MacOS Sonoma.

linux-foundation-easycla[bot] commented 1 month ago

CLA Signed


The committers listed above are authorized under a signed CLA.