Update index.js #37

sunjun · 2020-07-03T14:20:43Z

修复内存泄漏，一直增长

zhw2590582 · 2020-07-04T02:08:18Z

this.videoframes.length - 1 > index这个判断只是为了确保播放帧的下标在帧数组之内，就算改为>=对于后面的释放也是没影响的，我不用>=的原因是为了最起码让this.videoframes能保留至少一帧，这个也对内存没多大影响。
而且this.videoframes的长度和this.timestamps是一一对应的，要改的话也是要一起改为>=。
想加快内存释放可以考虑降低options.videoChunk。
但至于为什么还是一直内存过大，我还没找到原因，最近也没怎么去研究（逃）

sunjun · 2020-07-04T08:14:43Z

我是使用直播播放的时候测试出来的这样改的
内存释放那一部分是因为这个条件不满足所以一直没有被调用
这样改的话内存释放那部分代码就可以被调用了所以内存就不会一直增长了
具体你可以测试一下，
内存变大的原因主要原因就是一直申请而没有释放，没有释放的话就看一下释放那部分代码为啥没有被调用

Update index.js

b6ba865

修复内存泄漏，一直增长

sunjun mentioned this pull request Jul 3, 2020

视频解码方案 #8

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update index.js #37

Update index.js #37

sunjun commented Jul 3, 2020

zhw2590582 commented Jul 4, 2020

sunjun commented Jul 4, 2020 •

edited

Loading

Update index.js #37

Are you sure you want to change the base?

Update index.js #37

Conversation

sunjun commented Jul 3, 2020

zhw2590582 commented Jul 4, 2020

sunjun commented Jul 4, 2020 • edited Loading

sunjun commented Jul 4, 2020 •

edited

Loading