src/libclient/directrenderer.cpp · 9b876c198e024153c1019f28fb792e853a515c69 · savoirfairelinux / jami-client-qt

1 year ago

videoprovider: cleanup synchronization with the scene graph renderer · 9b876c19

Vladimir Stoiakin authored 1 year ago and

Sébastien Blin committed 1 year ago

Applies some suggestions based on reviews by vstoiakin regarding concurrent resource access and overall structural simplification.
- Create and handoff ownership of QVideoFrames each frame
- use QReadWriteLocks when multiple reads are possible instead of QMutex
- remove the superfluous layer of unique_ptrs on QVideoFrames which are explicitly shared using QExplicitlySharedDataPointer for their underlying data.

This patch is required to prevent crashes when using Qt 6.4 and up.

Gitlab: #938
Change-Id: Ia9a9f0310344bcee47dbd88ec622bc3daa2ce6d7

9b876c19

History

videoprovider: cleanup synchronization with the scene graph renderer

Vladimir Stoiakin authored 1 year ago and

Sébastien Blin committed 1 year ago

Applies some suggestions based on reviews by vstoiakin regarding concurrent resource access and overall structural simplification.
- Create and handoff ownership of QVideoFrames each frame
- use QReadWriteLocks when multiple reads are possible instead of QMutex
- remove the superfluous layer of unique_ptrs on QVideoFrames which are explicitly shared using QExplicitlySharedDataPointer for their underlying data.

This patch is required to prevent crashes when using Qt 6.4 and up.

Gitlab: #938
Change-Id: Ia9a9f0310344bcee47dbd88ec622bc3daa2ce6d7

Code owners

Assign users and groups as approvers for specific file changes. Learn more.