phantop/doukutsu-rs
1
0
Fork 0
mirror of https://github.com/doukutsu-rs/doukutsu-rs synced 2024-11-24 22:52:51 +00:00
Code Issues Releases Wiki Activity

Hopefully this will improve I/O perf on Horizon

Browse source
This commit is contained in:
Alula 2024-08-12 23:08:04 +02:00
parent 87da45192d
commit 4ee73a8793
No known key found for this signature in database
GPG key ID: 3E00485503A1D8BA
1 changed files with 17 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/game/settings.rs
View file

@ -1,3 +1,5 @@
use std::io::{Read, Write};
use crate::framework::context::Context;
use crate::framework::error::GameResult;
use crate::framework::filesystem::{user_create, user_open};
@ -173,8 +175,13 @@ fn default_cutscene_skip_mode() -> CutsceneSkipMode {
impl Settings {
pub fn load(ctx: &Context) -> GameResult<Settings> {
if let Ok(file) = user_open(ctx, "/settings.json") {
match serde_json::from_reader::<_, Settings>(file) {
if let Ok(mut file) = user_open(ctx, "/settings.json") {
// we use a buffer to avoid hammering the OS with small reads/writes
// I/O on settings was particularly VERY SLOW on Horizon (well it's based on microkernel, so IPC overhead...)
let mut buf = Vec::new();
file.read_to_end(&mut buf)?;
match serde_json::from_slice::<Settings>(&buf) {
Ok(settings) => return Ok(settings.upgrade()),
Err(err) => log::warn!("Failed to deserialize settings: {}", err),
}
@ -183,6 +190,14 @@ impl Settings {
Ok(Settings::default())
}
pub fn save(&self, ctx: &Context) -> GameResult {
// see why in load()
let buffer = serde_json::to_vec_pretty(self)?;
user_create(ctx, "/settings.json")?.write_all(&buffer)?;
Ok(())
}
fn upgrade(mut self) -> Self {
let initial_version = self.version;
@ -362,13 +377,6 @@ impl Settings {
self
}
pub fn save(&self, ctx: &Context) -> GameResult {
let file = user_create(ctx, "/settings.json")?;
serde_json::to_writer_pretty(file, self)?;
Ok(())
}
pub fn create_player1_controller(&self) -> Box<dyn PlayerController> {
if self.touch_controls {
return Box::new(TouchPlayerController::new());