-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
RA2/YR Intro Movie Crash Fix #425
base: develop
Are you sure you want to change the base?
Conversation
Adds the [Intro] and Play=no section to RA2MD.ini to prevent an error that occurs only when a fresh copy of the client goes on a game install that has yet to play the intro movie, causing the game to never open
ForceLowestDetailLevel.SetDefaultIfNonexistent(); | ||
DoubleTapInterval.SetDefaultIfNonexistent(); | ||
ScrollDelay.SetDefaultIfNonexistent(); | ||
PlayIntroSetting.SetDefaultIfNonexistent(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not just contain this single line (PlayIntroSetting) inside #if? No need to repeat the original three lines I think
Optimally this should be fixed in the spawners. |
I think it would still be beneficial to have this fix even if it should be fixed on the spawner side. Imagine a situation where the spawner is not updated but the client is, and the mod author just doesn't know about the issue. We could prevent more issues with fixes on both sides in place. |
I'd agree if there was something preventing us from updating the spawners, but we have full control over all existing YR spawners. We can just notify mod authors to update their spawner. If we start fixing spawner bugs in the client, we could eventually have a massive number of spawner bug fixes on the client side, making the client code bloated and more difficult to maintain than necessary. |
I don't think there's a whole ton of spawner bugs though? @Belonit do you have this issue fixed already in your spawner reimpl? |
Yes. |
i.e., this issue will not exist on the latest spawner, right? If so we could just close this issue with a "won't fix" |
Yes, that's right. But this only applies to |
Adds the [Intro] and Play=no section to RA2MD.ini to prevent an error that occurs only when a fresh copy of the client goes on a game install that has yet to play the intro movie, causing the game to never open