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

Support playing Audio CD from optical disc menu #835

Merged
merged 3 commits into from
Feb 10, 2025

Conversation

hambre
Copy link
Contributor

@hambre hambre commented Jan 4, 2024

Add support to play an audio CD from the 'Play Optical Disc' menu. The media monitor callback function was extended by a 'forcePlayback' parameter which will override the auto play option for some of the media handlers when started explicitly by the user via the menu.

@hambre hambre force-pushed the hambre/play-audio-cd-from-menu branch from cff4fc0 to 84d35e8 Compare January 4, 2024 23:57
@hambre hambre changed the title Support playing Audio CD from menu Support playing Audio CD from optical disk menu Jan 12, 2024
@hambre hambre changed the title Support playing Audio CD from optical disk menu Support playing Audio CD from optical disc menu Jan 12, 2024
@hambre hambre force-pushed the hambre/play-audio-cd-from-menu branch 2 times, most recently from dff4d1d to f10251a Compare February 14, 2024 17:59
@hambre hambre force-pushed the hambre/play-audio-cd-from-menu branch 2 times, most recently from 150ee6e to f0acb3b Compare January 11, 2025 14:27
@hambre hambre force-pushed the hambre/play-audio-cd-from-menu branch from b972e76 to 8f775c3 Compare January 19, 2025 20:07
Add support to play an audio CD from the 'Play Optical Disc' menu.
The media monitor callback function was extended by a 'forcePlayback'
parameter which will override the auto play option for some of the
media handlers when started explicitly by the user via the menu.
@hambre hambre force-pushed the hambre/play-audio-cd-from-menu branch from 8f775c3 to 3a6c57e Compare January 19, 2025 20:09
@linuxdude42
Copy link
Contributor

This works when the media monitor is running, but I can no longer play DVDs when the media monitor isn't running.

@hambre
Copy link
Contributor Author

hambre commented Feb 1, 2025

This works when the media monitor is running, but I can no longer play DVDs when the media monitor isn't running.

Thanks for giving it a try. I will take a look and try to fix the issue.

@hambre
Copy link
Contributor Author

hambre commented Feb 9, 2025

@linuxdude42 , I fixed DVD playback when media monitor is not running. Could you have another look, please?

Convert an old style "typedef" statement into a modern "using"
statement.  Change made by clang-tidy.

https://clang.llvm.org/extra/clang-tidy/checks/modernize/use-using.html
@linuxdude42 linuxdude42 merged commit af063eb into MythTV:master Feb 10, 2025
10 of 11 checks passed
@hambre
Copy link
Contributor Author

hambre commented Feb 10, 2025

Great, thanks for merging it!

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.

2 participants