alex/nih-plug
1
0
Fork 0
Code Activity

Replace index based for loop

Browse source 
This initially did a linear search within the loop so iterating over the
collection wasn't possible. Now we need to use a hashmap anyways, so
this can be simplified again.
This commit is contained in:
Robbert van der Helm 2022-03-17 23:00:15 +01:00
parent 22b51f50bb
commit 8077f10c27
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/wrapper/vst3/param_units.rs
View file

@ -93,12 +93,7 @@ impl ParamUnits {
.map(|(unit_id, (group_name, _))| (*group_name, unit_id as i32 + 1)) .map(|(unit_id, (group_name, _))| (*group_name, unit_id as i32 + 1))
.collect(); .collect();
for unit_id in 0..groups_units.len() { for (group_name, unit) in &mut groups_units {
// We'll do an index-based loop with some clones because here it doesn't matter much for
// performance anyways and otherwise it's a bit difficult to keep the borrow checker
// happy ehre
let group_name = groups_units[unit_id].0;
// If the group name does not contain any slashes then the unit's parent should stay at // If the group name does not contain any slashes then the unit's parent should stay at
// the root unit // the root unit
if let Some(sep_pos) = group_name.rfind('/') { if let Some(sep_pos) = group_name.rfind('/') {
@ -106,7 +101,7 @@ impl ParamUnits {
let parent_unit_id = *vst3_unit_id_by_group_name let parent_unit_id = *vst3_unit_id_by_group_name
.get(parent_group_name) .get(parent_group_name)
.ok_or("Missing parent group")?; .ok_or("Missing parent group")?;
groups_units[unit_id].1.parent_id = parent_unit_id; unit.parent_id = parent_unit_id;
} }
} }