Skip to content
Snippets Groups Projects

macosx: Refactor setupVoutForWindow in VLCVideoOutputProvider

Merged Claudio Cambra requested to merge claucambra/vlc:work/refactor-setupVoutForWindow into master

This PR's main changes regarding setupVoutForWindow are:

  1. It moves much of the introspection into mainInstance.libraryWindow out of VLCVideoOutputProvider and into VLCLibraryWindow, where the actions that need doing to setup/unsetup video playback called by the setter of hasActiveVideo (which we always call regardless of the video window type)
  2. It moves the creation of the borderless window (video wallpaper / no decoration window) into its own function, reducing the complexity of setupVoutForWindow
  3. Cleans up redundant conditional cases and unnecessary assignments

Signed-off-by: Claudio Cambra claudio.cambra@gmail.com

Edited by Claudio Cambra

Merge request reports

Merge request pipeline #252069 passed

Merge request pipeline passed for 1eb10010

Approved by

Merged by Hugo Beauzée-LuyssenHugo Beauzée-Luyssen 2 years ago (Aug 14, 2022 8:39pm UTC)

Merge details

  • Changes merged into master with 1eb10010.
  • Deleted the source branch.

Pipeline #252189 passed

Pipeline passed for 1eb10010 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
Please register or sign in to reply
Loading