matrix/matrix-react-sdk
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-react-sdk.git synced 2025-11-10 09:22:25 +03:00
Code Activity

Don't recurse on arrayFastResample

Browse Source 
Fixes https://github.com/vector-im/element-web/issues/17136
This commit is contained in:
Travis Ralston
2021-05-03 15:20:29 -06:00
parent 8dbcc85249
commit 5dbe882c77
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/utils/arrays.ts
View File

@@ -36,14 +36,12 @@ export function arrayFastResample(input: number[], points: number): number[] {
}
} else {
// Smaller inputs mean we have to spread the values over the desired length. We
// end up overshooting the target length in doing this, so we'll resample down
// before returning. This recursion is risky, but mathematically should not go
// further than 1 level deep.
// end up overshooting the target length in doing this, but we're not looking to
// be super accurate so we'll let the sanity trims do their job.
const spreadFactor = Math.ceil(points / input.length);
for (const val of input) {
samples.push(...arraySeed(val, spreadFactor));
}
samples = arrayFastResample(samples, points);
}
// Sanity fill, just in case