Support battery percentage display on SDL

CMakeLists.txt

@@ -926,7 +926,6 @@ add_library(Common STATIC
 	Common/SysError.cpp
 	Common/TimeUtil.cpp
 	Common/TimeUtil.h
-	Common/Battery/Battery.h
 )
 
 include_directories(Common)

Common/Battery/AppleBatteryClient.m

@@ -5,7 +5,8 @@
 //  Created by Serena on 24/01/2023.
 //
 
-#include "Battery.h"
+#include "ppsspp_config.h"
+
 #import <Foundation/Foundation.h>
 
 #if PPSSPP_PLATFORM(MAC)
@@ -90,7 +91,7 @@ - (void)setNeedsToUpdateLevel {
 
 @end
 
-
-int getCurrentBatteryCapacity() {
+// TODO: Move this.
+int Apple_GetCurrentBatteryCapacity() {
     return [[AppleBatteryClient sharedClient] batteryLevel];
 }

Common/System/System.h

@@ -213,6 +213,9 @@ enum SystemProperty {
 	SYSPROP_OK_BUTTON_LEFT,
 
 	SYSPROP_MAIN_WINDOW_HANDLE,
+
+	SYSPROP_CAN_READ_BATTERY_PERCENTAGE,
+	SYSPROP_BATTERY_PERCENTAGE,
 };
 
 enum class SystemNotification {

SDL/SDLMain.cpp

@@ -513,6 +513,12 @@ std::vector<std::string> System_GetPropertyStringVec(SystemProperty prop) {
 	}
 }
 
+#if PPSSPP_PLATFORM(MAC)
+extern "C" {
+int Apple_GetCurrentBatteryCapacity();
+}
+#endif
+
 int64_t System_GetPropertyInt(SystemProperty prop) {
 	switch (prop) {
 	case SYSPROP_AUDIO_SAMPLE_RATE:
@@ -543,6 +549,18 @@ int64_t System_GetPropertyInt(SystemProperty prop) {
 		return g_DesktopWidth;
 	case SYSPROP_DISPLAY_YRES:
 		return g_DesktopHeight;
+	case SYSPROP_BATTERY_PERCENTAGE:
+#if PPSSPP_PLATFORM(MAC)
+	// Let's keep using the old code on Mac for safety. Evaluate later if to be deleted.
+		return Apple_GetCurrentBatteryCapacity();
+#else
+		{
+			int seconds = 0;
+			int percentage = 0;
+			SDL_GetPowerInfo(&seconds, &percentage);
+			return percentage;
+		}
+#endif
 	default:
 		return -1;
 	}
@@ -604,15 +622,16 @@ bool System_GetPropertyBool(SystemProperty prop) {
 #if PPSSPP_PLATFORM(MAC)
 	case SYSPROP_HAS_FOLDER_BROWSER:
 	case SYSPROP_HAS_FILE_BROWSER:
-		return true;
 #endif
 	case SYSPROP_HAS_ACCELEROMETER:
 #if defined(MOBILE_DEVICE)
 		return true;
 #else
 		return false;
 #endif
-	default:
+	case SYSPROP_CAN_READ_BATTERY_PERCENTAGE:
+		return true;
+default:
 		return false;
 	}
 }

UI/DebugOverlay.cpp

@@ -453,13 +453,14 @@ void DrawFPS(UIContext *ctx, const Bounds &bounds) {
 		}
 	}
 
-#ifdef CAN_DISPLAY_CURRENT_BATTERY_CAPACITY
-	if (g_Config.iShowStatusFlags & (int)ShowStatusFlags::BATTERY_PERCENT) {
-		char temp[256];
-		snprintf(temp, sizeof(temp), "%s Battery: %d%%", fpsbuf, getCurrentBatteryCapacity());
-		snprintf(fpsbuf, sizeof(fpsbuf), "%s", temp);
+	if (System_GetPropertyBool(SYSPROP_CAN_READ_BATTERY_PERCENTAGE)) {
+		if (g_Config.iShowStatusFlags & (int)ShowStatusFlags::BATTERY_PERCENT) {
+			char temp[256];
+			const int percentage = System_GetPropertyInt(SYSPROP_BATTERY_PERCENTAGE);
+			snprintf(temp, sizeof(temp), "%s Battery: %d%%", fpsbuf, percentage);
+			snprintf(fpsbuf, sizeof(fpsbuf), "%s", temp);
+		}
 	}
-#endif
 
 	ctx->Flush();
 	ctx->BindFontTexture();

UI/GameSettingsScreen.cpp

@@ -33,7 +33,6 @@
 #include "Common/System/Display.h"  // Only to check screen aspect ratio with pixel_yres/pixel_xres
 #include "Common/System/Request.h"
 #include "Common/System/OSD.h"
-#include "Common/Battery/Battery.h"
 #include "Common/System/NativeApp.h"
 #include "Common/Data/Color/RGBAUtil.h"
 #include "Common/Math/curves.h"
@@ -622,11 +621,11 @@ void GameSettingsScreen::CreateGraphicsSettings(UI::ViewGroup *graphicsSettings)
 	});
 
 	graphicsSettings->Add(new ItemHeader(gr->T("Overlay Information")));
-	BitCheckBox *showFPSCtr = graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::FPS_COUNTER, gr->T("Show FPS Counter")));
-	BitCheckBox *showSpeed = graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::SPEED_COUNTER, gr->T("Show Speed")));
-#ifdef CAN_DISPLAY_CURRENT_BATTERY_CAPACITY
-	BitCheckBox *showBattery = graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::BATTERY_PERCENT, gr->T("Show Battery %")));
-#endif
+	graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::FPS_COUNTER, gr->T("Show FPS Counter")));
+	graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::SPEED_COUNTER, gr->T("Show Speed")));
+	if (System_GetPropertyBool(SYSPROP_CAN_READ_BATTERY_PERCENTAGE)) {
+		graphicsSettings->Add(new BitCheckBox(&g_Config.iShowStatusFlags, (int)ShowStatusFlags::BATTERY_PERCENT, gr->T("Show Battery %")));
+	}
 	AddOverlayList(graphicsSettings, screenManager());
 }
 

ios/main.mm

@@ -316,6 +316,10 @@ bool jb_enable_ptrace_hack(void) {
 	}
 }
 
+extern "C" {
+int Apple_GetCurrentBatteryCapacity();
+}
+
 int64_t System_GetPropertyInt(SystemProperty prop) {
 	switch (prop) {
 		case SYSPROP_AUDIO_SAMPLE_RATE:
@@ -324,6 +328,8 @@ int64_t System_GetPropertyInt(SystemProperty prop) {
 			return DEVICE_TYPE_MOBILE;
 		case SYSPROP_SYSTEMVERSION:
 			return g_iosVersionMajor;
+		case SYSPROP_BATTERY_PERCENTAGE:
+			return Apple_GetCurrentBatteryCapacity();
 		default:
 			return -1;
 	}
@@ -381,6 +387,9 @@ bool System_GetPropertyBool(SystemProperty prop) {
 		case SYSPROP_SUPPORTS_HTTPS:
 			return true;
 #endif
+		case SYSPROP_CAN_READ_BATTERY_PERCENTAGE:
+			return true;
+
 		default:
 			return false;
 	}