tuxmain/HexoDSP
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Make Matrix::place less crashy if HexoSynth is doing something wrong

Browse source
This commit is contained in:
Weird Constructor 2022-07-14 22:54:23 +02:00
parent cb2baad323
commit 40f8786023
1 changed files with 8 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/matrix.rs
View file

@ -656,6 +656,9 @@ impl Matrix {
/// does not have cycles, otherwise an upload to the DSP thread via /// does not have cycles, otherwise an upload to the DSP thread via
/// [Matrix::sync] will fail. /// [Matrix::sync] will fail.
/// ///
/// If you try to place a cell outside the grid, it will not be placed
/// and just silently ignored.
///
/// You can safely check the DSP topology of changes using /// You can safely check the DSP topology of changes using
/// the convenience function [Matrix::change_matrix] /// the convenience function [Matrix::change_matrix]
/// or alternatively: [Matrix::save_matrix], [Matrix::restore_matrix] /// or alternatively: [Matrix::save_matrix], [Matrix::restore_matrix]
@ -665,6 +668,11 @@ impl Matrix {
pub fn place(&mut self, x: usize, y: usize, mut cell: Cell) { pub fn place(&mut self, x: usize, y: usize, mut cell: Cell) {
cell.x = x as u8; cell.x = x as u8;
cell.y = y as u8; cell.y = y as u8;
if x >= self.w || y >= self.h {
return;
}
self.matrix[x * self.h + y] = cell; self.matrix[x * self.h + y] = cell;
} }