Skip to content
Snippets Groups Projects

vout: Use a fair display lock and interrupt wait

Merged Denis Charmet requested to merge typx/vlc:fair-display-lock into master

This merge request is a draft based on my proposal of the TC discussion. This is what was discussed during the workshop plus Romain's clock interruption.

~~While this particular MR doesn't try to execute the resize function in a background UI thread, my tests on a karaoke MKV with "-I dummy -Vopengl" on a 10 years old bad laptop already show promises.

I was never stuck in a window_ReportSize() callback for more than 16 ms (ie my screen refresh rate) in the most extreme case and usually stay under the 500 us so I wonder if there is an actual need to add a background thread to not block the UI.~~

Edited by Denis Charmet

Merge request reports

Merge request pipeline #250095 passed

Merge request pipeline passed for 56918c6c

Merged by Thomas GuillemThomas Guillem 2 years ago (Aug 9, 2022 6:22am UTC)

Loading

Pipeline #250179 passed

Pipeline passed for 56918c6c on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading