-
Notifications
You must be signed in to change notification settings - Fork 208
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
Remove GMS 2.3 checks for profile mode and GMS 2.3 warning upon loading #1000
Conversation
Remove GMS 2.3 checks for profile mode and GMS 2.3 warning upon loading
Download the artifacts for this pull request here: GUI:
CLI: |
/*if (data.GMS2_3 && SettingsWindow.Warn_About_GMS23) | ||
{ | ||
this.ShowWarning("This game was built using GameMaker Studio 2.3 (or above). Support for this version is a work in progress, and you will likely run into issues decompiling code or in other places.", "GMS 2.3"); | ||
} | ||
}*/ |
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.
This should be removed, not commented out.
Also, if you want to remove the GMS2.3 warning, you should also probably nuke that from the settings as well, as I don't think Warn_About_GMS23
is used anywhere else.
@@ -113,7 +113,7 @@ public void ReplaceTextInGML(UndertaleCode code, string keyword, string replacem | |||
string codeName = code.Name.Content; | |||
GlobalDecompileContext DECOMPILE_CONTEXT = context is null ? new(Data, false) : context; | |||
|
|||
if (Data.ToolInfo.ProfileMode == false || Data.GMS2_3) | |||
if (Data.ToolInfo.ProfileMode == false) |
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.
You can reduce this to if (!Data.ToolInfo.ProfileMode)
/* if (data.GMS2_3) | ||
{ | ||
this.ShowWarning("Profile mode is not currently supported for GameMaker Studio 2.3 games."); | ||
return; | ||
} | ||
|
||
*/ |
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.
This should be removed, not commented out.
Besides what @Miepee has noticed, I don't see any issues. |
Remove GMS 2.3 checks for profile mode and GMS 2.3 warning upon loading
Description
Remove GMS 2.3 checks for profile mode and GMS 2.3 warning upon loading.
Caveats
I might have missed some checks for profile mode, and there may or may not be reason for that GMS 2.3 warning to still exist.
Notes
Long overdue, similar warnings need to be removed from the scripts.