Skip to content
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

Detect power saving mode on Android #8871

Merged
merged 3 commits into from
Aug 6, 2016

Conversation

unknownbrackets
Copy link
Collaborator

This doesn't affect games (although, it could), but instead most importantly notifies users.

I'm sure that low battery may notify users anyway, but power saving is more likely not to: I remember on my Galaxy S3 (which I no longer have), it was easy not to notice it was even on. This shows a message on startup when it is on.

Potentially we could send a low battery level to games which might in some rare cases influence how they render or something.

Anyway, this also disables sending reporting CRCs when power is low / in saving mode.

-[Unknown]

This shows a message when it's on, or when power drops to low.
Sometimes users don't realize these are on, and they can slow down
gameplay.  Of course, they can also save battery.
@Anuskuss
Copy link
Contributor

I know you and iOS ain't best friends, but could the Low Power Mode get detected on iOS? When LPM is on it actually underclocks the processor so it might be important to detect on iOS too.

@unknownbrackets
Copy link
Collaborator Author

It sure could. Someone would just need to write the iOS-specific code to call this API and notify the PPSSPP core about low-power mode. It's intentionally been written so that other platforms can integrate easily.

Pull requests accepted.

-[Unknown]

@VIRGINKLM
Copy link
Contributor

Nice one, to me little details like this one make PPSSPP look like a very serious app/project.

@sum2012
Copy link
Collaborator

sum2012 commented Aug 6, 2016

@hrydgard how about this one ?

@hrydgard
Copy link
Owner

hrydgard commented Aug 6, 2016

Oh, was meaning to read it through but forgot about it. Looking at it now.

@hrydgard hrydgard merged commit ef95bb0 into hrydgard:master Aug 6, 2016
@unknownbrackets unknownbrackets deleted the android-power branch August 6, 2016 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants