-
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
Remove outdated TrueColor setting. #11667
Conversation
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.
Seems reasonable. I'm concerned that people might be unhappy about the timer hack removal in #11666 and this mixes a change for that in, though.
-[Unknown]
Core/Reporting.cpp
Outdated
@@ -458,8 +458,6 @@ namespace Reporting | |||
bool IsSupported() | |||
{ | |||
// Disabled when using certain hacks, because they make for poor reports. | |||
if (g_Config.bTimerHack) |
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, not meant for this PR.
-[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.
thanks, moved to the other PR.
474beca
to
a66ea45
Compare
a66ea45
to
56bc7b1
Compare
Are you using this @Emulatorer or why the thumbs down on this one? |
Yeah I use it in some games but it seems no improvement in performance as i tested the latest git so.. changing thumbs down to thumbs up hehe 😅 |
Really no reason for this to exist anymore. Even if we can support low-bit-depth hardware rendering, noone will normally use it and it will go untested and eventually probably broken. Nobody renders at 16-bit color these days.