Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log running commands and setting options and properties #4926

Merged
merged 2 commits into from
May 25, 2024
Merged

Conversation

low-batt
Copy link
Contributor

This commit will:

  • Change PlayerCore to pass the value of playerCoreCounter when creating the MPVController object

  • Change MPVController to use a logger subsystem name that includes the counter to identify the player the instance is associated with

  • Change MPVController to log commands sent to mpv

  • Change MPVController to log setting of mpv options and properties

  • I have read CONTRIBUTING.md

  • This implements/fixes issue #.


Description:

This commit will:
- Change PlayerCore to pass the value of playerCoreCounter when creating
  the MPVController object
- Change MPVController to use a logger subsystem name that includes the
  counter to identify the player the instance is associated with
- Change MPVController to log commands sent to mpv
- Change MPVController to log setting of mpv options and properties
@low-batt low-batt self-assigned this May 11, 2024
@uiryuu uiryuu merged commit ef3ca8b into develop May 25, 2024
1 check failed
@uiryuu
Copy link
Member

uiryuu commented May 25, 2024

Very useful feature!

@uiryuu uiryuu deleted the mpv-logging branch May 25, 2024 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants