asticode / go-astits

78%

push

github

fix(packet_pool): don't drop a payload that is flushed by the same packet that completes a single-packet PES (#78)

* fix(packet_pool): do not drop a payload flushed together with a complete single-packet PES

When a packet both flushes the previous payload and completes a PES on its own
(bounded PES that fits in one packet, added in #72), the second assignment to
ps overwrote the first one, so the previous payload was silently dropped.

This happens in practice with GStreamer's mpegtsmux, which writes PES length 0
only for video PES larger than 65535 bytes: a large IDR access unit followed by
a tiny P frame makes the IDR disappear, with no demuxer error.

Keep the newly started payload queued in that case; it is flushed on the next
payload start, as before #72 for this specific sequence.

6 of 6 new or added lines in 1 file covered. (100.0%)

3588 of 4588 relevant lines covered (78.2%)

16.68 hits per line