Rendering optimizations

Cargo.toml

@@ -2,7 +2,7 @@
 members = ["audio", "common", "input", "io", "render", "text"]
 
 [workspace.package]
-version = "0.2.2"
+version = "0.2.3"
 authors = ["Esther Alter <[email protected]>"]
 description = "A minimalist and ergonomic MIDI sequencer"
 documentation = "https://github.com/subalterngames/cacophony"
@@ -86,7 +86,7 @@ speech_dispatcher_0_9 = ["text/speech_dispatcher_0_9"]
 
 [package]
 name = "cacophony"
-version = "0.2.2"
+version = "0.2.3"
 authors = ["Esther Alter <[email protected]>"]
 description = "A minimalist and ergonomic MIDI sequencer"
 documentation = "https://github.com/subalterngames/cacophony"
@@ -122,7 +122,7 @@ path = "text"
 name = "Cacophony"
 identifier = "com.subalterngames.cacophony"
 icon = ["icon/32.png", "icon/64.png", "icon/128.png", "icon/256.png"]
-version = "0.2.2"
+version = "0.2.3"
 resources = ["data/*"]
 copyright = "Copyright (c) Subaltern Games LLC 2023. All rights reserved."
 short_description = "A minimalist and ergonomic MIDI sequencer."

changelog.md

@@ -1,5 +1,9 @@
 # 0.2.x
 
+## 0.2.3
+
+- Optimized text and rectangle rendering, which reduces CPU usage by around 5%.
+
 ## 0.2.2
 
 - Fixed: There was an input bug where the play/start key (spacebar) was sometimes unresponsive for the first few presses. This is because audio was still decaying from a previous play, meaning that technically the previous play was still ongoing.

common/src/lib.rs

@@ -74,3 +74,8 @@ pub fn get_bytes(path: &Path) -> Vec<u8> {
 pub fn get_default_data_folder() -> PathBuf {
  current_dir().unwrap().join("data")
 }
+
+#[cfg(debug_assertions)]
+pub fn get_test_config() -> ini::Ini {
+ ini::Ini::load_from_file("../data/config.ini").unwrap()
+}

common/src/paths.rs

@@ -38,6 +38,17 @@ impl Paths {
  /// Setup the paths, needs to be be called at least once.
  pub fn init(data_directory_from_cli: &Path) {
  let data_directory = get_data_directory(data_directory_from_cli);
+ PATHS.set(Self::new(&data_directory)).unwrap();
+ }
+
+ /// Returns a new `Paths` that can be used for testing.
+ #[cfg(debug_assertions)]
+ pub fn get_test_paths() -> Self {
+ Self::new(&PathBuf::from("../data"))
+ }
+
+ /// Returns a new `Paths` object.
+ fn new(data_directory: &Path) -> Self {
  let user_directory = match UserDirs::new() {
  Some(user_dirs) => match user_dirs.document_dir() {
  Some(documents) => documents.join("cacophony"),
@@ -60,20 +71,18 @@ impl Paths {
  let export_directory = get_directory("exports", &user_directory);
  let splash_path = data_directory.join("splash.png");
  let default_soundfont_path = data_directory.join("CT1MBGMRSV1.06.sf2");
- PATHS
- .set(Self {
- default_ini_path,
- user_directory,
- user_ini_path,
- text_path,
- soundfonts_directory,
- saves_directory,
- export_directory,
- splash_path,
- default_soundfont_path,
- data_directory,
- })
- .unwrap();
+ Self {
+ default_ini_path,
+ user_directory,
+ user_ini_path,
+ text_path,
+ soundfonts_directory,
+ saves_directory,
+ export_directory,
+ splash_path,
+ default_soundfont_path,
+ data_directory: data_directory.to_path_buf(),
+ }
  }
 
  /// Get a reference to the paths, panics when not initialized.

common/src/view.rs

@@ -199,10 +199,7 @@ impl View {
 
 #[cfg(test)]
 mod tests {
- use crate::time::PPQ_U;
- use crate::view::View;
- use crate::EditMode;
- use ini::Ini;
+ use crate::{get_test_config, EditMode, View, PPQ_U};
 
  const VIEW_T1: u64 = PPQ_U * 133;
 
@@ -250,6 +247,6 @@ mod tests {
  }
 
  fn get_new_view() -> View {
- View::new(&Ini::load_from_file("../data/config.ini").unwrap())
+ View::new(&get_test_config())
  }
 }

io/src/piano_roll/edit_mode_deltas.rs

@@ -77,12 +77,11 @@ impl EditModeDeltas {
 #[cfg(test)]
 mod tests {
  use super::EditModeDeltas;
- use common::PPQ_U;
- use ini::Ini;
+ use common::{get_test_config, PPQ_U};
 
  #[test]
  fn edit_mode_deltas() {
- let e = EditModeDeltas::new(&Ini::load_from_file("../data/config.ini").unwrap());
+ let e = EditModeDeltas::new(&get_test_config());
  assert_eq!(e.quick_time_factor, 4, "{}", e.quick_time_factor);
  assert_eq!(e.precise_time, PPQ_U / 32, "{}", e.precise_time);
  assert_eq!(e.normal_note, 1, "{}", e.normal_note);

render/src/export_panel.rs

@@ -31,6 +31,7 @@ impl ExportPanel {
  position[1] + 1,
  ],
  decaying,
+ renderer,
  );
  let writing = text.get("EXPORT_PANEL_WRITING");
  let writing_label = Label::new(
@@ -39,6 +40,7 @@ impl ExportPanel {
  position[1] + 1,
  ],
  writing,
+ renderer,
  );
  Self {
  panel,
@@ -67,10 +69,7 @@ impl Drawable for ExportPanel {
  + self.panel.background.grid_rect.size[0] / 2
  - w / 2;
  let y = self.panel.background.grid_rect.position[1] + 1;
- let label = Label {
- position: [x, y],
- text: samples,
- };
+ let label = Label::new([x, y], samples, renderer);
  renderer.text(&label, &ColorKey::FocusDefault);
  }
  ExportState::AppendingDecay => {