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

(New) Many WMP9 Video Playback fixes for Yuzusoft VN's #62

Closed
wants to merge 1 commit into from
Closed

Conversation

CreeperofDoom
Copy link
Contributor

This will fix 100% WMP9 Video Playback for many Yuzusoft VN's, Including Riddle Joker. I am very certain this works, excluding even a single command will either cause a black screen, laggy or no audio or it just wont play. (I tested this very well)

@CreeperofDoom
Copy link
Contributor Author

Failed? Nothing is wrong with it, what am i supposed to change here?

@R1kaB3rN
Copy link
Member

Thanks for the pull request. I'll test them in a bit since I own them, and I think it would probably be a good idea to create fixes in GOG or umu because they're available in other storefronts -- GOG and NekoNyan


def main():
"""
"""
Copy link
Member

Choose a reason for hiding this comment

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

It's the empty docstring which is unnecessary. See the linter errors for more details

Copy link
Contributor Author

Choose a reason for hiding this comment

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

and does this mean i have to submit another pull request with those gone in the script

@R1kaB3rN
Copy link
Member

To clarify, this fixes the video playback specifically for the opening and ending videos, correct?

@CreeperofDoom
Copy link
Contributor Author

To clarify, this fixes the video playback specifically for the opening and ending videos, correct?

Yes, ive tested. In the normal playthrough they play with full video and sound. Replaying them through the scene select also has full video and sound

@CreeperofDoom
Copy link
Contributor Author

i will now submit yet another Pull Request with the spaces gone so it can pass the checks.

@CreeperofDoom CreeperofDoom closed this by deleting the head repository Apr 16, 2024
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