cea708mux: Don't push empty services into packets

This is invalid. Only "padding" service 0 can be empty.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/merge_requests/1987>
This commit is contained in:
Edward Hervey 2024-12-16 04:33:01 +01:00
parent 6a8f1bdc61
commit 73663d7352

View file

@ -348,7 +348,7 @@ impl AggregatorImpl for Cea708Mux {
let mut packet = DTVCCPacket::new(state.dtvcc_seq_no & 0x3);
state.dtvcc_seq_no = state.dtvcc_seq_no.wrapping_add(1);
for (_service_no, service) in services.into_iter() {
for (_service_no, service) in services.into_iter().filter(|(_, s)| !s.codes().is_empty()) {
// FIXME: handle needing to split services
gst::trace!(
CAT,