Ppsspp: Should prevent setting hardware scaling settings that don't work

Created on 20 Jun 2018  路  5Comments  路  Source: hrydgard/ppsspp

Got an email report of a case where the screen when black after fiddling with graphics settings. I got the ppsspp.ini, it had the following interesting settings:

InternalResolution = 2
AndroidHwScale = 5

The device in question, a Kindle Fire 10 HD, only has 1920x1200 resolution which is less than 5x the PSP resolution, which would be 2400x1360 so the user should not have been able to set that hw scaler resolution. And either way, we should catch it with some sanity checks vs the device resolution...

ppssppini.txt

Platform-specific (Android) RELEASE BLOCKER

Most helpful comment

I'm thinking:

  • Round desired_width to closest 4 pixels, to try to work around the thing that looks like a stride issue
  • Default to off (forcibly through changing the setting name), prevent smaller settings than 2x on android devices >= Kitkat
  • If no complaints, just kill the feature entirely

All 5 comments

Related to #11154, #11289, #11102.

Perhaps for now we make the setting ini only and see how that goes?

-[Unknown]

I'm thinking:

  • Round desired_width to closest 4 pixels, to try to work around the thing that looks like a stride issue
  • Default to off (forcibly through changing the setting name), prevent smaller settings than 2x on android devices >= Kitkat
  • If no complaints, just kill the feature entirely

Should we punt the rest of this to 1.8.0?

-[Unknown]

Yeah. I'm gonna try to do a release this week.

Our current checks should have caught this, and I have not been able to reproduce the fault on any other device without disabling the checks and increasing to really extreme resolutions (at which point I got black instead of a crash). I'm gonna close it.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

SwiftBMan picture SwiftBMan  路  3Comments

soredake picture soredake  路  4Comments

Deivmsr picture Deivmsr  路  6Comments

HelloDefender12 picture HelloDefender12  路  6Comments

nassau-tk picture nassau-tk  路  5Comments