search

redotvideo / revideo

Create Videos with Code
https://re.video
MIT License
2.58k stars 88 forks source link

perf: decrease decoder memory by limiting decode queue size #236

Closed justusmattern27 closed 3 months ago

justusmattern27 commented 3 months ago

@hkonsti this should be ready for review now. The bug I still had was because we we called decoder.flush() when the file was fully loaded (and we set this.responseFinished=true) However, this is not necessarily the case anymore now because we don't immediately decode all chunks of the file we download. My last commit fixes this.

I think we should still test this a bit before merging & releasing though