[Bug 748259] [PATCH] New audio/video level element

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=748259

--- Comment #7 from Vivia Nikolaidou <vivia at ahiru.eu> ---
@Stefan: It's not easy to make this share code with the existing level element.
One reason is that alevel requires both audio and video to pass through it. The
other is the completely different timing. Basically, the way they are now they
only share part of the calculations, which is two functions IIRC. If they
became merged (video pad made request, and requesting it would raise a flag?),
looks like the biggest part of the alevel code would become nested under an
if{...} ... so I think I wouldn't like that... @slomo what do you think? :)

Unless we ... patch all the demuxers?... and make them send metadata, that
would solve a lot of tricky timing issues. However, imagine a scenario where
your data doesn't come from one single muxer but from several different video
sources which then pass through a compositor (which is part of what Heinrich
describes). Then we'd also need to patch compositor. And I really don't know
what more to patch, looks to me like we'd need to patch all the things [see
meme] :)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.