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

Fix playmus.py example (broken in 2.0.1) #2745

Merged
merged 2 commits into from Oct 6, 2021
Merged

Fix playmus.py example (broken in 2.0.1) #2745

merged 2 commits into from Oct 6, 2021

Conversation

rethanon
Copy link
Contributor

@rethanon rethanon commented Oct 4, 2021

This PR is identical to PR #2738 which I closed as I needed to update base code to prevent automated build test failures.

=====

This PR is to fix an issue with the bundled example file playmus.py which stopped working correctly as of pygame 2.0.1 due to a change in returning False for pygame.mixer.music.get_busy() when music is paused (previously returned True).

Color scheme updated to be easier on the eye
Docs updated to reflect that rewind() is only supported for certain file types
Tidied up the wording in playmus.py a little bit and fixed a typo

Related to a closed issue that was raised under #2734

Copy link
Member

@illume illume left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 Thanks.

@Starbuck5
Copy link
Contributor

For the documentation note, I’d rather have it say that it doesn’t support WAV, than just listing all other file types mixer.music supports.

@rethanon
Copy link
Contributor Author

rethanon commented Oct 5, 2021

For the documentation note, I’d rather have it say that it doesn’t support WAV, than just listing all other file types mixer.music supports.

Done, updated.

Copy link
Contributor

@Starbuck5 Starbuck5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for figuring this one out @rethanon!

@Starbuck5 Starbuck5 linked an issue Oct 6, 2021 that may be closed by this pull request
@Starbuck5 Starbuck5 added this to the 2.0.2 milestone Oct 6, 2021
@Starbuck5 Starbuck5 merged commit 6ca7d61 into pygame:main Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Playmus example is broken
3 participants