-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Speed hack setting reorganization #16348
Conversation
@@ -875,7 +875,7 @@ static ConfigSetting graphicsSettings[] = { | |||
ConfigSetting("CameraDevice", &g_Config.sCameraDevice, "", true, false), | |||
ConfigSetting("VendorBugChecksEnabled", &g_Config.bVendorBugChecksEnabled, true, false, false), | |||
ConfigSetting("UseGeometryShader", &g_Config.bUseGeometryShader, true, true, true), | |||
ReportedConfigSetting("RenderingMode", &g_Config.iRenderingMode, 1, true, true), | |||
ConfigSetting("SkipBufferEffects", &g_Config.bSkipBufferEffects, false, true, true), |
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.
Hm, this setting can still affect whether it's running well, so maybe should still be reported. It can factor into the recommendations it gives you if you say the game isn't running well, i.e. turn off skip buffer effects.
-[Unknown]
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.
oops, yeah.
Seems like maybe missed some Android-only/Qt-only refactors of the setting name. -[Unknown] |
Indeed so, was just working on it. |
Fixes #16339, though there's likely more to do.
Realizing now I forgot to make the spline thing a bool, will fix. Or I'll just leave it as-is..