Merge pull request #71 from charlesneimog/fix-ambi_roomsim_setWallAbsCoeff

jananifernandez · web-flow · commit e2080d032b40 · 2025-04-05T15:36:09.000+03:00
fix assertion on `ambi_roomsim_setWallAbsCoeff`
diff --git a/examples/src/ambi_roomsim/ambi_roomsim.c b/examples/src/ambi_roomsim/ambi_roomsim.c
@@ -332,7 +332,7 @@ void ambi_roomsim_setRoomDimZ(void* const hAmbi, float newValue)
 void ambi_roomsim_setWallAbsCoeff(void* const hAmbi, int xyz_idx, int posNeg_idx, float new_value)
 {
     ambi_roomsim_data *pData = (ambi_roomsim_data*)(hAmbi);
-    saf_assert(xyz_idx<4, "xyz_idx indicates each spatial axis, so cannot exceed 4");
+    saf_assert(xyz_idx<3, "xyz_idx indicates each spatial axis, so cannot exceed 3");
     saf_assert(posNeg_idx==0 || posNeg_idx==1, "posNeg_idx is a bool");
     if(new_value!=pData->abs_wall[2*xyz_idx+posNeg_idx]){
         pData->abs_wall[2*xyz_idx+posNeg_idx] = new_value;

Original file line number	Diff line number	Diff line change
`@@ -332,7 +332,7 @@ void ambi_roomsim_setRoomDimZ(void* const hAmbi, float newValue)`
`332`	`332`	`void ambi_roomsim_setWallAbsCoeff(void* const hAmbi, int xyz_idx, int posNeg_idx, float new_value)`
`333`	`333`	`{`
`334`	`334`	`ambi_roomsim_data pData = (ambi_roomsim_data)(hAmbi);`
`335`		`- saf_assert(xyz_idx<4, "xyz_idx indicates each spatial axis, so cannot exceed 4");`
	`335`	`+ saf_assert(xyz_idx<3, "xyz_idx indicates each spatial axis, so cannot exceed 3");`
`336`	`336`	`saf_assert(posNeg_idx==0 \|\| posNeg_idx==1, "posNeg_idx is a bool");`
`337`	`337`	`if(new_value!=pData->abs_wall[2*xyz_idx+posNeg_idx]){`
`338`	`338`	`pData->abs_wall[2*xyz_idx+posNeg_idx] = new_value;`