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/1989>
This commit is contained in:
Edward Hervey 2024-12-16 04:33:01 +01:00 committed by Backport Bot
parent f0a4cfebd6
commit cd8d2d6d04

View file

@ -329,7 +329,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,