Skip to content

Commit

Permalink
[WIP] more on that demux fun!
Browse files Browse the repository at this point in the history
tweaked liboggz to reenable forward reconstruction of
vorbis granulepos, which slightly shaves down the demuxing
time which peaks when pulling apart a bunch of tiny vorbis
packets between frames.

Mostly an issue on very slow machines like 32-bit iOS
(tested iPad 3) where it can be ~15ms at best and 30-60ms
spikes are not uncommon.
  • Loading branch information
bvibber committed May 30, 2016
1 parent af94652 commit 66cbdb5
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 5 deletions.
3 changes: 2 additions & 1 deletion demo/demo.js
Original file line number Diff line number Diff line change
Expand Up @@ -837,7 +837,8 @@
maxmem = 1024 * 1024 * 128;
}
//var debugFilter = /setting a timer|ready to draw frame|decode frame|decoded frame|drew frame/;
var debugFilter = /drew frame.*mux: [^0]|audio checkin/;
var debugFilter = /drew frame.*mux: [^0]/;
//var debugFilter = /drew frame.*mux: [^0]|audio checkin/;
if (playerBackend == 'js') {
player = new OGVPlayer({
debug: true,
Expand Down
2 changes: 1 addition & 1 deletion liboggz
2 changes: 1 addition & 1 deletion src/js/OGVPlayer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1360,7 +1360,7 @@ var OGVPlayer = function(options) {
// @todo networkState == NETWORK_LOADING
stream = new StreamFile({
url: self.src,
bufferSize: 512, //4096, // 32768, //65536 * 4,
bufferSize: 32768, //65536 * 4,
onstart: function() {
waitingOnInput = false;
loading = false;
Expand Down
2 changes: 1 addition & 1 deletion src/js/OGVWrapperCodec.js
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,7 @@ var OGVWrapperCodec = (function(options) {
function doProcessData() {
if (inputQueue.length) {
var data = inputQueue.shift();
console.log('demuxing ' + data.byteLength);
//console.log('demuxing ' + data.byteLength);
demuxer.process(data, finish);
} else {
// out of data! ask for more
Expand Down
4 changes: 3 additions & 1 deletion src/js/modules/ogv-demuxer.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,9 @@ function time(func) {
var start = getTimestamp(),
ret;
ret = func();
Module.cpuTime += (getTimestamp() - start);
var delta = (getTimestamp() - start);
Module.cpuTime += delta;
console.log('demux time ' + delta);
return ret;
}

Expand Down

0 comments on commit 66cbdb5

Please sign in to comment.