Stop using .apply() to prevent call stack size range error, use lazy …

…getter
ddennis · Oct 10, 2019 · 22f5ee4 · 22f5ee4
1 parent 09797f5
commit 22f5ee4
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/demux/tsdemuxer.js b/src/demux/tsdemuxer.js
@@ -17,6 +17,7 @@ import SampleAesDecrypter from './sample-aes';
 // import Hex from '../utils/hex';
 import { logger } from '../utils/logger';
 import { ErrorTypes, ErrorDetails } from '../errors';
+import { utf8ArrayToStr } from './id3';
 
 // We are using fixed track IDs for driving the MP4 remuxer
 // instead of following the TS PIDs.
@@ -710,8 +711,10 @@ class TSDemuxer {
                 pts: pes.pts,
                 payloadType: payloadType,
                 uuid: uuidStrArray.join(''),
-                userData: String.fromCharCode.apply(null, userDataPayloadBytes),
-                userDataBytes: userDataPayloadBytes
+                userDataBytes: userDataPayloadBytes,
+                get userData () {
+                  return utf8ArrayToStr(new Uint16Array(this.userDataBytes).buffer);
+                }
               });
             }
           } else if (payloadSize < expGolombDecoder.bytesAvailable) {